----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122445/#review76145 -----------------------------------------------------------
Ship it! Thanks. Remember to adjust the commit log, in case it still looks like the text in reviewboard. - David Faure On Feb. 16, 2015, 2:50 p.m., Guy Maurel wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122445/ > ----------------------------------------------------------- > > (Updated Feb. 16, 2015, 2:50 p.m.) > > > Review request for KDE Frameworks, Cornelius Schumacher, Nicolás Alvarez, and > Thiago Macieira. > > > Repository: kconfig > > > Description > ------- > > The if statements at lines 84, 91 > if (path.startsWith(dir) + '/') { > cannot be correct. The value of (path.startsWith(dir) + '/') is always 47. > > I propose to use: > QString plus = QLatin1String("/"); > QString dirPlus; > ... > Q_FOREACH (...) { > dirPlus = dir; > dirPlus.append(plus); > if (path.startsWith(dirPlus)) { > > > Diffs > ----- > > src/core/kdesktopfile.cpp 6381cb30ba9382de492e2d8d4cf1f9e342f1c509 > > Diff: https://git.reviewboard.kde.org/r/122445/diff/ > > > Testing > ------- > > > Thanks, > > Guy Maurel > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
