PATCH: PR ld/11146: Add "readelf --dyn-syms"

Daniel Jacobowitz dan@codesourcery.com
Mon Jan 11 13:51:00 GMT 2010


On Mon, Jan 11, 2010 at 12:28:03PM +0000, Nick Clifton wrote:
> Hi H.J.
> 
> >binutils/
> >2010-01-07  H.J. Lu  <hongjiu.lu@intel.com>
> >
> >	PR ld/11146
> >	* readelf.c (do_dyn_syms): New.
> >	(OPTION_DYN_SYMBS): Likewise.

Why syms versus symbs?

It seems to me very confusing to have -Ds (dynamic symbols) and 
--dyn-syms.  The description in the manual sounds like -Ds and
--dyn-syms should both do what your new patch does.  I wouldn't
be able to tell the difference.  Is the existing -Ds useful?  If both
are useful, the manual should differentiate them.  If the other is not
useful, just change -Ds.

-- 
Daniel Jacobowitz
CodeSourcery



More information about the Binutils mailing list