This is the mail archive of the binutils@sources.redhat.com 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]

Re: [PATCH] Support display of debug_macinfo in readelf.


Hi Daniel,

> + 2001-07-04  Daniel Berlin  <dan@cgsoftware.com>
> + 
> + 	* readelf.c (display_debug_macinfo): New function, display
> + 	.debug_macinfo section.
> + 	(do_debug_macinfo): New variable.
> + 	(parse_args): Handle "-w[mM]" to mean display macro info.
> + 	(process_section_headers): Handle debug_macinfo.
> + 	(debug_displays): Replace unsupported function with macinfo
> + 	function for .debug_macinfo display. 
> + 

Thanks very much for this submission.  I have applied it to the
sources.

In the future though, could you make a couple of changes please ?

  - Submit the ChangeLog as straight text not a context diff

  - Include a patch to the documentation (binutils.texi) mentioning
    the new command line option.

I took care of the documentation thing this time, and I also put a
line in the NEWS file, since I thought that it was worthwhile.

Cheers
        Nick


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