----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122300/#review74967 -----------------------------------------------------------
Shouldn't splitArgs be fixed instead? After all a real shell would not split on that character, would it? - David Faure On Jan. 28, 2015, 11:24 p.m., Albert Astals Cid wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122300/ > ----------------------------------------------------------- > > (Updated Jan. 28, 2015, 11:24 p.m.) > > > Review request for KDE Frameworks, David Faure, Eike Hein, and Michael Pyne. > > > Bugs: 343380 > https://bugs.kde.org/show_bug.cgi?id=343380 > > > Repository: kcoreaddons > > > Description > ------- > > We need to also quote spaces like the Unicode character 'IDEOGRAPHIC SPACE' > (U+3000) in KShell::quoteArg since KShell::splitArgs is using QChar::isSpace > to decide when a new argument starts, without the quoting, one argument will > get turned into two > > > Diffs > ----- > > autotests/kshelltest.cpp 451bbe9 > src/lib/util/kshell_unix.cpp e0e884f > > Diff: https://git.reviewboard.kde.org/r/122300/diff/ > > > Testing > ------- > > New test passes, bug 343380 is fixed > > > Thanks, > > Albert Astals Cid > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel