Review Request 117454: Implement KUser::faceIconPath for Windows XP
Nicolás Alvarez
nicolas.alvarez at gmail.com
Wed Apr 9 17:06:15 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117454/
-----------------------------------------------------------
Review request for KDE Frameworks.
Repository: kcoreaddons
Description
-------
The undocuented API we're using has a different ordinal and parameters on Windows XP vs Vista/7.
I refactored the code to use a template and structs encapsulating the differences, as otherwise the code became too redundant.
Feedback welcome on identifier names, that's a known hard problem :)
Diffs
-----
src/lib/util/kuser_win.cpp aa48c04
Diff: https://git.reviewboard.kde.org/r/117454/diff/
Testing
-------
Ran faceicontest on Windows 7 and on XP.
Thanks,
Nicolás Alvarez
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140409/36a2b756/attachment.html>
More information about the Kde-frameworks-devel
mailing list