From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-52661-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 636 invoked by alias); 14 Dec 2007 06:23:47 -0000
Received: (qmail 626 invoked by uid 22791); 14 Dec 2007 06:23:46 -0000
X-Spam-Check-By: sourceware.org
Received: from zigzag.lvk.cs.msu.su (HELO zigzag.lvk.cs.msu.su) (158.250.17.23)     by sourceware.org (qpsmtpd/0.31) with ESMTP; Fri, 14 Dec 2007 06:23:42 +0000
Received: from Debian-exim by zigzag.lvk.cs.msu.su with spam-scanned (Exim 4.50) 	id 1J33xO-0007e8-MH 	for gdb-patches@sources.redhat.com; Fri, 14 Dec 2007 09:23:39 +0300
Received: from localhost ([127.0.0.1] helo=ip6-localhost) 	by zigzag.lvk.cs.msu.su with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA:32) 	(Exim 4.50) 	id 1J33xD-0007dk-NA; Fri, 14 Dec 2007 09:23:23 +0300
From: Vladimir Prus <ghost@cs.msu.su>
To: Eli Zaretskii <eliz@gnu.org>
Subject: Re: Pending breakpoints in MI
Date: Fri, 14 Dec 2007 06:27:00 -0000
User-Agent: KMail/1.9.6
Cc: gdb-patches@sources.redhat.com
References: <200711081412.14452.ghost@cs.msu.su> <fjrgdd$bhn$1@ger.gmane.org> <uodcuwd5q.fsf@gnu.org>
In-Reply-To: <uodcuwd5q.fsf@gnu.org>
MIME-Version: 1.0
Content-Type: text/plain;   charset="iso-8859-15"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
Message-Id: <200712140923.21382.ghost@cs.msu.su>
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
X-SW-Source: 2007-12/txt/msg00183.txt.bz2

On Thursday 13 December 2007 23:30:25 Eli Zaretskii wrote:
> > From:  Vladimir Prus <ghost@cs.msu.su>
> > Date:  Thu, 13 Dec 2007 17:42:53 +0300
> > 
> > I've done so. Eli, does the manual changes in the attached patch look
> > good to you?
> 
> Yes, except for one minor gotcha:
> 
> > +If @var{location} cannot be parsed (for example if it
> > +refers to unknown files or functions), create a pending
> > +breakpoint.
> 
> It's better to say "... this command creates a pending breakpoint."
> 
> Also, please put 2 spaces after the period that ends a sentence.

Thanks, I've adjusted the patch.

Unfortunately, it seems that one of the prerequisite patches is still
not reviewed, but I'll ping that separately.

- Volodya