Review Request: Add thumbnail support for .lnk files

Nicholas van Oudtshoorn vanoudt at gmail.com
Mon Aug 2 11:45:24 BST 2010


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

(Updated 2010-08-02 10:45:24.542511)


Review request for kdelibs.


Changes
-------

Redundant Whitespace removed


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 (updated)
-----

  /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/20100802/e2da5d9f/attachment.htm>


More information about the kde-core-devel mailing list