> On Gru 14, 2015, 8:10 po południu, Sergio Martins wrote: > > src/klocalizedstring.cpp, line 389 > > <https://git.reviewboard.kde.org/r/126350/diff/1/?file=422775#file422775line389> > > > > this will break the build with MSVC, it doesn't support QStringLiteral > > with multi-line literals. > > > > (hint: > > export > > CLAZY_EXTRA_OPTIONS="qstring-uneeded-heap-allocations-msvc-compat" > > ) > > Nick Shaforostoff wrote: > why QString is used for debug output in the first place? > there is operator<<(const char * s) for latin1 text, which is the case > (i.e. no non-latin symbols are used in the literals) > > using strings for debugging output is suboptimal > > Artur Puzio wrote: > look at line 487: > ``` > qWarning() << QStringLiteral( > "Plural argument to message {%1} not > supplied before conversion.") > > .arg(shortenMessage(QString::fromUtf8(text))); > ``` > Some of QString-s can by changed to const char *, but in other places > they are required. > > Aleix Pol Gonzalez wrote: > `qWarning() << "Plural argument to message {" << > shortenMessage(QString::fromUtf8(text))<< "} not supplied before > conversion.";` > > Still, I don't believe it's up to this iteration to fix this one, the > code was already like this. > Artur, can you look at the issue pointed out by Sergio?
Just fixed the issue pointed out by Sergio. Also chenged to char[] when writeing literals to warning. - Artur ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126350/#review89492 ----------------------------------------------------------- On Gru 15, 2015, 12:34 po południu, Artur Puzio wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126350/ > ----------------------------------------------------------- > > (Updated Gru 15, 2015, 12:34 po południu) > > > Review request for KDE Frameworks and Aleix Pol Gonzalez. > > > Repository: ki18n > > > Description > ------- > > Fixed `warning: KuitSetup has dtor, copy-ctor but not copy-assignment > [-Wclazy-rule-of-three]` > Fixed `warning: Reserve candidate [-Wclazy-reserve-candidates]` > Fixed `warning: Use QVariant::toFoo() instead of QVariant::value<Foo>() > [-Wclazy-variant-sanitizer]` > Fixed `warning: Use midRef() instead [-Wclazy-qstring-ref]` > Fixed `warning: Pass small and trivially-copyable type by value (const class > QChar &) [-Wclazy-foreach]` > Fixed `warning: QString::fromLatin1() being passed a literal > [-Wclazy-qstring-uneeded-heap-allocations]` > Fixed `warning: QString(const char*) being called > [-Wclazy-qstring-uneeded-heap-allocations]` > Fixed `warning: QString(QLatin1String) being called > [-Wclazy-qstring-uneeded-heap-allocations]` > Changed `QLatin1String`-s to `char[]` when writteing to warning. > Changed to `QStringLitera`-s in some lines, where Clazy isn't warning. > > > Diffs > ----- > > src/common_helpers.cpp dad7f84 > src/kcatalog.cpp 7711e9b > src/klocalizedcontext.cpp 3bc42dd > src/klocalizedstring.cpp 69950d2 > src/ktranscript.cpp 04dda66 > src/kuitmarkup.h d110ca3 > src/kuitmarkup.cpp 02b891a > > Diff: https://git.reviewboard.kde.org/r/126350/diff/ > > > Testing > ------- > > Automated tests 1,3 and 4 passing. > Test 2 fails on my system both: after changes and before. > > > Thanks, > > Artur Puzio > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel