From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id oNhbHfHH/V+aRwAAWB0awg (envelope-from ) for ; Tue, 12 Jan 2021 11:01:53 -0500 Received: by simark.ca (Postfix, from userid 112) id 6BAE11EF7E; Tue, 12 Jan 2021 11:01:53 -0500 (EST) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on simark.ca X-Spam-Level: X-Spam-Status: No, score=0.2 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,MAILING_LIST_MULTI,RDNS_NONE,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.2 Received: from sourceware.org (unknown [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id E598F1E940 for ; Tue, 12 Jan 2021 11:01:52 -0500 (EST) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 8B9223894C39; Tue, 12 Jan 2021 16:01:52 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 8B9223894C39 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1610467312; bh=9uv+gky9Y40IJCoulVT9hzDRHY8RIn3Fuerz9n7SlGY=; h=Date:To:In-Reply-To:Subject:References:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc: From; b=RJKveYm0Bs7lNUpYKegGY631IZshuLHQDNhrf/hfKYX7vKlp1xezaE0pe8WINb3NJ JMZcH/iJj+93J2cdFozTVj0mzuguBSRZhsE2YL0qMiN87HyJhIC7Eah7+xR6bSzuWV 4sCjRIYgRSvfI8IVscXUqYZ4R6lprfYyvxIpt70w= Received: from eggs.gnu.org (eggs.gnu.org [IPv6:2001:470:142:3::10]) by sourceware.org (Postfix) with ESMTPS id 50E7B389366E for ; Tue, 12 Jan 2021 16:01:50 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 50E7B389366E Received: from fencepost.gnu.org ([2001:470:142:3::e]:34087) by eggs.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kzM7B-0002cB-Td; Tue, 12 Jan 2021 11:01:49 -0500 Received: from 84.94.185.95.cable.012.net.il ([84.94.185.95]:1500 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1kzM7A-0001jp-7E; Tue, 12 Jan 2021 11:01:49 -0500 Date: Tue, 12 Jan 2021 18:02:00 +0200 Message-Id: <83a6tedtzb.fsf@gnu.org> To: Andrew Burgess In-Reply-To: <20210112131829.3108184-1-andrew.burgess@embecosm.com> (message from Andrew Burgess on Tue, 12 Jan 2021 13:18:29 +0000) Subject: Re: [PATCH] gdb/doc: reorder and group sections relating to aliases References: <20210112131829.3108184-1-andrew.burgess@embecosm.com> X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Eli Zaretskii via Gdb-patches Reply-To: Eli Zaretskii Cc: gdb-patches@sourceware.org Errors-To: gdb-patches-bounces@sourceware.org Sender: "Gdb-patches" > From: Andrew Burgess > Date: Tue, 12 Jan 2021 13:18:29 +0000 > > - Move the section on aliases earlier in the manual, this is now > immediately after the section about creating user defined > commands. This made more sense to me. > > - Rename the section on aliases to 'Command Aliases'. > > - Update the wording of the first paragraph in the 'Command Aliases' > section so that it reads better given the new name. > > - Add a cross-reference from the 'Command Aliases' section to the > 'Python' section now that the aliases section comes first. These changes are OK. > - Capitalised @var{default-args} to @var{DEFAULT-ARGS} in the > 'Command Aliases' section, this is consistent with the other uses > of @var in this section. Use of @var{UPPER-CASE} is discouraged, as it looks sub-optimally in the printed version of the manual. The argument of @var gets up-cased in the Info output anyway, but in the printed manual and in the HTML format it is rendered in slanted typeface, which looks nicer. When you up-case the argument in Texinfo, we get upper-case slanted text in print and HTML, which is less nice. So I'd appreciate if we could down-case all arguments of @var in this section, even though the original was using upper case. > - Move the section on default args to become a sub-section of the > 'Command Aliases' section, and rename this sub-section to just > 'Default Arguments'. This is fine, but you must also add a @menu in the parent section "Command Aliases", because otherwise at least some versions of makeinfo will barf or display a warning: the @menu is necessary for makeinfo to understand the structure of the nodes (which is UP of which) and generate the necessary links. > - Add a @cindex entry to the default arguments sub-section. > > gdb/doc/ChangeLog: > > * gdb.texinfo (Commands): Update menu. > (Extending GDB): Likewise. > (Command aliases default args): Moved later into the document, > added a cindex entry. > (Aliases): Moved earlier in the document. Minor rewording of the > first paragraph, capitalised some @var{default-aliases} inline > with the other uses or @var in this section, and added a cross > reference to the Python node. Please mention in the ChangeLog the old names of these sections, so that if someone looks for their history in the log, they will find this changeset. Thanks again for doing this.