[C++ Patch] PR 32614

Richard Guenther rguenther@suse.de
Mon Oct 17 11:44:00 GMT 2011


On Mon, 17 Oct 2011, Gabriel Dos Reis wrote:

> On Mon, Oct 17, 2011 at 6:08 AM, Richard Guenther <rguenther@suse.de> wrote:
> 
> > The initial patch, split between rev. 31343 and 31999 indeed added
> 
> Thanks for helping tracking history.
> 
> > +  /* Enable automatic line wrapping by default */
> > +  set_message_length (72);
> >
> > to C++ lang_decode_option.  Later it got appearantly lost somehow,
> > probably during some of the Great Option Reorgs.
> 
> Yes, most likely.
> 
> >
> > I still think automatic wrapping (at 72 columns!?  A terminal
> > is 80x24!) should not be done by default.
> 
> The choice of 72 at the time was based on the 80 of the terminal
> and Emacs guidelines.

I see (and yes, editors probably do not break lines by default - but
my terminals do, and I usually enlarge them to be able to decipher
C++ diagnostics).  I still think that not breaking existing consumers
is more important than to restore something that hasn't been in
effect for years.

Oh, and yes, making documentation match reality is an improvement.

Let's wait for Jason to break the tie.

Richard.


More information about the Gcc-patches mailing list