Users really don't like clang-format going off on one and breaking lines without their control, this piece of code which I'm removing dates all the way back to 2013, I think it was added because it people think
os << "from" << "asdf" << "def" << "ghi" << "ghi";
is more pleasing.. however, when you start using other types, then this just becomes crazy... and leaves the user wondering why. (see bugs for other odd cases)
os << "from" << "def" << 1 << "ghi" << "lmn" << "opq" << endl << "abc" << "ghi";
The rules I'm removing was to FORCE (MustBreak) a line break between "<String>" << "<String>" but not between "<String>" << AnyOtherType or AnyOtherType << "<String>"
This might be considered too much for a change, somehow breaking compatibility but I feel it's wrong and really plays to the contempt people have towards clang-format that it goes off and formats things how it wants to without obeying its own line limit rules.
This may need a configuration switch as it's likely to cause clang-format changes in already formatted code (as here with UnwrappedLineParser) (Polly also breaks)
...I also noticed no unit tests broke when I made this change...