bug-binutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Bug binutils/4907] readelf doesn't dump .eh_frame_hdr section


From: nickc at redhat dot com
Subject: [Bug binutils/4907] readelf doesn't dump .eh_frame_hdr section
Date: 27 Feb 2008 14:17:22 -0000

------- Additional Comments From nickc at redhat dot com  2008-02-27 14:17 
-------
Hi Denis,

  I have looked over your patch and I think that it is a very good first stab at
this.  There are some improvements which I think can be made however, such as:

  * Adding a new command line option (eg -E) to enable this feature without
having to use the -a option.  Plus the feature needs to be mentioned in
binutils/NEWS and documented in binutils/doc/binutils.texi.

  * Removing the read_sleb128 code and using the read_leb128 function in dwarf.c
instead.  (Of course it will have to be exported from there).  You could also
delete the read_uleb128 function and replace all references to that as well... 
Avoiding code duplication is a good thing IMHO.

  * Please do not declare variables in the middle of a block.  It is a gcc
extention and we would like the code to be able to be compiled by any ISO-C
compliant compiler.

  * This code might be better as a patch to binutils/dwarf.c.

  * You have to use a different printf formatter in order to print 64-bit values
on a Cygwin based system.  (See other places in readelf.c where %llx is used). 
[Actually this should be broken out into a separate function].


Cheers
  Nick



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |WAITING


http://sourceware.org/bugzilla/show_bug.cgi?id=4907

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]