Review Request: Add thumbnail support for .lnk files

[Pali Rohár]

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



/trunk/KDE/kdebase/runtime/kioslave/thumbnail/windowslnkcreator.cpp
<http://reviewboard.kde.org/r/4817/#comment6676>

    after IcoUtils::convertExeToIco() call IcoUtils::convertIcoToPng()
    
    IcoUtils::convertIcoToPng() has better conversation from Windows ICO file to PNG. QImage only load first icon from ICO file, but ICO file can has more then one icon (with higher resolution). And then do not use QImage::scaled(). You can specify size of icon in IcoUtils::convertIcoToPng().
    
    See windowsexecreator.cpp


- Pali


On 2010-08-02 10:45:24, Nicholas van Oudtshoorn wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4817/
> -----------------------------------------------------------
> 
> (Updated 2010-08-02 10:45:24)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> This patch implements thumbnailing support for Microsoft Windows link files (.lnk) via the thumbnail kioslave
> 
> It uses the external application "winepath" - part of the wine package (www.winehq.org) to translate windows paths to their unix equivalents. If said path (or indeed the winepath application) doesn't exist, no thumbnail is drawn.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/runtime/kioslave/thumbnail/windowslnkthumbnail.desktop PRE-CREATION 
>   /trunk/KDE/kdebase/runtime/kioslave/thumbnail/icoutils.h 1158174 
>   /trunk/KDE/kdebase/runtime/kioslave/thumbnail/icoutils.cpp 1158174 
>   /trunk/KDE/kdebase/runtime/kioslave/thumbnail/windowslnkcreator.h PRE-CREATION 
>   /trunk/KDE/kdebase/runtime/kioslave/thumbnail/windowslnkcreator.cpp PRE-CREATION 
>   /trunk/KDE/kdebase/runtime/kioslave/thumbnail/CMakeLists.txt 1158174 
> 
> Diff: http://reviewboard.kde.org/r/4817/diff
> 
> 
> Testing
> -------
> 
> Works for me!
> 
> 
> Thanks,
> 
> Nicholas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100805/85fc794d/attachment.htm>