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