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] |
> 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] |