Hi David, >> + /* PR 84195: Replace control characters in the message with their >> + escaped equivalents. Allow newlines if -fmessage-length has >> + been set to a non-zero value. > > I'm not quite sure why we allow newlines in this case, sorry.
Because the documentation for -fmessage-length says: Try to format error messages so that they fit on lines of about N characters. If N is zero, then no line-wrapping is done; each error message appears on a single line. This is the default for all front ends. So with a non-zero message length, multi-line messages are allowed. At least that was my understanding of the option. Thanks for the patch review. I will get onto fixing the points you raised today. Cheers Nick