From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 2775 invoked by alias); 4 Dec 2011 01:47:09 -0000 Received: (qmail 2766 invoked by uid 22791); 4 Dec 2011 01:47:08 -0000 X-SWARE-Spam-Status: No, hits=-1.7 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sun, 04 Dec 2011 01:46:55 +0000 Received: from nat-jpt.mentorg.com ([192.94.33.2] helo=PR1-MAIL.mgc.mentorg.com) by relay1.mentorg.com with esmtp id 1RX1AM-0002My-CK from Yao_Qi@mentor.com ; Sat, 03 Dec 2011 17:46:54 -0800 Received: from [127.0.0.1] ([172.16.63.104]) by PR1-MAIL.mgc.mentorg.com with Microsoft SMTPSVC(6.0.3790.1830); Sun, 4 Dec 2011 10:46:52 +0900 Message-ID: <4EDAD0EF.20405@codesourcery.com> Date: Sun, 04 Dec 2011 01:47:00 -0000 From: Yao Qi User-Agent: Mozilla/5.0 (X11; Linux i686; rv:7.0.1) Gecko/20110930 Thunderbird/7.0.1 MIME-Version: 1.0 To: oza Pawandeep CC: =?UTF-8?B?UGV0ciBIbHV6w61u?= , Tom Tromey , paawan oza , "gdb-patches@sourceware.org" , chandra krishnappa Subject: Re: [PATCH] arm reversible : References: <998639.46560.qm@web112516.mail.gq1.yahoo.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2011-12/txt/msg00091.txt.bz2 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 > + > + * 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 (齐尧)