Hi Danny,
2. Cleanup in my_symbol_for_address. There's none now. Should there be?
Yes. :-)
Also it looks like my_symbol_for_address is going to quite slow for large
symbol tables. You might want to consider ensuring that the symbol time is
sorted by address and then performing some kind of binary search on it. (Or do
you know that my_symbol_for_address will always be called with incrementally
increasing addresses, in which case you could just cache the last returned
value and start your search there).
3. Which macros should be used for the conditional compilation ? I've
used
#if defined(ARM_WINCE) || defined(SH4)
in my code. This happens to work for me because our build chain defines
ARM_WINCE, but I don't suppose this is right.
The correct way to handle this is to add a new function pointer field to the
bfd_coff_backend_data structure. Arrange for this field to be initialized by
the various backends that need it (IA64, ARM, MIPS, SH) pointing at static
functions defined in the appropriate target specific source file (pe-arm.c
etc), and otherwise for it to be empty(*). Then in
_bfd_XX_print_private_bfd_data_common test the field and call it if it is not
empty.
Cheers
Nick
(*) Or maybe to point at a default pdata display function which does not try to
provide any target specific interpretation.
_______________________________________________
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils
