[Differential] [Accepted] D3002: Save/Restore Tool Views order
brauch (Sven Brauch)
noreply at phabricator.kde.org
Thu Nov 17 18:58:12 UTC 2016
brauch accepted this revision.
brauch added a reviewer: brauch.
brauch added a comment.
This revision is now accepted and ready to land.
Nitpicks:
- some things could be const auto&, esp. in loops
- in my (and other people around kdev) preference, always write { } even for one line
Otherwise, +1, thanks!
REPOSITORY
rKDEVPLATFORM KDevPlatform
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D3002
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: antonanikin, mwolff, #kdevelop, brauch
Cc: brauch, apol, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20161117/3080a5f6/attachment.html>
More information about the KDevelop-devel
mailing list