[Kde-graphics-devel] Review Request 110349: KSnapshot: Do not open unexistent files

[Albert Astals Cid]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110349/
-----------------------------------------------------------

(Updated May 7, 2013, 6:32 p.m.)


Review request for KDE Graphics and Richard Moore.


Description
-------

KSnapshot cannot open unsaved files because it uses the next incremented file number.
Only increment the file name on a new screengrab.
Now it is possible to edit temporary files that get removed after closing. This might confuse users but it still is better than not opening the file at all. (Maybe open temp file read only?)

I would like to commit this and cherry pick to 4.10.

Original patch by Jonathan Marten


This addresses bug 258431.
    http://bugs.kde.org/show_bug.cgi?id=258431


Diffs
-----

  ksnapshot.cpp f5f3e40 

Diff: http://git.reviewboard.kde.org/r/110349/diff/


Testing
-------

'Open with' works. Not only for the first file but for all snapshots. Saved and unsaved.


Thanks,

Maarten De Meyer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-graphics-devel/attachments/20130507/9cae56d5/attachment.html>