[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