----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126328/#review89411 -----------------------------------------------------------
Ship it! Ship It! - Valentin Rusu 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
