From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1475 invoked by alias); 18 Apr 2016 19:42:57 -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 1462 invoked by uid 89); 18 Apr 2016 19:42:56 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.3 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 spammy=Hx-languages-length:1205, spent X-HELO: mail-vk0-f45.google.com Received: from mail-vk0-f45.google.com (HELO mail-vk0-f45.google.com) (209.85.213.45) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Mon, 18 Apr 2016 19:42:46 +0000 Received: by mail-vk0-f45.google.com with SMTP id n67so3058310vkf.3 for ; Mon, 18 Apr 2016 12:42:46 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=E+2yh5sovNQVPEs/4lpuE2V8Lq7uZKW5Ofz3j2U8WlE=; b=DP2CHcFMrDyL6Mu3iv2pT79uPgqfmusZqh6S2yaeIYm226GXU5W6qxsWp1WJOScU8b y7xWCilE/3wkeZxggVH/ttmGcHAdNEQtuCafNaKEs0yV9Lh7V4FCWAYtHtOYL4MT5bnp 12wkrKiu7GTVqGxG7+c373AeQ4vSjbrXNzEBhwxvg1PNpNxgxTjmkaCFxDlN0mBraXe1 +Rx1ByIdtAeQiqcl13GVe3DCjxe6IOiShgGjA2bn+QbB9lUkSYuYzdsRf1pvDKLUmK8F tJCiY2fHh4OuOwN8NvDb3X51iWTkvxsVtRdNPQSp72poQexE1wgdTQ0O9z+KCFltbU5I MTZg== X-Gm-Message-State: AOPr4FVqx8Jlpwu/XeWvWW3uSYO7RUL82ab+TSHUFEwuG6xSktD6ONV7D0Op/l18NVMO1OlmWDztbk5AvKaInX64 X-Received: by 10.176.2.65 with SMTP id 59mr1416761uas.16.1461008564152; Mon, 18 Apr 2016 12:42:44 -0700 (PDT) MIME-Version: 1.0 Received: by 10.31.136.141 with HTTP; Mon, 18 Apr 2016 12:42:04 -0700 (PDT) In-Reply-To: <83zisq923i.fsf@gnu.org> References: <047d7bdc05e2ada5ef0530c446d1@google.com> <83zisq923i.fsf@gnu.org> From: Doug Evans Date: Mon, 18 Apr 2016 19:42:00 -0000 Message-ID: Subject: Re: [PATCH 2/3] deprecate "skip enable/etc.", doc string cleanup To: Eli Zaretskii Cc: gdb-patches Content-Type: text/plain; charset=UTF-8 X-IsSubscribed: yes X-SW-Source: 2016-04/txt/msg00422.txt.bz2 On Mon, Apr 18, 2016 at 11:57 AM, Eli Zaretskii wrote: >> Date: Mon, 18 Apr 2016 15:47:21 +0000 >> From: Doug Evans >> >> This patch deprecates "skip enable", "skip disable", "skip delete" >> in favor of "enable skip", "disable skip", "delete skip". >> And it uses the doc string macros of 1/3. > > OK, but I still don't like those macros. They make it harder to > understand how the doc string will read. I'm just going with the flow here. The explicit locations patch did this (perhaps not as parameterized as I've done here, but I am improving the docs here - before it was just "Give breakpoint numbers." and now it's something vastly more useful). The alternative is to describe such things (in this case how to specify breakpoint/etc. numbers) in one place and then add a "See mumble." to all the other places. I actually did that, and then remembered the explicit locations discussion. And then spent a lot of time waffling "Crap, what will upstream accept?" and in the decided to stick with existing practice. I certainly don't want to replicate all that text again and again (hence the macros).