[rfc] [01/05] Get rid of current_gdbarch in gdbarch.{c,h,sh}

Markus Deuling deuling@de.ibm.com
Fri Nov 9 07:30:00 GMT 2007


Hi Uli,

Ulrich Weigand schrieb:
> 
> Finally, I do think the change in inself is correct (provided the comments
> are adapted).  In fact, the very same should be done to the other two
> functions in gdbarch.sh that use the "local current_gdbarch variable" hack,
> verify_gdbarch and gdbarch_dump.

thank you very much. Now I understand what was meant. I reworked the patch to update
the comments and also rework verify_gdbarch and gdbarch_dump.

Tested on x86 without regression. 

Would this be ok to commit ?

ChangeLog:

	* gdbarch.sh (gdbarch_alloc): Replace current_gdbarch by gdbarch. Remove
	obsolete comment.
	(verify_gdbarch, gdbarch_dump): Likewise.
	(do_read): Update comment.
	(addr_bit, bfd_arch_info, target_desc, long_long_bit, floatformat)
	(ptr_bit, name_of_malloc): Replace current_gdbarch by gdbarch.
	* gdbarch.{c,h}: Regenerate.


-- 
Markus Deuling
GNU Toolchain for Linux on Cell BE
deuling@de.ibm.com

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: diff-gdbarch
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20071109/945d8486/attachment.ksh>


More information about the Gdb-patches mailing list