From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 28944 invoked by alias); 24 Jun 2013 19:04:09 -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 28934 invoked by uid 89); 24 Jun 2013 19:04:09 -0000 X-Spam-SWARE-Status: No, score=-6.7 required=5.0 tests=AWL,BAYES_00,RCVD_IN_HOSTKARMA_W,RCVD_IN_HOSTKARMA_WL,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.1 Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Mon, 24 Jun 2013 19:04:08 +0000 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r5OJ4633005290 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Mon, 24 Jun 2013 15:04:06 -0400 Received: from psique (ovpn-113-175.phx2.redhat.com [10.3.113.175]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id r5OJ43i4027676 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NO); Mon, 24 Jun 2013 15:04:05 -0400 From: Sergio Durigan Junior To: Wei-cheng Wang Cc: gdb-patches@sourceware.org Subject: Re: [patch] Parse quoted string for restore/dump commands. References: <51B20120.8070909@gmail.com> X-URL: http://www.redhat.com Date: Mon, 24 Jun 2013 19:11:00 -0000 In-Reply-To: (Wei-cheng Wang's message of "Mon, 24 Jun 2013 20:56:15 +0800") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-SW-Source: 2013-06/txt/msg00671.txt.bz2 Hi Wei-cheng, On Monday, June 24 2013, Wei-cheng Wang wrote: >>> Also, when you commit the patch, please remember to include your info on >>> gdb/MAINTAINERS, since you will be able to commit after approval. >> >> Just to be totally clear: this modification to gdb/MAINTAINERS should be >> considered a totally different patch, and should not be >> included/committed with this one :-). > > Actually, I didn't have CVS write access yet. I may need a sponsor. Yeah, a sponsor needs to be a maintainer, so I can't help you with that :-(. > Before that, I need someone help me checking this patch in, if approved :-) If the patch is approved, I can check it in for you, no problem :-). > -- > > gdb/ChangeLog | 6 +++++ > gdb/cli/cli-dump.c | 54 ++++++++++++++++++++++++++++++++++----- > gdb/testsuite/ChangeLog | 4 +++ > gdb/testsuite/gdb.base/dump.exp | 34 +++++++++++++++++++++++- > 4 files changed, 91 insertions(+), 7 deletions(-) > > diff --git a/gdb/ChangeLog b/gdb/ChangeLog > index 1efd3f9..b7a87eb 100644 > --- a/gdb/ChangeLog > +++ b/gdb/ChangeLog > @@ -1,3 +1,9 @@ > +2013-06-24 Wei-cheng Wang > + Sergio Durigan Junior No need to put my name in the entry :-). I just reviewed your patch, but the idea and the work are still yours. -- Sergio