From mboxrd@z Thu Jan  1 00:00:00 1970
Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail
From: Eli Zaretskii <eliz@gnu.org>
Newsgroups: gmane.emacs.bugs
Subject: bug#67310: [PATCH] Include the project--list as history when
 prompting for a project
Date: Sat, 25 Nov 2023 16:10:09 +0200
Message-ID: <83zfz1c4am.fsf@gnu.org>
References: <ier8r6s42on.fsf@janestreet.com>
 <dc5ad8f4-b857-ea87-d178-bae903cee517@gutov.dev>
 <421e3ea8-d150-566e-f645-cbf1ccbcba61@gutov.dev>
 <ier34wz3zkx.fsf@janestreet.com>
 <81b6ed58-3fdc-3b5a-41a9-84d66475d5f1@gutov.dev>
 <ierr0khlq19.fsf@janestreet.com>
 <5c3eb6a1-38da-8af4-419a-e0567b163e3a@gutov.dev>
 <iercyw1l6rz.fsf@janestreet.com> <83bkblgeja.fsf@gnu.org>
 <ab8f5ad1-356e-e90d-eab4-d36a0b0821fc@gutov.dev> <83edgedy1j.fsf@gnu.org>
 <96636783-62bb-0bb1-ecb5-c510630402e3@gutov.dev>
Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214";
	logging-data="36418"; mail-complaints-to="usenet@ciao.gmane.io"
Cc: 67310@debbugs.gnu.org, sbaugh@janestreet.com, juri@linkov.net
To: Dmitry Gutov <dmitry@gutov.dev>
Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Sat Nov 25 15:11:11 2023
Return-path: <bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org>
Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org
Original-Received: from lists.gnu.org ([209.51.188.17])
	by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
	(Exim 4.92)
	(envelope-from <bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org>)
	id 1r6tNG-0009Hk-Dq
	for geb-bug-gnu-emacs@m.gmane-mx.org; Sat, 25 Nov 2023 15:11:10 +0100
Original-Received: from localhost ([::1] helo=lists1p.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.90_1)
	(envelope-from <bug-gnu-emacs-bounces@gnu.org>)
	id 1r6tN6-0007yN-67; Sat, 25 Nov 2023 09:11:00 -0500
Original-Received: from eggs.gnu.org ([2001:470:142:3::10])
 by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
 (Exim 4.90_1) (envelope-from <Debian-debbugs@debbugs.gnu.org>)
 id 1r6tN3-0007y4-P4
 for bug-gnu-emacs@gnu.org; Sat, 25 Nov 2023 09:10:57 -0500
Original-Received: from debbugs.gnu.org ([2001:470:142:5::43])
 by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128)
 (Exim 4.90_1) (envelope-from <Debian-debbugs@debbugs.gnu.org>)
 id 1r6tN3-00021t-GH
 for bug-gnu-emacs@gnu.org; Sat, 25 Nov 2023 09:10:57 -0500
Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2)
 (envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1r6tN8-0000ee-Bt
 for bug-gnu-emacs@gnu.org; Sat, 25 Nov 2023 09:11:02 -0500
X-Loop: help-debbugs@gnu.org
Resent-From: Eli Zaretskii <eliz@gnu.org>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces@debbugs.gnu.org>
Resent-CC: bug-gnu-emacs@gnu.org
Resent-Date: Sat, 25 Nov 2023 14:11:02 +0000
Resent-Message-ID: <handler.67310.B67310.17009214442483@debbugs.gnu.org>
Resent-Sender: help-debbugs@gnu.org
X-GNU-PR-Message: followup 67310
X-GNU-PR-Package: emacs
X-GNU-PR-Keywords: patch
Original-Received: via spool by 67310-submit@debbugs.gnu.org id=B67310.17009214442483
 (code B ref 67310); Sat, 25 Nov 2023 14:11:02 +0000
Original-Received: (at 67310) by debbugs.gnu.org; 25 Nov 2023 14:10:44 +0000
Original-Received: from localhost ([127.0.0.1]:38214 helo=debbugs.gnu.org)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <debbugs-submit-bounces@debbugs.gnu.org>)
 id 1r6tMp-0000dy-GU
 for submit@debbugs.gnu.org; Sat, 25 Nov 2023 09:10:43 -0500
Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:52436)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <eliz@gnu.org>) id 1r6tMn-0000dm-Ci
 for 67310@debbugs.gnu.org; Sat, 25 Nov 2023 09:10:42 -0500
Original-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 <eliz@gnu.org>)
 id 1r6tMa-000206-Iv; Sat, 25 Nov 2023 09:10:30 -0500
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org;
 s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date:
 mime-version; bh=BijP+rpq0t9zygDdsaDvkdm9TNI3Hu0s7GY1MccXp/4=; b=CxJ4vWyiLJno
 5VE4H12Gs1pgRI6g3GRVj4wynOKNczYqUIqk6+DW5ZIkywdXnqxB5tmz7MHrBIRKp6PJtsQG4ojJa
 +xLVJDpD5kfnqfuodhzXzqImzlWEkN90PBHGpDMV+6+2/K86pcE56jfv0OueoiNLUUAsXMhyiK6Gf
 D9aCbEaZ1tNF50LO+OAPALGtkV7CRlxT87CYAyPqpi1IDj/5IDCct3AA1HRN1gACtQxURNkGM9/VZ
 o2J64xE/yBOEVpYAmc4D055G8mOKlOYc3GwH924HyMDUQ1QKXBk31QRd6psGp+8RLf62QSng1UWme
 vh5vCxKZNjNXAY8FzKgxEA==;
In-Reply-To: <96636783-62bb-0bb1-ecb5-c510630402e3@gutov.dev> (message from
 Dmitry Gutov on Sat, 25 Nov 2023 16:05:03 +0200)
X-BeenThere: debbugs-submit@debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
X-BeenThere: bug-gnu-emacs@gnu.org
List-Id: "Bug reports for GNU Emacs,
 the Swiss army knife of text editors" <bug-gnu-emacs.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/bug-gnu-emacs>,
 <mailto:bug-gnu-emacs-request@gnu.org?subject=unsubscribe>
List-Archive: <https://lists.gnu.org/archive/html/bug-gnu-emacs>
List-Post: <mailto:bug-gnu-emacs@gnu.org>
List-Help: <mailto:bug-gnu-emacs-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/bug-gnu-emacs>,
 <mailto:bug-gnu-emacs-request@gnu.org?subject=subscribe>
Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org
Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org
Xref: news.gmane.io gmane.emacs.bugs:274979
Archived-At: <http://permalink.gmane.org/gmane.emacs.bugs/274979>

> Date: Sat, 25 Nov 2023 16:05:03 +0200
> Cc: sbaugh@janestreet.com, 67310@debbugs.gnu.org, juri@linkov.net
> From: Dmitry Gutov <dmitry@gutov.dev>
> 
> > I generally request and expect people not to make unnecessary changes,
> > since doing that makes forensics harder: you see changes which don't
> > change the code's semantics, and need to waste time studying such
> > "changes" and deciding that they are no-ops.
> > 
> > Please everyone keep this in mind when you make changes.
> 
> In general we don't frown in making minor cosmetic changes in the same 
> area as major meaningful changes are done.

Yes, if the changes are for the better.  In this case, they seem to be
due to someone's personal stylistic preferences, so their value is
questionable at best.

> > I think this is not a good idea, regardless of the reasons.  Users
> > have these options to control how history functionality behaves in
> > Emacs, and here you take away that control with no "fire escape".
> 
> Actually, never mind: the latest version of the patch doesn't use 
> 'add-to-history', or reference these variables.

Good.