From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3270 invoked by alias); 12 Dec 2012 18:59:31 -0000 Received: (qmail 3259 invoked by uid 22791); 12 Dec 2012 18:59:30 -0000 X-SWARE-Spam-Status: No, hits=-4.5 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED,RCVD_IN_DNSWL_NONE,RCVD_IN_HOSTKARMA_NO,SPF_SOFTFAIL X-Spam-Check-By: sourceware.org Received: from mtaout21.012.net.il (HELO mtaout21.012.net.il) (80.179.55.169) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 12 Dec 2012 18:59:25 +0000 Received: from conversion-daemon.a-mtaout21.012.net.il by a-mtaout21.012.net.il (HyperSendmail v2007.08) id <0MEX00K00M24SM00@a-mtaout21.012.net.il> for gdb-patches@sourceware.org; Wed, 12 Dec 2012 20:59:21 +0200 (IST) Received: from HOME-C4E4A596F7 ([87.69.4.28]) by a-mtaout21.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0MEX00KOHM2XRY30@a-mtaout21.012.net.il>; Wed, 12 Dec 2012 20:59:21 +0200 (IST) Date: Wed, 12 Dec 2012 18:59:00 -0000 From: Eli Zaretskii Subject: Re: [PATCH 4/8] command 'set debug notification'. In-reply-to: <50C8C564.6090405@redhat.com> To: Pedro Alves Cc: yao@codesourcery.com, gdb-patches@sourceware.org Reply-to: Eli Zaretskii Message-id: <83zk1j6qrc.fsf@gnu.org> References: <1355208017-9204-1-git-send-email-yao@codesourcery.com> <1355208017-9204-5-git-send-email-yao@codesourcery.com> <83r4mx9ifb.fsf@gnu.org> <50C8C564.6090405@redhat.com> 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: 2012-12/txt/msg00421.txt.bz2 > Date: Wed, 12 Dec 2012 17:56:52 +0000 > From: Pedro Alves > CC: Yao Qi , gdb-patches@sourceware.org > > On 12/11/2012 07:06 AM, Eli Zaretskii wrote: > > > >> > + add_setshow_boolean_cmd ("notification", no_class, ¬if_debug, > >> > + _("\ > >> > +Set debugging of async remote notification."), _("\ > >> > +Show debugging of async remote notification."), _("\ > >> > +When non-zero, async remote notification specific" > >> > +" internal debugging is enabled."), > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > I think not "debugging" is enabled, but "debugging info" is enabled. > > This command just enables/disables internal debugging/logging output, > it is not about debug info, as in dwarf, etc. Granted, I understand that. If you suggest to say When non-zero, debugging output about async remote notifications is enabled. then I have no objections. But saying that "internal debugging is enabled" is certainly inaccurate and perhaps even misleading, IMO.