----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126423/#review89792 -----------------------------------------------------------
Ship it! Ship It! - Michael Pyne On Dec. 19, 2015, 8:34 a.m., Jonathan Marten wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126423/ > ----------------------------------------------------------- > > (Updated Dec. 19, 2015, 8:34 a.m.) > > > Review request for KDE Frameworks. > > > Bugs: 352471 > https://bugs.kde.org/show_bug.cgi?id=352471 > > > Repository: kcmutils > > > Description > ------- > > The API documentation says that the 4-argument form of > KPluginSelector::addPlugins() can be called with the last 'config' parameter > as null (the default), meaning the standard application config file. > However, there seems to be a misplaced assert which means that this will not > be accepted. > > > Diffs > ----- > > src/kpluginselector.cpp 34a7897 > > Diff: https://git.reviewboard.kde.org/r/126423/diff/ > > > Testing > ------- > > Built kcmutils with this change, checked with Konqueror's "Configure > Extensions" dialogue as described in the bug report. Observed assert/crash > without this change, and correct operation with it. > > > Thanks, > > Jonathan Marten > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel