[PATCH] Regression setting breakpoint

Joel Brobecker brobecker@adacore.com
Thu Jan 16 05:37:00 GMT 2014


> Not really a code review, but the ChangeLog entries should be of the
> form:
> 
> 2014-01-16  Your Name  <you@email.com>
> 
> 	* file.c (function): What changed.

FWIW, I have been using:

| gdb/ChangeLog:
|
|         * file.c (function): What changed.
|
| gdb/testsuite/ChangeLog:
|
|         * gdb.base/file.exp: Bla bla bla.

Two reasons: It makes it obvious which ChangeLog the entries refer to,
and it avoids the need to put a date in the revision log, which may
become obe due to the time it takes to get the patch in after it is
written.

The format with dates are accepted as well, of course. And I have
seen people add the directory usually on the line just ahead of
the CL entry. Eg:

| gdb/:
| 2014-01-16  Your Name  <you@example.com>

-- 
Joel



More information about the Gdb-patches mailing list