Review Request: Jpeg thumbnailer honouring jpeg rotation info

[Pino Toscano]

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



branches/KDE/4.3/kdebase/runtime/kioslave/thumbnail/CMakeLists.txt
<http://reviewboard.kde.org/r/1444/#comment1489>

    target_link_libraries() is additive, meaning you can simply do:
      target_link_libraries(jpegthumbnail [always used libraries...])
      if (have foobar)
        target_link_libraries(jpegthumbnail [more libraries needed for foobar..])
      endif (have foobar)
    
    Also, there's a target_link_libraries() call more (I guess copy/paste).


- Pino


On 2009-08-30 10:54:10, Jacopo De Simoi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1444/
> -----------------------------------------------------------
> 
> (Updated 2009-08-30 10:54:10)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> This patch makes the jpeg thumbnailer honor jpeg rotation infos stored in exif metadata. 
> The method is quite simple, although I don't like ifdefs, this time they seem to me to be necessary.
> It's my first nontrivial CMake modification; please check that I did not do something stupid there.
> Also, the orientationMatrix method could be ifdeffed if you feel it is necessary; I just didn't want to add another ifdef.
> 
> I am not sure if this is a good solution performance-wise; please comment on that if you have better ideas.
> 
> 
> Diffs
> -----
> 
>   branches/KDE/4.3/kdebase/runtime/kioslave/thumbnail/CMakeLists.txt 1016603 
>   branches/KDE/4.3/kdebase/runtime/kioslave/thumbnail/jpegcreator.h 1016603 
>   branches/KDE/4.3/kdebase/runtime/kioslave/thumbnail/jpegcreator.cpp 1016603 
> 
> Diff: http://reviewboard.kde.org/r/1444/diff
> 
> 
> Testing
> -------
> 
> Works good with a *clean* .thumbnails directory. Cached thumbnails are indeed a problem; not sure how to solve this issue yet.
> 
> 
> Screenshots
> -----------
> 
> Dolphin showing correctly rotated jpegs
>   http://reviewboard.kde.org/r/1444/s/192/
> 
> 
> Thanks,
> 
> Jacopo
> 
>