From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Lars Ingebrigtsen Newsgroups: gmane.emacs.bugs Subject: bug#57266: Maintaining the base_line_number cache Date: Tue, 23 Aug 2022 16:21:55 +0200 Message-ID: <8735dnrqf0.fsf@gnus.org> References: <831qteccli.fsf@gnu.org> <834jy4bhqp.fsf@gnu.org> <83y1vga0l5.fsf@gnu.org> <83edx89rl4.fsf@gnu.org> 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="2185"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/29.0.50 (gnu/linux) Cc: 57266@debbugs.gnu.org, Stefan Monnier To: Eli Zaretskii Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Tue Aug 23 16:23:19 2022 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 1oQUoJ-0000Mp-3G for geb-bug-gnu-emacs@m.gmane-mx.org; Tue, 23 Aug 2022 16:23:19 +0200 Original-Received: from localhost ([::1]:38704 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1oQUoI-0004gr-5z for geb-bug-gnu-emacs@m.gmane-mx.org; Tue, 23 Aug 2022 10:23:18 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:42404) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oQUo3-0004ej-Fb for bug-gnu-emacs@gnu.org; Tue, 23 Aug 2022 10:23:03 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:54974) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1oQUo1-0007Nu-Un for bug-gnu-emacs@gnu.org; Tue, 23 Aug 2022 10:23:03 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1oQUo1-0006gR-QX for bug-gnu-emacs@gnu.org; Tue, 23 Aug 2022 10:23:01 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Lars Ingebrigtsen Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 23 Aug 2022 14:23:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 57266 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 57266-submit@debbugs.gnu.org id=B57266.166126452925614 (code B ref 57266); Tue, 23 Aug 2022 14:23:01 +0000 Original-Received: (at 57266) by debbugs.gnu.org; 23 Aug 2022 14:22:09 +0000 Original-Received: from localhost ([127.0.0.1]:44717 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1oQUnB-0006f4-AL for submit@debbugs.gnu.org; Tue, 23 Aug 2022 10:22:09 -0400 Original-Received: from quimby.gnus.org ([95.216.78.240]:38862) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1oQUn8-0006eZ-Mq for 57266@debbugs.gnu.org; Tue, 23 Aug 2022 10:22:08 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnus.org; s=20200322; h=Content-Transfer-Encoding:Content-Type:MIME-Version:Message-ID :Date:References:In-Reply-To:Subject:Cc:To:From:Sender:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=SmKcCH6wVfAlMcEcjysnrDeHt7YBWDtCYvUpVQdQMfI=; b=YXd+dGDnPcRwaCs8Z/27Lr4d9v 3mTM7dg3OQ7XDKTWK9pgdJnoQJJ7Hyjg+ntJkDSc+DEp4K0yKi86LuLobqyjVcJgT2iAoy8ocHfxc Yaj+pzCVS+sQLEF9R5vqS0gMqe8p66mH4vxYw2AIpLbF6NZ8msBm3HEwizffaaCkfGbU=; Original-Received: from [84.212.220.105] (helo=joga) by quimby.gnus.org with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1oQUmy-0000lJ-CS; Tue, 23 Aug 2022 16:21:58 +0200 In-Reply-To: (Stefan Monnier via's message of "Mon, 22 Aug 2022 14:02:44 -0400") Face: iVBORw0KGgoAAAANSUhEUgAAADAAAAAwBAMAAAClLOS0AAAABGdBTUEAALGPC/xhBQAAACBj SFJNAAB6JgAAgIQAAPoAAACA6AAAdTAAAOpgAAA6mAAAF3CculE8AAAAFVBMVEWUcpy2lqNPNEQt GFupX1e/hnv///9ng8G/AAAAAWJLR0QGYWa4fQAAAAd0SU1FB+YIFw4SDIqrZyoAAAFcSURBVDjL ldPNbsMgDABg8tN72Jp7ati9al4gRea+Vfb7v8pskimB0MNQD1G+2gbjGJNW0wKkh3beltkWAEw1 6COtIV0JzPQ/APTkazVCROZahJdUXIloPY/0XQEAZFrOIMfG+AYiYQnNoACe3wI2djiC3YDwWQVE XgZTAYiMeQ1rTb8CNSUYuMkJWXMVEL5+RsYawNMSnGAQAHuVZJUItFepnp9ci0McX9KuvIkbkAfp Y97dXoG9bCuDQWEZOcQC0slvrEDni1LwJ4AVgkxQBtrDJ/PUSpEM2gR0BmlIuPBrkqnLQDPhhYMA nWEk8B5DVjxsAB7hDPJ5AIbpCOnCgYIGwKHGlEYEGSVAsu2Q/i+NfWmA7LuAJQYf0mMZgdv7HeSz 71sBXN+7HWRKrlokltCI6LY2uHfmPplJfqbX9v6VcGa2N5n6j7X4vpxe9WC6TwX3OMDDzW5+zO4X iqyiS72eP0EAAAAldEVYdGRhdGU6Y3JlYXRlADIwMjItMDgtMjNUMTQ6MTg6MTIrMDA6MDACQ5Oh AAAAJXRFWHRkYXRlOm1vZGlmeQAyMDIyLTA4LTIzVDE0OjE4OjEyKzAwOjAwcx4rHQAAAABJRU5E rkJggg== X-Now-Playing: No Bra's _Candy_: "Construction Worker" 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:240546 Archived-At: Stefan Monnier via "Bug reports for GNU Emacs, the Swiss army knife of text editors" writes: > Indeed, my proposed patch is not a bug fix. > It's a code-maintenance patch. It's meant to improve the code, not > the behavior. My two cents: I was reading through parts of the newline (cache) code last week while I was pondering whether to do the `pos-eol' functions or not, and I have to say that I found it pretty impregnable. So in a way I'm relieved to learn that it's (partly) because the code doesn't make as much sense as it should. =F0=9F=98=80 Apparently Eli is the only person that understands the code at present, and I understand his reluctance to change something that works. But making the code easier to understand would enable more people to actually handle the code. So I'm in favour of improving the code.