----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112009/ -----------------------------------------------------------
(Updated Aug. 17, 2013, 1:43 p.m.) Review request for KDE Runtime, kdewin and Pali Rohár. Changes ------- Removed all the dynamic loading on the Windows side. Removed the "Cursor/Group Cursor" part on the wrestool side (so it can be added later when Qt is patched). CMake fixes as per comments below. A few style fixes. Renamed icotools_icoutils.cpp to icotools_wrestool.cpp to make it more explicit that wrestool is used as a "backend" on non win32. Note: The beforementioned LNK thumbnailer will go into a different review request, after this one gets in. Description ------- This patch intends to enable Windows exe/dll thumbnailing by using winapi. It derives from the unsubmitted patch from Pali Rohár from https://svn.reviewboard.kde.org/r/5156/ as a starting point. I've made a few adjustments on the original patch, split that patch into a common part and a icoutils-specific part, and then created the winapi-based part to replace the icoutils one on Windows (porting icoutils to windows wasn't going to be easy). Diffs (updated) ----- kioslave/thumbnail/CMakeLists.txt b81339b kioslave/thumbnail/icoutils.h 6468bc1 kioslave/thumbnail/icoutils.cpp 31db85d kioslave/thumbnail/icoutils_common.cpp PRE-CREATION kioslave/thumbnail/icoutils_win.cpp PRE-CREATION kioslave/thumbnail/icoutils_wrestool.cpp PRE-CREATION kioslave/thumbnail/windowsexecreator.h a407982 kioslave/thumbnail/windowsexecreator.cpp 9e24aee kioslave/thumbnail/windowsexethumbnail.desktop f10efef kioslave/thumbnail/windowsimagecreator.h 0b68cc6 kioslave/thumbnail/windowsimagecreator.cpp 08b063d Diff: http://git.reviewboard.kde.org/r/112009/diff/ Testing ------- Tested on a Windows 7 64-bit machine, with intel compiler (32-bit). Tested using ico files and both 32-bit and 64-bit executables and dlls, including jumbo-size icons. I've used QLibrary for all winapi functions in order to avoid issues with MinGW compiler, but I don't have a MinGW setup to check. Not tested on *nix, but the original patch was not changed except for iterating order in the common part and namespacing. File Attachments ---------------- screenshot http://git.reviewboard.kde.org/media/uploaded/files/2013/08/11/Icons.png Thanks, Andrius da Costa Ribas
_______________________________________________ Kde-windows mailing list Kde-windows@kde.org https://mail.kde.org/mailman/listinfo/kde-windows