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