From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 52392 invoked by alias); 17 Sep 2018 22:42:08 -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 52377 invoked by uid 89); 17 Sep 2018 22:42:07 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-25.9 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 spammy= X-HELO: mailsec101.isp.belgacom.be Received: from mailsec101.isp.belgacom.be (HELO mailsec101.isp.belgacom.be) (195.238.20.97) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 17 Sep 2018 22:42:06 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=skynet.be; i=@skynet.be; q=dns/txt; s=securemail; t=1537224126; x=1568760126; h=message-id:subject:from:to:date:in-reply-to:references: mime-version:content-transfer-encoding; bh=Keu6WwgZtdbU1pJL9cqEJckVCxGCXbp7iYkbAcfZkcg=; b=uG3e1sp4ULJvMDcOd+nV1Njei/f0auk7PaI8hLoaPlv2aokVfD5Ip2sr Fz3oq4ODIsBmycgYR67UyKjUUI0v7w==; Received: from 65.33-242-81.adsl-dyn.isp.belgacom.be (HELO md) ([81.242.33.65]) by relay.skynet.be with ESMTP/TLS/AES256-GCM-SHA384; 18 Sep 2018 00:41:49 +0200 Message-ID: <1537224109.3924.6.camel@skynet.be> Subject: Re: [PATCH] [OBVIOUS] enable frame-filter short help uses disable instead of enable From: Philippe Waroquiers To: gdb-patches@sourceware.org Date: Mon, 17 Sep 2018 22:42:00 -0000 In-Reply-To: <20180917222334.27753-1-philippe.waroquiers@skynet.be> References: <20180917222334.27753-1-philippe.waroquiers@skynet.be> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2018-09/txt/msg00613.txt.bz2 Note that the Usage: string in the below still looks strange to me: the second 'enable' in Usage: enable frame-filter enable DICTIONARY [NAME] should be removed (and similarly in the disable frame-filter). Then, reading the 'help enable frame-filter', it looks like DICTIONARY is optional, but the Usage: seems to indicate it is mandatory. I am wondering if the correct Usage: would be something like: Usage: enable frame-filter enable [DICTIONARY [NAME]] (and similar for disable frame-filter). The texinfo documentation seems also somewhat different from the 'help enable frame-filter' e.g. it tells that DICTIONARY may be the name of an object file, while the help does not tell that (at least not that clearly). So, would be good if someone mastering frame filter would clarify the doc/help. Philippe On Tue, 2018-09-18 at 00:23 +0200, Philippe Waroquiers wrote: > Without the patch: > (gdb) apropos able frame-filter > disable frame-filter -- GDB command to disable the specified frame-filter > enable frame-filter -- GDB command to disable the specified frame-filter > > With the patch: > (gdb) apropos able frame-filter > disable frame-filter -- GDB command to disable the specified frame-filter > enable frame-filter -- GDB command to enable the specified frame-filter > > Pushed as obvious > --- > gdb/python/lib/gdb/command/frame_filters.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/gdb/python/lib/gdb/command/frame_filters.py b/gdb/python/lib/gdb/command/frame_filters.py > index 3afe5e9ac1..16a40ed98e 100644 > --- a/gdb/python/lib/gdb/command/frame_filters.py > +++ b/gdb/python/lib/gdb/command/frame_filters.py > @@ -207,7 +207,7 @@ def _complete_frame_filter_name(word, printer_dict): > return flist > > class EnableFrameFilter(gdb.Command): > - """GDB command to disable the specified frame-filter. > + """GDB command to enable the specified frame-filter. > > Usage: enable frame-filter enable DICTIONARY [NAME] >