whitespace now returns a textStream so we don't need to stream() << whitespace()
Use whitespace where we had stream() << " followed by 4 spaces
There are plenty of manual space manipulation on the code that I plan to
get rid of, but that will have to wait as it will break every single
testcase.
Run unittests
| No Linters Available |
| No Unit Test Coverage |
| Buildable 21588 | |
| Build 21606: arc lint + arc unit |
I'm not sure I like whitespace() returning the stream and then being used. In effect it leads to hiding the stream object making the code more terse indeed but less obvious I think.
Maybe it's a question of personal taste, @dfaure any opinion?
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() << ...
?
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.