[KPhotoAlbum] Latest patch

[Robert L Krawitz]

   From: "Jesper K. Pedersen" <blackie at blackie.dk>
   Date: Sun, 9 Jul 2006 13:09:35 +0200
   Cc: Robert L Krawitz <rlk at alum.mit.edu>

   | | 4) An option to allow specifying thumbnail size.
   |
   | Why would you want this? All you have to do is grab the thumbnail view with
   | the middle mouse button and drag, to resize.
   |
   | If it simple is a matter of not having recognized this yet, and you agree
   | that we do not need this options, then please remove this code from the
   | patch.

   Actually looking at your patch, I found that you added a new
   SettingsData for the thumbnail size, it is already in there, namely
   in:

       void setThumbSize( int value );
       int thumbSize() const;

   So I believe your change in this respect should be reverted from the patch.

I removed the corresponding code from SettingsData.cpp, leaving the
change in SettingsData.h.  I've also included the fix for massive
memory consumption when zooming out.

I have not backed out the smooth vs. fast scale for the thumbnails
yet; I'd like to think about that some more (it presumably slows down
thumbnail generation significantly, although admittedly I haven't
timed it).
-------------- next part --------------
A non-text attachment was scrubbed...
Name: new-kphoto-diffs-1.txt.gz
Type: application/x-gunzip
Size: 9111 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kphotoalbum/attachments/20060709/fcd52d80/attachment.bin>