From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 29279 invoked by alias); 17 Feb 2015 23:10:26 -0000 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 Received: (qmail 29268 invoked by uid 89); 17 Feb 2015 23:10:26 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00,SPF_PASS,UNPARSEABLE_RELAY autolearn=ham version=3.3.2 X-HELO: out1134-241.mail.aliyun.com Received: from out1134-241.mail.aliyun.com (HELO out1134-241.mail.aliyun.com) (42.120.134.241) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 17 Feb 2015 23:10:21 +0000 X-Alimail-AntiSpam:AC=CONTINUE;BC=0.08909883|-1;FP=0|0|0|0|0|-1|-1|-1;HT=r41g03048;MF=gang.chen@sunrus.com.cn;PH=DS;RN=1;RT=1;SR=0; Received: from ShengShiZhuChengdeMacBook-Pro.local(mailfrom:gang.chen@sunrus.com.cn ip:223.72.65.79) by smtp.aliyun-inc.com(10.194.100.130); Wed, 18 Feb 2015 07:10:13 +0800 Message-ID: <54E3CC2D.40402@sunrus.com.cn> Date: Tue, 17 Feb 2015 23:10:00 -0000 From: Chen Gang S User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: gdb-patches@sourceware.org Subject: Re: [PATCH] sim: Be sure of calling freeargv() after successfully call buildargv(). References: <54CC0E99.1070501@sunrus.com.cn> <20150217103425.GB544@vapier> In-Reply-To: <20150217103425.GB544@vapier> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-SW-Source: 2015-02/txt/msg00469.txt.bz2 On 2/17/15 18:34, Mike Frysinger wrote: > On 31 Jan 2015 07:07, Chen Gang S wrote: >> --- a/sim/ChangeLog >> +++ b/sim/ChangeLog >> @@ -1,5 +1,10 @@ >> 2014-01-31 Chen Gang >> >> + * microblaze/interp.c (sim_do_command): Call freeargv() before >> + return. > > this should be in sim/microblaze/ChangeLog instead. it looks like your last 4 > entries in sim/ChangeLog need to get relocated. please do so. > Oh, really, I shall change the related comments. > while you're there, you should also fix your gentmap.c entry in > sim/common/ChangeLog -- only one space after the * is used. > Oh, really, I shall change the related comments. > as for the actual code, lgtm. thanks for fixing up the various error paths. > -mike That what I should do, since I focus on binutils and gdb. :-) And excuse me, I am not quite familiar with the related working flow. Can I send 1 patch to fix the 2 comments, and "git push" it after it is reviewed?. Thanks. -- Chen Gang Open, share, and attitude like air, water, and life which God blessed