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