FreeChainXenon/binutils-gdb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

* exec.c (print_section_info): Add FIXME comments regarding format

Browse source 
string choices.
This commit is contained in:
Kevin Buettner 2002-12-23 23:36:39 +00:00
parent 11e6387acb
commit bcf16802ca
2 changed files with 12 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
gdb/ChangeLog
View file

@ -1,3 +1,8 @@
2002-12-20 Kevin Buettner <kevinb@redhat.com>
* exec.c (print_section_info): Add FIXME comments regarding format
string choices.
2002-12-23 Daniel Jacobowitz <drow@mvista.com>
* config/pa/nm-hppab.h: Delete duplicate CANNOT_STORE_REGISTER decl.

7
gdb/exec.c
View file

@ -545,6 +545,7 @@ void
print_section_info (struct target_ops *t, bfd *abfd)
{
struct section_table *p;
/* FIXME: "016l" is not wide enough when TARGET_ADDR_BIT > 64. */
char *fmt = TARGET_ADDR_BIT <= 32 ? "08l" : "016l";
printf_filtered ("\t`%s', ", bfd_get_filename (abfd));
@ -560,6 +561,12 @@ print_section_info (struct target_ops *t, bfd *abfd)
{
printf_filtered ("\t%s", local_hex_string_custom (p->addr, fmt));
printf_filtered (" - %s", local_hex_string_custom (p->endaddr, fmt));
/* FIXME: A format of "08l" is not wide enough for file offsets
larger than 4GB. OTOH, making it "016l" isn't desirable either
since most output will then be much wider than necessary. It
may make sense to test the size of the file and choose the
format string accordingly. */
if (info_verbose)
printf_filtered (" @ %s",
local_hex_string_custom (p->the_bfd_section->filepos, "08l"));