From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 4358 invoked by alias); 31 Aug 2011 17:27:35 -0000 Received: (qmail 4342 invoked by uid 22791); 31 Aug 2011 17:27:33 -0000 X-SWARE-Spam-Status: No, hits=-1.0 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_SOFTFAIL X-Spam-Check-By: sourceware.org Received: from mtaout22.012.net.il (HELO mtaout22.012.net.il) (80.179.55.172) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 31 Aug 2011 17:27:15 +0000 Received: from conversion-daemon.a-mtaout22.012.net.il by a-mtaout22.012.net.il (HyperSendmail v2007.08) id <0LQS00000Z000L00@a-mtaout22.012.net.il> for gdb-patches@sourceware.org; Wed, 31 Aug 2011 20:24:51 +0300 (IDT) Received: from HOME-C4E4A596F7 ([84.229.83.44]) by a-mtaout22.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0LQS00MNXZ1EGSR0@a-mtaout22.012.net.il>; Wed, 31 Aug 2011 20:24:51 +0300 (IDT) Date: Wed, 31 Aug 2011 17:27:00 -0000 From: Eli Zaretskii Subject: Re: [PATCH] PR/12691 Add the inferior to Python exited event In-reply-to: To: Kevin Pouget Cc: tromey@redhat.com, gdb-patches@sourceware.org Reply-to: Eli Zaretskii Message-id: <83ty8xebob.fsf@gnu.org> MIME-version: 1.0 Content-type: text/plain; charset=iso-8859-1 Content-transfer-encoding: 8BIT References: <83vctdej0n.fsf@gnu.org> 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-08/txt/msg00666.txt.bz2 > From: Kevin Pouget > Date: Wed, 31 Aug 2011 16:57:48 +0200 > Cc: tromey@redhat.com, gdb-patches@sourceware.org > > [1:text/plain Hide] > On Wed, Aug 31, 2011 at 4:46 PM, Eli Zaretskii wrote: > >> From: Kevin Pouget > >> Date: Wed, 31 Aug 2011 16:33:55 +0200 > >> Cc: gdb-patches@sourceware.org > >> > >> The doc was accepted, but I had to reshape the paragraph during the > >> update, Eli, could you please tell me what you think about it? > > > > I have a couple of minor comments: > > >> +@defivar ExitedEvent inferior > >> +A reference to the inferior which triggered the exited event. > > > > I'd suggest to use @code{exited}, since this is a Python symbol. > > this one is fine, > > >>  @defivar ExitedEvent exit_code > >> -An integer representing the exit code which the inferior has returned. > >> +An integer representing the exit code which the inferior has returned. > >                                                            ^^^ > > I think we would be better off without that "has" word. > > > >>                                                                 (Optional, > >> +will exist only in the case that the inferior exited with some status.) > > > > I understand you want to tell that the exit_code attribute is > > optional?  This calls for some rewording, but I actually don't > > understand how can it be that an inferior doesn't return any exit code > > at all.  AFAIK, the exit code could be random garbage, but it's always > > there.  Can you explain? > > but for these two, I didn't write it myself: as you can read in the patch, > I only rearranged the sentence to include my information, the points > you mention were not changed. > (Let me know if you want me to change something anyway) I would rephrase like this: An integer representing the exit code, if any, that the inferior returned. I'd still like to hear from someone "in the know" how is it possible that the exit code won't be available, but let's not block the commit on that behalf. Thanks.