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]
Other format: [Raw text]

Re: RFC objdump new feature to generate debug info using ctags style


Hi Nick

I managed to modify prdbg.c to print the information in a format compatible with Exuberant Ctags (plus some fields not available in
E.Ctags).



Thanks very much for submitting this patch.


Since this patch adds a new feature to objdump it definitely needs a
copyright assignment before it can be accepted.  If you are willing
to make such an assignment please fill out the email form attached
below and send it off the FSF.  The process will take a while but it
should be worth it as they we can consider accepting your patches.

Ok.

As for the patch itself, there are a couple of issues:

* It needs a ChangeLog entry.

 * Since you are adding a new feature it needs to be documented.
   Please extend the patch to include changes to
   binutils/doc/binutils.texi and binutils/NEWS.

Ok.

 * Please try to follow the GNU coding standard.  (In the patch there
   are several comments that ought to be reformatted to include a
   terminating period followed by two spaces.  Also there are places
   where there is no space between a function name and its opening
   parenthesis).

Can I use some automatic tool like indent? In this case: where can I find the indent parameters? I know indent have a "GNU" mode, is that ok?

SET

--
Salvador Eduardo Tropea (SET). (Electronics Engineer)
Visit my home page: http://welcome.to/SetSoft or
http://www.geocities.com/SiliconValley/Vista/6552/
Alternative e-mail: set@computer.org set@ieee.org Address: Curapaligue 2124, Caseros, 3 de Febrero
Buenos Aires, (1678), ARGENTINA Phone: +(5411) 4759 0013





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