On Mon, 17 Oct 2011, Gabriel Dos Reis wrote: > On Mon, Oct 17, 2011 at 5:53 AM, Richard Guenther <rguent...@suse.de> wrote: > > On Mon, 17 Oct 2011, Paolo Carlini wrote: > > > >> On 10/17/2011 12:26 PM, Gabriel Dos Reis wrote: > >> > On Mon, Oct 17, 2011 at 4:42 AM, Paolo Carlini<paolo.carl...@oracle.com> > >> > wrote: > >> > > FWIW, I still believe that tweaking the documentation to match the long > >> > > standing reality, would be a small improvement. I'm not going to > >> > > further > >> > > insist, anyway. > >> > It isn't improvement. > >> > The improvement would be to restore the documented default. > >> Well either my English is even weaker than I thought, or "restoring" > >> doesn't > >> apply here: the line of code at issue, pp_set_line_maximum_length (pp, 0), > >> has been added by Gaby in Rev 70777, and nothing similar with 72 as second > >> argument existed before. > > > > Thus clearly the documentation is wrong ;) > > ;-) > Not necessarily. Paolo does not say why that line was added. > I don't remember adding that line to change the default.
The initial patch, split between rev. 31343 and 31999 indeed added + /* 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. I still think automatic wrapping (at 72 columns!? A terminal is 80x24!) should not be done by default. You probably will break a lot of existing scripts that assume the default of zero. Richard.
