This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB 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 for testing purposes


Michael,

> does this suffice as a submission for review, or do you suggest an
> alternative format?

Thank you for sending your contribution.  The procedure for contributing
is described in gdb/CONTRIBUTE, and I suggestion you give it a read.

It was hard to determine how large your contribution really is, because
of the diff format you chose (see below). Small obvious contributions
can be accepted without copyright assignment.  But if you're thinking
of making more than one or two small contributions, you need to file
a copyright assignment with the FSF.  This is briefly explained in
the CONTRIBUTE file.  Let me know if you'd like to start the process.
It takes a few weeks to complete, so the sooner you start it, the better.

Just a couple of quick remarks that may not be obvious after you finished
reading CONTRIBUTE:

> 115,134c115

Your diff does not provide any context, so it's very hard to read.
Most maintainers here prefer "unified" diffs (use "diff -u").

Make also sure to use diff option "-p", which also really helps
understanding where your patch is changing things.

> < // print the status word
> < // updated 20091227 (development@codenamezero.org)

GDB must follow ISO C 90 (I believe that's the standard we're sticking
too), which means comments should use the normal C comment markers.
The "updated ..." should go, as this information is available through
the version control system.


-- 
Joel


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