From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.ciao.gmane.io!not-for-mail From: Alan Mackenzie Newsgroups: gmane.emacs.bugs Subject: bug#41897: 28.0.50; JavaScript comment filling with mhtml-mode Date: Tue, 23 Jun 2020 19:17:13 +0000 Message-ID: <20200623191713.GC6957@ACM> References: <20200620171827.7855.qmail@mail.muc.de> <87d05ta8z9.fsf@simenheg@gmail.com> <20200622191750.GA11506@ACM> <20200623083613.GA6957@ACM> <20200623162837.GB6957@ACM> <10235ec5-17c3-c281-b5ed-2c65a07bd02f@yandex.ru> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Injection-Info: ciao.gmane.io; posting-host="ciao.gmane.io:159.69.161.202"; logging-data="124058"; mail-complaints-to="usenet@ciao.gmane.io" Cc: Simen =?UTF-8?Q?Heggest=C3=B8yl?= , acm@muc.de, 41897@debbugs.gnu.org To: Dmitry Gutov Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Tue Jun 23 21:18:20 2020 Return-path: 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 ) id 1jnoR2-000WAQ-HD for geb-bug-gnu-emacs@m.gmane-mx.org; Tue, 23 Jun 2020 21:18:20 +0200 Original-Received: from localhost ([::1]:60062 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jnoR1-0001Xr-Gb for geb-bug-gnu-emacs@m.gmane-mx.org; Tue, 23 Jun 2020 15:18:19 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:59884) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jnoQk-0001We-Py for bug-gnu-emacs@gnu.org; Tue, 23 Jun 2020 15:18:03 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:53810) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1jnoQk-0002I1-6I for bug-gnu-emacs@gnu.org; Tue, 23 Jun 2020 15:18:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1jnoQk-0006Q9-2S for bug-gnu-emacs@gnu.org; Tue, 23 Jun 2020 15:18:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Alan Mackenzie Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 23 Jun 2020 19:18:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 41897 X-GNU-PR-Package: emacs Original-Received: via spool by 41897-submit@debbugs.gnu.org id=B41897.159293984324618 (code B ref 41897); Tue, 23 Jun 2020 19:18:02 +0000 Original-Received: (at 41897) by debbugs.gnu.org; 23 Jun 2020 19:17:23 +0000 Original-Received: from localhost ([127.0.0.1]:37123 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jnoQ6-0006P0-Oj for submit@debbugs.gnu.org; Tue, 23 Jun 2020 15:17:22 -0400 Original-Received: from colin.muc.de ([193.149.48.1]:41394 helo=mail.muc.de) by debbugs.gnu.org with smtp (Exim 4.84_2) (envelope-from ) id 1jnoQ4-0006Oc-FN for 41897@debbugs.gnu.org; Tue, 23 Jun 2020 15:17:21 -0400 Original-Received: (qmail 24749 invoked by uid 3782); 23 Jun 2020 19:17:13 -0000 Original-Received: from acm.muc.de (p4fe157be.dip0.t-ipconnect.de [79.225.87.190]) by localhost.muc.de (tmda-ofmipd) with ESMTP; Tue, 23 Jun 2020 21:17:12 +0200 Original-Received: (qmail 26962 invoked by uid 1000); 23 Jun 2020 19:17:13 -0000 Content-Disposition: inline In-Reply-To: <10235ec5-17c3-c281-b5ed-2c65a07bd02f@yandex.ru> X-Delivery-Agent: TMDA/1.1.12 (Macallan) X-Primary-Address: acm@muc.de 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" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.io gmane.emacs.bugs:182324 Archived-At: Hello, Dmitry. On Tue, Jun 23, 2020 at 20:59:22 +0300, Dmitry Gutov wrote: > Hi Alan, > On 23.06.2020 19:28, Alan Mackenzie wrote: > >> So it's really fine if it's called from HTML/CSS hunks as well? > > Not only fine, but necessary. The literal cache contains entries > > that record things like "C comment between positions 23 and 130". If > > somebody inserts text before that comment, or inside of it, that > > cache entry is no longer valid, and must be invalidated. Hence the > > necessity of the before-change function. > But isn't CC Mode confused by chunks of text with a totally different > syntax? It might well be, probably is. But this isn't CC Mode we're talking about - just a tiny part of its low level functionality, namely the bit dealing with literals and filling them. > >> And there's no way to just "reset" it to an appropriate value? > > No. Not without killing its utility as a cache. > What do you mean? Even if the cache is reset at the beginning of a > function, if the function refers to it multiple times, the first time > should refill the cache, and the rest of the calls will be able to make > use of it properly. That's what I mean. The cache persists over commands, reducing the amount of recalculation needed, particularly for fast typing. Refilling it from scratch on every keypress would likely make it sluggish. Anyhow, it works fine at the moment, so why change it? [ .... ] > >> mmm-mode is a minor mode, it doesn't always deal with CC Mode. > > The question to consider here is whether any sub-mode of mmm-mode > > uses CC Mode's comment filling without initialising CC Mode. js-mode > > and mhtml-mode do this. > js-mode can be one of its submodes. c-mode as well, but none of CC Mode > family of major modes ever worked okay with it, I think. Having several major modes in a single buffer has always been problematic in Emacs. Personally, I think there needs to be amendments in the low-level C code to support it properly, but I'm not able to do this work on my own, and there doesn't seem to be enough enthusiasm on other people's part to help out. > js-mode mostly works, aside from features like this one. With the current patch, comment filling should work fine in js-mode. > >> Have you considered adding variables that hold the cache to > >> mhtml--crucial-variable-prefix as well? Would that make it work? > > Not without the before-change function, no. I'm trying to see what the > > point of putting these variables into mhtml's crucial variables would be. > Hopefully, it would make the submode regions inside independent > "islands", so to speak. Each of them having its own cache structure > (used or not). Ah, OK. So, buffer positions would be offsets from the island start, or something like that. > TBH I'm not sure if mhtml-mode does the save-and-restore dance which > would be necessary for this. mmm-mode does, though. mhtml-mode does do saving and restoring of local variables. I can't judge how well in comparison with, say, mmm-mode. -- Alan Mackenzie (Nuremberg, Germany).