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. > So, I'll approve the patch > for the release branches (where we don't want to change the default). > We can bikeshed a bit more for trunk. > > Richard. >