D26131: Braces around for, break early.

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Tue Jan 28 22:42:10 GMT 2020


kossebau added a comment.


  Getting rid of the inversion might be easier to read for simple humans (as me :) ):
  
    const bool isAscii = std::all_of(std::begin(s), std::end(s), [](QChar a) { return a.unicode() <= 127; });

REPOSITORY
  R237 KConfig

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

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


More information about the Kde-frameworks-devel mailing list