[RFC] Why does ui_out_field_core_addr pad with leading zeroes?
Doug Evans
dje@google.com
Thu Mar 18 19:19:00 GMT 2010
On Thu, Mar 18, 2010 at 11:55 AM, Mark Kettenis <mark.kettenis@xs4all.nl> wrote:
>> Date: Thu, 18 Mar 2010 11:23:38 -0700
>> From: Doug Evans <dje@google.com>
>>
>> > It also aids visual alignment I suppose. Try also adding
>> > a breakpoint in a shared library in your example,
>> > and redoing `info break'. The column is presently
>> > aligned left. I suspect it won't look as neat after
>> > your patch.
>>
>> I realize that *could* happen, but since we're doing this fancy ui-out
>> thing, I would expect it's the job of the front end to do any desired
>> alignment.
>>
>> For reference sake, this is what I get with my patch:
>>
>> (gdb) b main
>> Breakpoint 2 at 0x40049c: file hello.c, line 6.
>> (gdb) i b
>> Num Type Disp Enb Address What
>> 2 breakpoint keep y 0x40049c in main at hello.c:6
>> (gdb) b printf
>> Breakpoint 3 at 0x7ffff7acbe40
>> (gdb) i b
>> Num Type Disp Enb Address What
>> 2 breakpoint keep y 0x40049c in main at hello.c:6
>> 3 breakpoint keep y 0x7ffff7acbe40 <printf>
>> (gdb)
>
> So since the CLI obviously doesn't align the unpadded addresses
> properly, I'd say your diff shouldn't go in until it does.
I'm not sure I understand. Clearly there is some alignment going on
otherwise the "What" column would be all messed up, but it is not.
I'm going to assume you're saying that the proper alignment would be
to right-justify the addresses. Correct me if this assumption is
wrong.
Having stared at both alignments, I don't see aligning the address to
the right as being easier to read than aligning them to the left, and
even less so.
Right Aligned
(gdb) i b
Num Type Disp Enb Address What
2 breakpoint keep y 0x40049c in main at hello.c:6
3 breakpoint keep y 0x7ffff7acbe40 <printf>
(gdb)
Left Aligned
(gdb) i b
Num Type Disp Enb Address What
2 breakpoint keep y 0x40049c in main at hello.c:6
3 breakpoint keep y 0x7ffff7acbe40 <printf>
(gdb)
Did you have something else in mind?
More information about the Gdb-patches
mailing list