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] arm reversible : <phase_2_complete>


On 12/04/2011 03:01 AM, oza Pawandeep wrote:
> diff -urN arm_orig/ChangeLog arm_new/ChangeLog
> --- arm_orig/ChangeLog	2011-12-03 18:05:04.000000000 +0530
> +++ arm_new/ChangeLog	2011-12-03 19:48:56.000000000 +0530
> @@ -1,3 +1,9 @@
> +2011-12-03  Oza Pawandeep   <oza.pawandeep@gmail.com>
> +
> +	* arm-linux-tdep.c: Added arm-reversible and process record
> +	* arm-tdep.c: arm-reversible-debugging implementation
> +	* arm-tdep.h: arm-reversible data structures
> +	

ChangeLog like this is not accurate to reflect your changes.  "Add
reverse debugging and process record for ARM" is sort of summary of your
patch, but in ChangeLog, we usually need more details on function level.

Please read other ChangeLog entries in ChangeLog, and see how it is
written.  Or, you can read this

http://www.gnu.org/prep/standards/standards.html#Change-Logs

-- 
Yao (éå)


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