This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Regression in exec.c (print_section_info) ?


> This works for me, e.g. with "info target" but perhaps there is a bigger
> picture and this line was removed for another reason.

I reviewed the history of the change, and I think it was an oversight.
The change that removed the newline was meant to get rid of some uses
of current_gdbarch, no more.

Could you test the attached? I think it's generally better for
internationalization to have one string rather than put together several
string blocks. I doubt it would make much difference in this case, but
might as well.

I would also like to see a new test if we don't already have one to
prevent this type of regression in the future. Please confirm that
you did run the patch against the testcase on at least one architecture.

One last thing: You also forgot to provide a ChangeLog entry. But *thank
you* for sending a unified diff - I just can't read context diffs!

-- 
Joel

Attachment: exec.c.diff
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]