> On Sept. 8, 2015, 4:46 p.m., Milian Wolff wrote: > > src/kconf_update/kconf_update.cpp, line 250 > > <https://git.reviewboard.kde.org/r/125106/diff/1/?file=402276#file402276line250> > > > > QLatin1String > > Albert Astals Cid wrote: > Seems like a bug in clazy, sometimes it knows that it should use > QLatin1String but it seems it didn't in this case.
If you file a bug with test case I will fix these. - Sergio ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125106/#review85019 ----------------------------------------------------------- On Oct. 5, 2015, 1:16 p.m., Albert Astals Cid wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125106/ > ----------------------------------------------------------- > > (Updated Oct. 5, 2015, 1:16 p.m.) > > > Review request for KDE Frameworks and Matthew Dawson. > > > Repository: kconfig > > > Description > ------- > > Ran the clazy tool > (http://www.kdab.com/use-static-analysis-improve-performance/) > > Mostly QStringLiteral/QLatin1String additions > A few const & additions to non public methods > > > Diffs > ----- > > src/kconfig_compiler/kconfig_compiler.cpp 5c515f4 > src/kconf_update/kconfigutils.cpp 0fec63a > src/kconf_update/kconf_update.cpp a1c98cb > src/gui/kwindowconfig.cpp a32c6cc > src/core/kdesktopfile.cpp 49f37c3 > src/core/ksharedconfig.cpp e059b87 > src/core/kconfigini.cpp 856b7b7 > src/core/kcoreconfigskeleton_p.h d098ef6 > src/core/kauthorized.cpp 4280524 > src/core/kconfig.cpp 4be9e6d > src/core/kconfigbackend.cpp 67bdefa > > Diff: https://git.reviewboard.kde.org/r/125106/diff/ > > > Testing > ------- > > Compiles, test pass > > > Thanks, > > Albert Astals Cid > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
