This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Display configuration details in --help
- From: Doug Evans <dje at google dot com>
- To: Eli Zaretskii <eliz at gnu dot org>
- Cc: Tom Tromey <tromey at redhat dot com>, Joel Brobecker <brobecker at adacore dot com>, Pedro Alves <palves at redhat dot com>, gdb-patches <gdb-patches at sourceware dot org>
- Date: Tue, 9 Apr 2013 12:31:16 -0700
- Subject: Re: [PATCH] Display configuration details in --help
- References: <83r4jaynqq dot fsf at gnu dot org> <CADPb22R4o4gkQk8oeqGvmWZerVf7gdkw8XOKz9mYxD6ubFPRig at mail dot gmail dot com> <514B4AC3 dot 1090103 at redhat dot com> <83zjxwy4ux dot fsf at gnu dot org> <20130321191051 dot GF5447 at adacore dot com> <83r4j8y1kh dot fsf at gnu dot org> <20130321203919 dot GG5447 at adacore dot com> <87r4j8zd0k dot fsf at fleche dot redhat dot com> <83ehf85u7x dot fsf at gnu dot org> <83wqsbadfe dot fsf at gnu dot org> <20836 dot 17485 dot 525118 dot 795474 at ruffy2 dot mtv dot corp dot google dot com> <83txnfa9m7 dot fsf at gnu dot org> <CADPb22SQaWrBd4MOmMqFaAMc=KnX15hd=be3Ta_u3wGg=HGmXQ at mail dot gmail dot com> <83sj2za65y dot fsf at gnu dot 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.