From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: [PATCH] lisp/bookmark.el: make bookmark-fontify nil by default Date: Tue, 18 May 2021 11:45:10 +0300 Message-ID: References: <1df50c0d035675f851387d689030d588@purelymail.com> <87k0nw1qaq.fsf@red-bean.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="32977"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: K-9 Mail for Android Cc: Emacs Devel To: "Paul W. Rankin" , "Paul W. Rankin via Emacs development discussions." , Karl Fogel Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Tue May 18 10:46:37 2021 Return-path: Envelope-to: ged-emacs-devel@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 ) id 1livN6-0008Jn-Mv for ged-emacs-devel@m.gmane-mx.org; Tue, 18 May 2021 10:46:36 +0200 Original-Received: from localhost ([::1]:39026 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1livN5-0000tQ-9q for ged-emacs-devel@m.gmane-mx.org; Tue, 18 May 2021 04:46:35 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:50356) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1livLp-0007Bp-BJ for emacs-devel@gnu.org; Tue, 18 May 2021 04:45:17 -0400 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]:57076) by eggs.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1livLo-0007cn-G0; Tue, 18 May 2021 04:45:16 -0400 Original-Received: from [2a02:14f:1fb:5a12::1c44:63b9] (port=33580) by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1livLm-0007cU-HL; Tue, 18 May 2021 04:45:16 -0400 In-Reply-To: Autocrypt: addr=eliz@gnu.org; keydata= mQENBF+pf4UBCAC6vjkWLSAsQpe8YIGKLQzNOJx/IjGtCdFF8uzmO5jmME+SD8ROuJN+t5KXVw58 uzu75EFD0vHTY9e+udJ2gkpuy0NnzkFcbumdLLo2ERKCoSctZZRhzKXI5z5cHxCqW0B2ygHRrRLt oNlGID7bAgcgSViT1ptGqTXO7zGVu4Airok7dNzcPtHgns8GlR5YAFX0TvE6oGd0l2VPghNeVJKJ OjrbfhoDxl3ucFpqbqMH8z9HTLDOFpz8UaYYUdJMi3xX6vwTZxI2sM2RRVLUpZyllAkSMI4lln1O OgazM/62DJUs/rKIHKBnF6h3/qsJUjUYXaAHbrXY26mWllAd536lABEBAAG0I0VsaSBaYXJldHNr aWkgKGVsaXopIDxlbGl6QGdudS5vcmc+iQE4BBMBAgAiBQJfqX+FAhsDBgsJCAcDAgYVCAIJCgsE FgIDAQIeAQIXgAAKCRCRwSYvAeuNOYUQB/4/iIKKOG45ijNaRoTvmJJZMvj1S07WQxEm7c5SHEeE QbLOAxB9vESOV7sLueuN3oqEndtzyYt4x1WTSBmHFF7h5fcCMjBs41siOIp5Sj/xD0Bvaa0IKGCR SZ7PAo8Mq3wgajXpTpn9vxE2PmtzA8KdEE0K1+f9pVAfOpUIcCl44rIxLUW352XG0y7iz6c/O6LB 1deOKMiKFctKO7pBti1dJEm1ImewLH3H8uTbwspLOs3EB8xhsESxmTidnze68HX2jt+2EeMgCdki NU+LWbexQZPfIS7+ZmE06ll0v6+Jy7ZdTkCCRypKWTnW7pIFsq/p4kybV8O/kHSV6B4vvQBfuQEN BF X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.io gmane.emacs.devel:269436 Archived-At: On May 18, 2021 11:17:04 AM GMT+03:00, "Paul W=2E Rankin via Emacs developm= ent discussions=2E" wrote: > On 2021-05-18 16:58, Karl Fogel wrote: > > On 18 May 2021, Paul W=2E Rankin" via "Emacs development discussions= =2E=20 > > wrote: > >> Ugh=2E Not sure how this slipped into master but it is not a > >> good/considerate idea to set default value of bookmark-fontify to > t=2E > >>=20 > >> If there are no reasonable objections I'll push this later today=2E > >=20 > > I don't have a strong opinion either way, but if you're curious as > to > > where this came from, it was here: > >=20 > > The discussion in > https://debbugs=2Egnu=2Eorg/cgi/bugreport=2Ecgi?bug=3D48179 > > is quite long, so it might be worth reading before reverting=2E >=20 > Hi Karl, >=20 > The discussion there only seems to account for compatibility with=20 > package bm=2Eel, not the more general "avoid pushing your idea onto=20 > everyone" idea=2E And given the overlay is temporary, it's quite opaque= =20 > for the average person to inspect it to see what's actually happening=2E >=20 > It's so ugly my first thought was that something had broken rather > than=20 > this was actually an intentional feature=2E >=20 > I'm going to commit this without further discussion -- which hopefully >=20 > should remind others to take the more considerate route when > introducing=20 > new features! Please don't make this change until you hear from Lars, who examined the o= riginal patch and decided to install it=2E It doesn't get more "considerat= e" than that=2E