Re: [PATCH] Display configuration details in --help

[Eli Zaretskii <eliz@gnu.org>]

> Date: Tue, 9 Apr 2013 12:31:16 -0700
> From: Doug Evans <dje@google.com>
> Cc: Tom Tromey <tromey@redhat.com>, Joel Brobecker <brobecker@adacore.com>, 
> 	Pedro Alves <palves@redhat.com>, gdb-patches <gdb-patches@sourceware.org>
> 
> On Tue, Apr 9, 2013 at 11:51 AM, Eli Zaretskii <eliz@gnu.org> wrote:
> >> >>  > +  fprintf_filtered (stream, _("\
> >> >>  > +             --without-expat\n\
> >> >>  > +"));
> >> >>  > +#endif
> >> >>
> >> >> If we've already discussed this, please ignore, but ... :-)
> >> >> I'd prefer one line per fprintf instead of three.
> >> >
> >> > Sorry, I'm not following: what 3 lines?  Can you show how would you
> >> > like the code be reformatted?
> >>
> >>   fprintf_filtered (stream, _("  -with-expat\n"));  // assuming 2
> >> space indentation
> >
> > OK, I used the style I did because it makes it much easier to keep the
> > alignment visible, without the need to count characters and columns.
> 
> I'm not sure.  13 is a lot to visualize and given the vertical span
> it's not straightforward, to me anyway.
> 
> Using a more explicit form for the indentation (making it a parameter
> to the printf) works better for me if the amount of indentation is
> going to be that much.

These are minor stylistic issues, really just personal preferences.
Are they important enough to block the commit?