This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]