Re: [patch] nto target: Code cleanup

Aleksandar Ristovski
2009-06-11T19:34:19Z

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.