----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119975/#review65442 -----------------------------------------------------------
src/kdbusservice.cpp <https://git.reviewboard.kde.org/r/119975/#comment45731> objectPath.remove(QLatin1Char('-')); - Kai Uwe Broulik On Aug. 28, 2014, 3:18 nachm., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119975/ > ----------------------------------------------------------- > > (Updated Aug. 28, 2014, 3:18 nachm.) > > > Review request for KDE Frameworks. > > > Repository: kdbusaddons > > > Description > ------- > > If the application name has a '-', Qt asserts. Instead of going there, remove > the '-' and output a warning so the replacement is noticed and fixable. > > > Diffs > ----- > > src/kdbusservice.cpp a9b7cc8 > > Diff: https://git.reviewboard.kde.org/r/119975/diff/ > > > Testing > ------- > > Ran tests, reproduced with an application. > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel