D26877: Simplify calls to whitespace() and use it in more places.

Tomaz Canabrava tcanabrava at kde.org
Sun Feb 2 10:38:25 GMT 2020


I like indentedStream.


On Sun, 2 Feb 2020 at 11:36 David Faure <noreply at phabricator.kde.org> wrote:

> dfaure requested changes to this revision.
> dfaure added a comment.
> This revision now requires changes to proceed. View Revision
> <https://phabricator.kde.org/D26877>
>
> I don't like it either. It doesn't "read" well.
> Looking at cout or qDebug it's much more common to [the usual stream] <<
> [some modifier] << some more stuff.
>
> Maybe it can be solved with naming though.
> indentedStream() << ...
> ?
>
> *REPOSITORY*
> R237 KConfig
>
> *REVISION DETAIL*
> https://phabricator.kde.org/D26877
>
> *To: *tcanabrava, dfaure, ervin
> *Cc: *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/20200202/4031dead/attachment.html>


More information about the Kde-frameworks-devel mailing list