kossebau added inline comments. INLINE COMMENTS
> apol wrote in framesvg.cpp:150 > if (prefix.endsWith(QLatin1Char('-'))) { > return hasElement(prefix % QLatin1String("center")); > } else > return hasElement(prefix % QLatin1String("-center")); But given the return, if the else needed? Actually some people recommend to not use else after return. I might be Monday blind, sorry :) REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D21174 To: kossebau, #plasma Cc: apol, kde-frameworks-devel, michaelh, ngraham, bruns