----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117131/#review54998 -----------------------------------------------------------
Ship it! Ship It! src/lib/util/kuser_win.cpp <https://git.reviewboard.kde.org/r/117131/#comment38366> You could keep the old code as a fallback here, but i don't really mind - Alexander Richardson On March 30, 2014, 5:35 p.m., Nicolás Alvarez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117131/ > ----------------------------------------------------------- > > (Updated March 30, 2014, 5:35 p.m.) > > > Review request for KDE Frameworks and kdewin. > > > Repository: kcoreaddons > > > Description > ------- > > Implement KUser::faceIconPath on Windows. > > I use an undocumented Windows API > (http://undoc.airesoft.co.uk/shell32.dll/SHGetUserPicturePath.php) that > stores the profile image in a temporary file and returns the path to it. The > previous code was just trying to load that temporary file, and would only > work if another app had created the file recently (such as the control panel > section where the image is changed). > > This only works on Windows Vista and later; on Windows XP the undocumented > API is different, and faceIconPath will just return an empty string. > > > Diffs > ----- > > src/lib/util/kuser_win.cpp 96cf2f0 > > Diff: https://git.reviewboard.kde.org/r/117131/diff/ > > > Testing > ------- > > Compiled with MSVC2010, tested via the new faceicontest on Windows 7. > > > Thanks, > > Nicolás Alvarez > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
