[Differential] [Commented On] D4108: Partial Fix: display the plugins that failed to load

Sven Brauch noreply at phabricator.kde.org
Tue Jan 17 20:24:29 UTC 2017


brauch added inline comments.

INLINE COMMENTS

> pluginpreferences.cpp:70
> +        foreach(const auto& plugin, failedPlugins) {
> +            text += plugin.first + " " + plugin.second + "\n";
> +        }

And actually, you can return the assembled string already, then you don't need the QPair stuff at all.

REPOSITORY
  R33 KDevPlatform

REVISION DETAIL
  https://phabricator.kde.org/D4108

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: tcanabrava, kfunk, apol, brauch
Cc: brauch, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170117/55492ad1/attachment.html>


More information about the KDevelop-devel mailing list