----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128397/#review97193 -----------------------------------------------------------
src/kiconengine.cpp (line 164) <https://git.reviewboard.kde.org/r/128397/#comment65647> Ideally, one would want that even if compiled with older Qt, this works as Qt is upgraded. The way to do that would be to hardcode the numerical value of QIconEngine::IsNullHook when compiling with older Qt. - Olivier Goffart On July 7, 2016, 9:55 p.m., David Rosca wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128397/ > ----------------------------------------------------------- > > (Updated July 7, 2016, 9:55 p.m.) > > > Review request for KDE Frameworks and Olivier Goffart. > > > Repository: kiconthemes > > > Description > ------- > > QIcon::hasThemeIcon(name) checks if QIcon::name() == name, so icon engine > must return empty string when icon doesn't exist. > Also implement IsNullHook for Qt 5.7. Comes with autotest. > > > Diffs > ----- > > autotests/CMakeLists.txt 0c7de50 > autotests/kiconengine_unittest.cpp PRE-CREATION > src/kiconengine.h 21a63f5 > src/kiconengine.cpp 3ccc7d1 > > Diff: https://git.reviewboard.kde.org/r/128397/diff/ > > > Testing > ------- > > Issue was reported in https://bugreports.qt.io/browse/QTBUG-54595 > Also similar issue https://bugs.kde.org/show_bug.cgi?id=365031 (there is a > check for hasThemeIcon before using the icon, but it returns true which > results in invalid icon name -> no icon). > > > Thanks, > > David Rosca > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
