* objdump.c (objdump_print_addr): If displaying file offsets, show

  the offset even if there are no symbols available.
  (dump_section): Display nothing if none of the section is going to
  be dumped.  Display the file offset, if requested, of the location
  from where the dump starts.
  * doc/binutils.texi (objdump): Mention that dumping via the -s
  switch is also affected by the -F option.

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index 20e9e4e1ff3f2cfb5172c6ce5407022a607e0513..63e104bda14886e44701b166f7b5078c5bd2f889 100644 (file)
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,13 @@
+2008-02-26  Nick Clifton  <nickc@redhat.com>
+
+       * objdump.c (objdump_print_addr): If displaying file offsets, show
+       the offset even if there are no symbols available.
+       (dump_section): Display nothing if none of the section is going to
+       be dumped.  Display the file offset, if requested, of the location
+       from where the dump starts.
+       * doc/binutils.texi (objdump): Mention that dumping via the -s
+       switch is also affected by the -F option.
+
 2008-02-26  Nick Clifton  <nickc@redhat.com>
 
        Re-apply this patch which was accidentally deleted:

diff --git a/binutils/doc/binutils.texi b/binutils/doc/binutils.texi
index e7d4dd1020232aefb03fb796e68bfbd80b5d3ebd..43e75092a45474bf62fbeadd803905ab7882d759 100644 (file)
--- a/binutils/doc/binutils.texi
+++ b/binutils/doc/binutils.texi
@@ -1723,7 +1723,8 @@ When disassembling sections, whenever a symbol is displayed, also
 display the file offset of the region of data that is about to be
 dumped.  If zeroes are being skipped, then when disassembly resumes,
 tell the user how many zeroes were skipped and the file offset of the
-location from where the disassembly resumes.
+location from where the disassembly resumes.  When dumping sections,
+display the file offset of the location from where the dump starts.
 
 @item --file-start-context
 @cindex source code context

diff --git a/binutils/objdump.c b/binutils/objdump.c
index 6f178177dc112c9265a5a48a218c56f7472d3a09..c46d2690577659145f18f3252cf458a20cb33b1e 100644 (file)
--- a/binutils/objdump.c
+++ b/binutils/objdump.c
@@ -864,7 +864,7 @@ objdump_print_addr_with_sym (bfd *abfd, asection *sec, asymbol *sym,
     }
 
   if (display_file_offsets)
-    info->fprintf_func (info->stream, " (File Offset: 0x%lx)",
+    info->fprintf_func (info->stream, _(" (File Offset: 0x%lx)"),
                        (long int)(sec->filepos + (vma - sec->vma)));
 }
 
@@ -877,18 +877,22 @@ objdump_print_addr (bfd_vma vma,
                    bfd_boolean skip_zeroes)
 {
   struct objdump_disasm_info *aux;
-  asymbol *sym = NULL; /* Initialize to avoid compiler warning.  */
+  asymbol *sym;
   bfd_boolean skip_find = FALSE;
 
+  aux = (struct objdump_disasm_info *) info->application_data;
+
   if (sorted_symcount < 1)
     {
       (*info->fprintf_func) (info->stream, "0x");
       objdump_print_value (vma, info, skip_zeroes);
+
+      if (display_file_offsets)
+       info->fprintf_func (info->stream, _(" (File Offset: 0x%lx)"),
+                           (long int)(aux->sec->filepos + (vma - aux->sec->vma)));
       return;
     }
 
-  aux = (struct objdump_disasm_info *) info->application_data;
-
   if (aux->reloc != NULL
       && aux->reloc->sym_ptr_ptr != NULL
       && * aux->reloc->sym_ptr_ptr != NULL)
@@ -2464,12 +2468,6 @@ dump_section (bfd *abfd, asection *section, void *dummy ATTRIBUTE_UNUSED)
   if ((datasize = bfd_section_size (abfd, section)) == 0)
     return;
 
-  printf (_("Contents of section %s:\n"), section->name);
-
-  data = xmalloc (datasize);
-
-  bfd_get_section_contents (abfd, section, data, 0, datasize);
-
   /* Compute the address range to display.  */
   if (start_address == (bfd_vma) -1
       || start_address < section->vma)
@@ -2490,6 +2488,18 @@ dump_section (bfd *abfd, asection *section, void *dummy ATTRIBUTE_UNUSED)
        stop_offset = datasize / opb;
     }
 
+  if (start_offset >= stop_offset)
+    return;
+  
+  printf (_("Contents of section %s:"), section->name);
+  if (display_file_offsets)
+    printf (_("  (Starting at file offset: 0x%lx)"), (long int)(section->filepos + start_offset));
+  printf ("\n");
+
+  data = xmalloc (datasize);
+
+  bfd_get_section_contents (abfd, section, data, 0, datasize);
+
   width = 4;
 
   bfd_sprintf_vma (abfd, buf, start_offset + section->vma);