> On Dec. 13, 2015, 1:01 a.m., Nick Shaforostoff wrote: > > i suggest using: > > > > #ifdef Q_OS_WIN > > #define U QLatin1String > > #else > > #define U QStringLiteral > > #endif
Patrick's approach corresponds to a widespread practice in KF5 sources. It also has the advantage to always use the same class, everywhere. - Valentin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126328/#review89407 ----------------------------------------------------------- On Dec. 12, 2015, 10:59 p.m., Patrick Spendrin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126328/ > ----------------------------------------------------------- > > (Updated Dec. 12, 2015, 10:59 p.m.) > > > Review request for KDE Frameworks. > > > Repository: kwallet > > > Description > ------- > > Strings that are separated into multiple parts don't work on Windows > together with QStringLiteral as the first string is interpreted as a > wide (16bit) string, and the second one as a narrow (8bit) string. > Replacing with QString::fromLatin1 is the easiest solution keeping > the code layout the same, joining the strings does work too though. > > > Diffs > ----- > > src/runtime/kwalletd/kwalletd.cpp 5f99f161a0911732c4d46ab36f2e4f3d3f3e3c4b > > Diff: https://git.reviewboard.kde.org/r/126328/diff/ > > > Testing > ------- > > > Thanks, > > Patrick Spendrin > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
