From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from eggs.gnu.org (eggs.gnu.org [IPv6:2001:470:142:3::10]) by sourceware.org (Postfix) with ESMTPS id BCB2F39484A9 for ; Sat, 17 Dec 2022 08:23:02 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org BCB2F39484A9 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=gnu.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gnu.org Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1p6STE-0007oO-Ro; Sat, 17 Dec 2022 03:23:00 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=MIME-version:References:Subject:In-Reply-To:To:From: Date; bh=/16AeuYL9r7l45EulTHvfGwOxMairGbZ5I0d+kPqYwI=; b=f2VuryG20b9xdwTM2/de xST8i8MJ0XWxBngZJsaF6dy98brZqIa+YMAX9nrGSPV1jF4eBW7Sgy3cC84lK8304dmb3eMeC7IPR +Iy4lpy35Gqv0i6+eplmrNGvgQ+GPG+IUgCfvRas0LCK9QimwLy2hHcqH1b8TJItW+mXT87VlJlGG f9eMELkCH4n/rLOFrEhwwVS41cnB2zFZo8Y3ADsiIkDSJ7fTztkaSGI3z8Lj78WtBqvN1vdmYE+hN 3I1GaMHxomXDl6MxXuf5F8gyqSZX/+H1UU7C59I9OOh4QBYzCUqESFhpl6uu0HkrQ6WL9nBUGfSl2 MUXHtmrTFhdM0A==; Received: from [87.69.77.57] (helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1p6STE-0005ny-Br; Sat, 17 Dec 2022 03:23:00 -0500 Date: Sat, 17 Dec 2022 10:23:03 +0200 Message-Id: <83edsyigm0.fsf@gnu.org> From: Eli Zaretskii To: Aaron Merey Cc: gdb-patches@sourceware.org, pedro@palves.net, tdevries@suse.de, mliska@suse.cz In-Reply-To: <20221217001525.82647-1-amerey@redhat.com> (message from Aaron Merey via Gdb-patches on Fri, 16 Dec 2022 19:15:25 -0500) Subject: Re: [PATCH v2] [gdb/debuginfod] Ask to cancel further downloads References: <20221217001525.82647-1-amerey@redhat.com> MIME-version: 1.0 Content-type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=1.9 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_BARRACUDACENTRAL,SPF_HELO_PASS,SPF_PASS,TXREP autolearn=no autolearn_force=no version=3.4.6 X-Spam-Level: * X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: > Cc: pedro@palves.net, tdevries@suse.de, Aaron Merey , > Martin Liška > Date: Fri, 16 Dec 2022 19:15:25 -0500 > From: Aaron Merey via Gdb-patches > > + add_setshow_enum_cmd ("cancel", class_run, debuginfod_cancel_enum, > + _("Set cancellation mode for debuginfod."), I suggest a more specific wording here: Set cancellation mode for debuginfod downloads. > + _("Show cancellation mode for debuginfod."), And similarly here. > +The cancellation mode modifies the behavior of ^C while a file is downloading\ > + from debuginfod.\nWhen set to one, two ^C cancels a single download.\n\ > +When set to all, two ^C cancels all further downloads.\n\ > +When set to ask, two ^C asks what to do.\nA single ^C during downloading is\ > + passed to the target process being debugged.\nA second ^C during downloading\ > + may raise a prompt asking whether to cancel the download or send ^C to the\ > + target.\nIf the download is to be cancelled, the cancellation mode takes\ > + effect and no ^C is sent to the target."), This should use "cancel" and "ask", in plural, not "cancels" and "asks" in singular, because it talks about "two ^C". However, it might be better to say ..., pressing ^C twice cancels a single download and similarly for the other instances, because "two ^C" is somewhat awkward. That is the wording you used for the manual, and it is better. > +@kindex set debuginfod cancel > +@anchor{set debuginfod cancel} > +@item set debuginfod cancel > +@itemx set debuginfod cancel one > +@cindex debuginfod cancellation mode > +Pressing @code{^C} twice during downloading will cancel the current I suggest to use @kbd{Ctrl-C} instead @code{^C}, here and elsewhere in the patch. > +@item set debuginfod cancel all > +Pressing @code{^C} twice during downloading will cancel all further > +downloads. I think "this and all further downloads" is more accurate. > +@item set debuginfod cancel ask > +Pressing @code{^C} twice during a download will prompt the user to Instead of "the user", please use "you", or even nothing. The manual is written from the POV of "the user", so we should not use "the user" as some third party. Also, "will prompt whether to cancel", since this is the essence of the question. The user is not asked to cancel, the user is asked whether GDB should cancel. > +cancel all further downloads before attempting to perform the next query. ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ I'm not sure I understand the emphasized part. What do you mean by "before attempting"? > +@kindex show debuginfod cancel > +@item show debuginfod cancel > +Display whether @code{debuginfod cancel} is set to @code{one}, @code{all} or > +@code{ask}. I suggest to reword this Display the current setting of @code{debuginfod cancel}. Thanks.