----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126423/ -----------------------------------------------------------
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