From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-84462-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 8664 invoked by alias); 17 Oct 2011 13:49:28 -0000
Received: (qmail 8651 invoked by uid 22791); 17 Oct 2011 13:49:27 -0000
X-SWARE-Spam-Status: No, hits=-2.3 required=5.0	tests=AWL,BAYES_00,RP_MATCHES_RCVD
X-Spam-Check-By: sourceware.org
Received: from fencepost.gnu.org (HELO fencepost.gnu.org) (140.186.70.10)    by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 17 Oct 2011 13:49:09 +0000
Received: from eliz by fencepost.gnu.org with local (Exim 4.71)	(envelope-from <eliz@gnu.org>)	id 1RFnYx-0005Dp-Gj; Mon, 17 Oct 2011 09:49:07 -0400
Date: Mon, 17 Oct 2011 15:07:00 -0000
Message-Id: <E1RFnYx-0005Dp-Gj@fencepost.gnu.org>
From: Eli Zaretskii <eliz@gnu.org>
To: pmuldoon@redhat.com
CC: tromey@redhat.com, eli@gnu.org, gdb-patches@sourceware.org
In-reply-to: <m3sjmr7rvm.fsf@redhat.com> (message from Phil Muldoon on Mon, 17	Oct 2011 12:57:17 +0100)
Subject: Re: [python] [patch] PR python/12656 (API for special blocks)
Reply-to: Eli Zaretskii <eliz@gnu.org>
References: <m339f4envs.fsf@redhat.com> <m3liswcz63.fsf@fleche.redhat.com>	<m34nzhchro.fsf@redhat.com> <m3ipnwbt3k.fsf@fleche.redhat.com>	<m3wrcb34ex.fsf@redhat.com> <m3d3e3a2ud.fsf@fleche.redhat.com>	<m3y5wj7vv9.fsf@redhat.com> <E1RFlAt-0004KD-FT@fencepost.gnu.org> <m3sjmr7rvm.fsf@redhat.com>
X-IsSubscribed: yes
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: 2011-10/txt/msg00465.txt.bz2

> From: Phil Muldoon <pmuldoon@redhat.com>
> Cc: tromey@redhat.com, eli@gnu.org, gdb-patches@sourceware.org
> Reply-to: pmuldoon@redhat.com
> Date: Mon, 17 Oct 2011 12:57:17 +0100
> 
> >> +@defvar Block.is_global ()
> >> +@code{True} if the @code{gdb.Block} object is the global block
> >> +for the inferior, @code{False} if not.  This attribute is not
> >> +writable.
> >
> > If this is an attribute, why do you use "()" on the @defvar line?
> 
> They used to be methods in a previous posting, I just forgot to remove
> the "()"'s when I converted them to attributes.  Fixed locally.
> 
> > Also, what do you mean by "the global block for the inferior"? why
> > "the" global block?  There could be more than one such block, right?
> 
> Oops, fixed and for static too.
> 
> > And the same goes for the static attribute.
> 
> Is this ok, or do you want me to re-post the patch?

Go ahead and commit, we can always fix later if needed.

Thanks.