[PATCH] Multi-process + multi-arch: GDB

Tom Tromey tromey@redhat.com
Tue Nov 13 20:50:00 GMT 2012


>>>>> "Michael" == Michael Eager <eager@eagerm.com> writes:

Michael> I'd recommend get_target_arch() rather than recasting
Michael> target_gdbarch from a variable to a function.  There are other
Michael> get_target_* () functions and this would parallel that usage.
Michael> Another benefit: a more substantive change in the name is less
Michael> likely to be overlooked when adapting or backporting patches.

In this particular case, I don't think there are big problems that can
arise.  target_gdbarch isn't used as a boolean, one of the major such
issues that can possibly arise.  Other issues, I think, will be caught
by type-checking.

That said, I don't really care all that much about the name, and if
Pedro agrees I will rename it to something else.

Tom



More information about the Gdb-patches mailing list