[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