[Differential] [Updated] D1955: projectmanagerview: Add Cut command into the context menu

aspotashev (Alexander Potashev) noreply at phabricator.kde.org
Sun Sep 4 19:54:22 UTC 2016


aspotashev marked 3 inline comments as done.
aspotashev added inline comments.

INLINE COMMENTS

> kfunk wrote in projectmanagerviewplugin.cpp:754
> Can be a free function inside `namespace {}`.
> 
> I'd also rename to `createSeparatorAction`

Changed to a static function instead because namespace {} takes more lines of code and is harder to read.

REPOSITORY
  rKDEVPLATFORM KDevPlatform

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

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

To: aspotashev, kfunk
Cc: kfunk, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20160904/d861e301/attachment.html>


More information about the KDevelop-devel mailing list