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

	string choices.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 14514fd..2e32e65 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -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.

diff --git a/gdb/exec.c b/gdb/exec.c
index 16b7e5f..b495a00 100644 (file)
--- a/gdb/exec.c
+++ b/gdb/exec.c
@@ -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"));