D15743: Fix removal of external scripts

Francis Herne noreply at phabricator.kde.org
Fri Sep 28 15:42:46 BST 2018


flherne accepted this revision.
flherne added a comment.
This revision is now accepted and ready to land.


  The QSet include is redundant now.
  
  Otherwise, looks good and still seems to work fine; with that tiny fix I think you can go ahead and push it.

INLINE COMMENTS

> externalscriptplugin.cpp:52
>  #include <QMenu>
> +#include <QSet>
>  

This isn't needed anymore.

REPOSITORY
  R32 KDevelop

BRANCH
  script-remove-bug

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

To: amhndu, #kdevelop, flherne
Cc: flherne, kdevelop-devel, glebaccon, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180928/cc1a21c7/attachment.html>


More information about the KDevelop-devel mailing list