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

Kevin Ottens noreply at phabricator.kde.org
Wed Feb 12 13:15:40 GMT 2020


ervin added a comment.


  In D26877#604672 <https://phabricator.kde.org/D26877#604672>, @dfaure wrote:
  
  > 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`.
  
  
  Yep, that's what made me jump in fact, it feels unusual. :-)
  
  > Maybe it can be solved with naming though.
  >  indentedStream() << ... 
  >  ?
  
  Not bad, it's a good compromise.

REPOSITORY
  R237 KConfig

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

To: tcanabrava, dfaure, ervin
Cc: ervin, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200212/473d9f80/attachment.html>


More information about the Kde-frameworks-devel mailing list