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: [patch] nto target: Code cleanup


Joel Brobecker wrote:
	* i386-nto-tdep.c (i386_nto_target): Remove definition.
	(init_i386nto_ops): Use macros to set fields to global
	current_nto_target directly.
	(i386nto_init_abi): Remove unused nto_set_target call.
	* nto-tdep.h (nto_set_target): Remove unused declaration.
	* nto-tdep.c (nto_set_target): Remove unused function.

Thanks for extracting this part out. This is OK.


I'm wondering how the other targets do this sort of thing,
but my brain seem to be refusing to function at the moment...

Thanks for quick review!

Committed.

Primary motivation for this gymnastics in nto was that we actually support 5 different cpu platforms and code in nto-tdep.c is "generic", that is it calls particular nto target's (that is cpu specific stuff) from there.

My motivation for cleanup is to remove things that don't seem to do what is expected (like in this example, by removing this indirection I haven't changed behaviour at all since the "nto_set_target" simply copies just set functions from i386_nto_target into current_nto_target.

The whole thing will undergo further cleanups, but right now I want to bring it as close to what I have as possible.


-- Aleksandar Ristovski QNX Software Systems


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