----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127817/#review95528 -----------------------------------------------------------
Sorry for coming late, I'm not subscribed to the list (and I'm not a maintainer either), just noticed the commit. I have nothing against the patch per se, and it apparently won't cause the wrong fallback to breeze either as reported in bug#360664. But: Won't we hit the same old problem again that applications won't find their oxygen icons (where they might have installed them in the first place) any more? Which was the reason why the previous patch to change the default to breeze got reverted. See https://bugs.kde.org/show_bug.cgi?id=336739#c9, and the discussion in https://mail.kde.org/pipermail/kde-frameworks-devel/2015-October/027900.html. At least kdenlive still seems to be affected according to http://bugzilla.opensuse.org/show_bug.cgi?id=975387... Sorry if I'm misunderstanding something here, just wanted to raise a concern for the sake of good... - Wolfgang Bauer On Mai 17, 2016, 12:23 nachm., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127817/ > ----------------------------------------------------------- > > (Updated Mai 17, 2016, 12:23 nachm.) > > > Review request for KDE Frameworks. > > > Repository: kiconthemes > > > Description > ------- > > We were using oxygen as the default theme, this meant that oxygen was > processed by every application. This was fixed in the past by changing this > setting to `breeze`, but this only brought new kinds of problems (e.g. review > 127232). > > See bug 336739, bug 360664. > > By just using hicolor, we don't make the application fetch `oxygen` for > little reason. > > > Diffs > ----- > > src/kiconloader.cpp 6ce7ecdc0b9c38647994c750e77980fbf9b6fdd4 > src/kicontheme.cpp f8c0a37754848a74e42bba5866aa4ce0998bce7c > > Diff: https://git.reviewboard.kde.org/r/127817/diff/ > > > Testing > ------- > > On dolphin: > This has an improvement on callgrind 1556M/1185M instructions (30% > improvement). > KIconLoader allocations correspond to 3.8% instead of 4.8%, going down by: > 535K/509K > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
