From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.bugs Subject: bug#34671: 27.0.50; Outdated code listings in (elisp) Example Major Modes Date: Sat, 02 Mar 2019 09:39:27 +0200 Message-ID: <83mumdhgi8.fsf@gnu.org> References: <87bm2yqyhj.fsf@tcd.ie> <877edmqy19.fsf@tcd.ie> <83imx2j3xn.fsf@gnu.org> <87d0naml7s.fsf@tcd.ie> Injection-Info: blaine.gmane.org; posting-host="blaine.gmane.org:195.159.176.226"; logging-data="204433"; mail-complaints-to="usenet@blaine.gmane.org" Cc: 34671@debbugs.gnu.org To: "Basil L. Contovounesios" Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Sat Mar 02 08:40:30 2019 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([209.51.188.17]) by blaine.gmane.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1gzzG0-000r44-OV for geb-bug-gnu-emacs@m.gmane.org; Sat, 02 Mar 2019 08:40:28 +0100 Original-Received: from localhost ([127.0.0.1]:49747 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gzzFz-0003bM-OV for geb-bug-gnu-emacs@m.gmane.org; Sat, 02 Mar 2019 02:40:27 -0500 Original-Received: from eggs.gnu.org ([209.51.188.92]:33124) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gzzFt-0003ZO-Bu for bug-gnu-emacs@gnu.org; Sat, 02 Mar 2019 02:40:22 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gzzFl-0008DU-Vh for bug-gnu-emacs@gnu.org; Sat, 02 Mar 2019 02:40:17 -0500 Original-Received: from debbugs.gnu.org ([209.51.188.43]:43414) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gzzFa-00082G-Mm for bug-gnu-emacs@gnu.org; Sat, 02 Mar 2019 02:40:07 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1gzzFa-0001KQ-BV for bug-gnu-emacs@gnu.org; Sat, 02 Mar 2019 02:40:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Eli Zaretskii Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Sat, 02 Mar 2019 07:40:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 34671 X-GNU-PR-Package: emacs Original-Received: via spool by 34671-submit@debbugs.gnu.org id=B34671.15515123955085 (code B ref 34671); Sat, 02 Mar 2019 07:40:02 +0000 Original-Received: (at 34671) by debbugs.gnu.org; 2 Mar 2019 07:39:55 +0000 Original-Received: from localhost ([127.0.0.1]:56958 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1gzzFT-0001Jx-Es for submit@debbugs.gnu.org; Sat, 02 Mar 2019 02:39:55 -0500 Original-Received: from eggs.gnu.org ([209.51.188.92]:46598) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1gzzFR-0001Jl-O8 for 34671@debbugs.gnu.org; Sat, 02 Mar 2019 02:39:54 -0500 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]:37462) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gzzFF-0007EY-Nw; Sat, 02 Mar 2019 02:39:43 -0500 Original-Received: from [176.228.60.248] (port=1883 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1gzzFD-00013t-S6; Sat, 02 Mar 2019 02:39:41 -0500 In-reply-to: <87d0naml7s.fsf@tcd.ie> (contovob@tcd.ie) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.org gmane.emacs.bugs:155945 Archived-At: > From: "Basil L. Contovounesios" > Cc: <34671@debbugs.gnu.org> > Date: Fri, 01 Mar 2019 19:46:31 +0000 > > Eli Zaretskii writes: > > >> The first patch reconciles the code listings in the manual with the > >> current state of the corresponding libraries. > > > > This part is definitely OK, please push to the emacs-26 branch. > > Sorry, I don't have write access, so someone else will have to do that. Done. > BTW, is one expected to be a package author before it is acceptable to > request write access, or is it a matter of amassing enough flight hours? The latter. I think you are there already, so feel free to create a user on Savannah and then request write access. > >> The second patch enables lexical-binding in text-mode.el along with some > >> minor aesthetic changes. > > > > I don't see the need for parts of this patch. Enabling > > lexical-binding is OK, but it should be a separate change unrelated to > > this bug report. > > Should I submit a new bug report for that, or message emacs-devel? The latter, I think, since the main issue here is how well was the package tested after turning on lexical-binding. > > And I'm not sure I see the reason for the other changes, nor even > > agree with them. In particular, please modify whitespace only where > > you actually make other non-trivial changes. > > (Oops, looking back I accidentally mangled some Texinfo escape > characters by copy-pasting.) > > I agree with this principle, but simply thought that a file-wide change > like enabling lexical-binding was sufficient excuse for minor "cleanups" > along the way. Not in such a sweeping manner. In general, only in the same function/code fragment where you are making changes. > I assume the parts of the patch you're least keen on are: > > * lisp/textmodes/text-mode.el (text-mode-syntax-table): Align comments > to comment-column. > (toggle-text-mode-auto-fill): Hoist save-current-buffer out of loop. > (center-region): Tiny simplification. > > but that you're okay with the following: > > * lisp/textmodes/text-mode.el: Use lexical-binding. > (text-mode-map, text-mode): Refill docstring. > (text-mode, paragraph-indent-minor-mode, text-mode-hook-identify): > Use setq-local. > (center-line): Tiny simplification. Refilling doc strings also caused me to raise a brow. IMO, that is only justified if the original doc string is badly formatted, like has overly-long lines, close to or longer than 80 columns. Otherwise, we don't generally refill doc strings we don't change, we only make sure they look well and read clearly on the screen. > >> The last patch fulfils an old promise in the manual to eventually forgo > >> setting indent-line-function in text-mode, which is considered > >> redundant. > > > > What if the user customizes the default values, shouldn't text-mode > > reset that in the buffer where it is turned on? > > I can think of two reasons for keeping the reset in text-mode: > > 0. If text-mode breaks when indent-line-function is set to anything > other than indent-relative. > > I'm not sure, but I don't think this is the case, as I set the > variable in text-mode-hook for a while without any noticeable > fallout. > > 1. To avoid a user-visible change of behaviour with user configurations > (like mine) which change the default global value of > indent-line-function. > > The only reason (I can think of) to remove the reset is in order to > fulfil the relevant promise in the manual. > > Keeping the reset is clearly less risky. I don't mind either way, so > long as the side-note in the manual accurately reflects future > intentions. Maybe we should make the change you propose, but it isn't clear-cut and requires discussion in a separate bug report, or perhaps even on emacs-devel. It certainly isn't cleanup. Thanks.