D26131: Braces around for, break early.

David Faure noreply at phabricator.kde.org
Tue Apr 14 21:46:22 BST 2020


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


  I'd remove the space after '!' and make the bool const, but I'm nitpicking, you can also push as is.
  
  cbegin/cend wouldn't change anything on a const container.

REPOSITORY
  R237 KConfig

BRANCH
  arcpatch-D26131

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

To: tcanabrava, ervin, dfaure
Cc: kossebau, ervin, patrickelectric, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200414/805584ca/attachment.html>


More information about the Kde-frameworks-devel mailing list