GNU bug report logs - #69920
29.2; gnus: article-mode: Toggling MIME inline attachment previews adds superfluous newlines

Please note: This is a static page, with minimal formatting, updated once a day.
Click here to see this page with the latest information and nicer formatting.

Package: emacs; Reported by: Alcor <alcor@HIDDEN>; dated Wed, 20 Mar 2024 18:11:02 UTC; Maintainer for emacs is bug-gnu-emacs@HIDDEN.

Message received at 69920 <at> debbugs.gnu.org:


Received: (at 69920) by debbugs.gnu.org; 22 Apr 2024 12:42:37 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Mon Apr 22 08:42:37 2024
Received: from localhost ([127.0.0.1]:45918 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1ryt0H-00021V-8G
	for submit <at> debbugs.gnu.org; Mon, 22 Apr 2024 08:42:37 -0400
Received: from tilde.club ([142.44.150.184]:54128 ident=postfix)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <alcor@HIDDEN>) id 1ryt0E-00021N-Jd
 for 69920 <at> debbugs.gnu.org; Mon, 22 Apr 2024 08:42:35 -0400
Received: from topazii.tilde.club (unknown
 [IPv6:2001:9e8:7e0:db00:6372:d642:7ea3:be0c])
 by tilde.club (Postfix) with ESMTPSA id 95E0D22565566;
 Mon, 22 Apr 2024 12:42:16 +0000 (UTC)
DKIM-Filter: OpenDKIM Filter v2.11.0 tilde.club 95E0D22565566
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tilde.club; s=mail;
 t=1713789737; bh=yl7ht51Hyel9C1Zl3iYcrB0JEMSH/LId5Gtudl/xsaY=;
 h=From:To:Cc:Subject:In-Reply-To:References:Date:From;
 b=TMMlpHi0aCN3hQIVxhSZ3vEulSQSC6jFORU6R3TUpGTWChd0rOqH7wWaJ0t35G5Ww
 lmQw1XGXWefAhYtaag/Koxa1mi0JoMZd7YrkYogN1ESRl7lmLWWf8iq48z2C7W7ZKu
 FuH6jw8ucDrxIgaVWdoiBBAGX9CLNkGgULI3McTs=
From: Alcor <alcor@HIDDEN>
To: Eric Abrahamsen <eric@HIDDEN>
Subject: Re: bug#69920: Proposed fix - Toggling MIME inline attachment
 previews adds superfluous newlines
In-Reply-To: <87sezejeoh.fsf@HIDDEN> (Eric Abrahamsen's message of
 "Sun, 21 Apr 2024 19:48:14 -0700")
References: <87wmpuouay.fsf@HIDDEN> <87y1a952gf.fsf@HIDDEN>
 <86sf0hzb5c.fsf@HIDDEN> <874jcxmfzh.fsf@HIDDEN>
 <86bk75z1un.fsf@HIDDEN> <864jce50ju.fsf@HIDDEN> <86bk67f3kq.fsf@HIDDEN>
 <87sezejeoh.fsf@HIDDEN>
Date: Mon, 22 Apr 2024 14:42:09 +0200
Message-ID: <875xw9y3fi.fsf@HIDDEN>
User-Agent: Gnus/5.13 (Gnus v5.13)
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=-=-="
X-Spam-Score: -0.0 (/)
X-Debbugs-Envelope-To: 69920
Cc: 69920 <at> debbugs.gnu.org, Eli Zaretskii <eliz@HIDDEN>
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -1.0 (-)

--=-=-=
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

Eric Abrahamsen <eric@HIDDEN> writes:

> I also prefer #2, mostly because the other `mm-inline-*' functions here
> all ensure that inlined content ends with a newline, and even if there's
> no immediate consequence to not having it, I'd prefer the consistency.
> Alcor, would you be willing to update your patch?

Hello Eric, Eli =E2=80=93

Attached is an updated patch adjusting the `delete-region' call in the
undisplayer to account for the additional newline, as discussed earlier.

I have tested this change on Emacs 29.3 (Linux/GTK) with multiple
image/png attachments and can confirm it works.

Cheers,
-A.


--=-=-=
Content-Type: text/x-diff
Content-Disposition: attachment;
 filename=0001-Ensure-that-inline-preview-undisplayer-removes-added.patch

From 31578f2c8d538eb4b944c073d4804f355f88f035 Mon Sep 17 00:00:00 2001
From: "F. Moukayed" <smfadi+emacs@HIDDEN>
Date: Mon, 22 Apr 2024 12:33:08 +0000
Subject: [PATCH] Ensure that inline preview undisplayer removes added newline

* lisp/gnus/mm-view.el (mm-inline-image): Remove the regioninterval b..b+2 (aka "x\n") instead of removing b..b+1 ("x") and leaving behind a superfluous newline (bug#69920).
---
 lisp/gnus/mm-view.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/gnus/mm-view.el b/lisp/gnus/mm-view.el
index 109b6c17c2..223da19a16 100644
--- a/lisp/gnus/mm-view.el
+++ b/lisp/gnus/mm-view.el
@@ -105,7 +105,7 @@ This is only used if `mm-inline-large-images' is set to
      (lambda ()
        (let ((inhibit-read-only t))
 	 (remove-images b b)
-	 (delete-region b (1+ b)))))))
+	 (delete-region b (+ b 2)))))))
 
 (defvar mm-w3m-setup nil
   "Whether gnus-article-mode has been setup to use emacs-w3m.")
-- 
2.34.1


--=-=-=--




Information forwarded to bug-gnu-emacs@HIDDEN:
bug#69920; Package emacs. Full text available.

Message received at 69920 <at> debbugs.gnu.org:


Received: (at 69920) by debbugs.gnu.org; 22 Apr 2024 02:48:43 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Sun Apr 21 22:48:42 2024
Received: from localhost ([127.0.0.1]:45313 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1ryjjV-0003ra-Vd
	for submit <at> debbugs.gnu.org; Sun, 21 Apr 2024 22:48:42 -0400
Received: from mail.ericabrahamsen.net ([52.70.2.18]:41790)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <eric@HIDDEN>) id 1ryjjS-0003qN-7Y
 for 69920 <at> debbugs.gnu.org; Sun, 21 Apr 2024 22:48:40 -0400
Received: from localhost (71-212-21-65.tukw.qwest.net [71.212.21.65])
 (Authenticated sender: eric@HIDDEN)
 by mail.ericabrahamsen.net (Postfix) with ESMTPSA id D26CEFA059;
 Mon, 22 Apr 2024 02:48:15 +0000 (UTC)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericabrahamsen.net;
 s=mail; t=1713754096;
 bh=VIu4XqGlNiwKkrXpCh3D3HkXNBbCEI02N31QKaR8SKA=;
 h=From:To:Cc:Subject:In-Reply-To:References:Date:From;
 b=KPwTKRR2S4hOSOLm+K2NuHekniBUcwgJmhb13Jx2dsidOXfv73NE4EJraWj0MmLWW
 ARi5VXp3kGtGxw4n2HLp9e5ocaYWBqkXttp0mNDyyh92tEgV7wJasK08kBv5uHAyq5
 3plzuNpAwgEWeXMddDPo81HMh6sgUqXcC6YCC89g=
From: Eric Abrahamsen <eric@HIDDEN>
To: Eli Zaretskii <eliz@HIDDEN>
Subject: Re: bug#69920: Proposed fix - Toggling MIME inline attachment
 previews adds superfluous newlines
In-Reply-To: <86bk67f3kq.fsf@HIDDEN> (Eli Zaretskii's message of "Thu, 18 Apr
 2024 12:00:21 +0300")
References: <87wmpuouay.fsf@HIDDEN> <87y1a952gf.fsf@HIDDEN>
 <86sf0hzb5c.fsf@HIDDEN> <874jcxmfzh.fsf@HIDDEN>
 <86bk75z1un.fsf@HIDDEN> <864jce50ju.fsf@HIDDEN>
 <86bk67f3kq.fsf@HIDDEN>
Date: Sun, 21 Apr 2024 19:48:14 -0700
Message-ID: <87sezejeoh.fsf@HIDDEN>
User-Agent: Gnus/5.13 (Gnus v5.13)
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
X-Spam-Score: -2.3 (--)
X-Debbugs-Envelope-To: 69920
Cc: 69920 <at> debbugs.gnu.org, alcor@HIDDEN
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -3.3 (---)


On 04/18/24 12:00 PM, Eli Zaretskii wrote:
> Ping! Ping! Eric, please chime in.

Sorry this has taken so long!

>> Cc: 69920 <at> debbugs.gnu.org, alcor@HIDDEN
>> Date: Sat, 06 Apr 2024 11:59:33 +0300
>> From: Eli Zaretskii <eliz@HIDDEN>
>>=20
>> Ping! Eric, can you please chime in?
>>=20
>> > Cc: 69920 <at> debbugs.gnu.org
>> > Date: Sat, 23 Mar 2024 12:20:16 +0200
>> > From: Eli Zaretskii <eliz@HIDDEN>
>> >=20
>> > > From: Alcor <alcor@HIDDEN>
>> > > Cc: 69920 <at> debbugs.gnu.org
>> > > Date: Sat, 23 Mar 2024 10:53:22 +0100
>> > >=20
>> > > Eli Zaretskii <eliz@HIDDEN> writes:
>> > >=20
>> > > > Are you saying that the problem is with the function that
>> > > > "un-displays" the inline image, in that it fails to remove the
>> > > > inserted newline?  (AFAIU, the code before the above commit also h=
ad
>> > > > the same issue.)  That wasn't clear from the description of the
>> > > > problem, and the Subject is ambiguous wrt what newlines are deemed
>> > > > "superfluous".  So please clarify what is the problem you are flag=
ging
>> > > > here.
>> > >=20
>> > > The problem is that the undisplayer does not correctly undo what the
>> > > displayer does.
>> > >=20
>> > > The undisplayer (as it is right now) just removes the image. That ca=
n be
>> > > confirmed by stepping through the code with edebug or by inspecting =
the
>> > > "b" variable.
>> > >=20
>> > > I think we can agree that (delete-region b (1+ b)) will always delete
>> > > _exactly one_ character, and in that case that would be the properti=
zed
>> > > "x" with the image. That would leave a dangling newline.
>> > >=20
>> > > There are two ways to solve this:
>> > >=20
>> > > 1. Do not add the extra newline (this is what the patch does).
>> > > 2. Remove the extra newline via (delete-region b (+ b 2)) =E2=80=93 =
note that I
>> > > have not tried this, but it would make sense to me.
>> > >=20
>> > > I happen to prefer option #1 as the extra newline does not seem to h=
ave
>> > > any meaningful function. But this is just my own preference (Emacs/g=
nus
>> > > maintainers may wish to retain the extra newline if it serves a vali=
d purpose).
>> >=20
>> > I prefer #2.  Eric, WDYT?

I also prefer #2, mostly because the other `mm-inline-*' functions here
all ensure that inlined content ends with a newline, and even if there's
no immediate consequence to not having it, I'd prefer the consistency.
Alcor, would you be willing to update your patch?

Thanks,
Eric




Information forwarded to bug-gnu-emacs@HIDDEN:
bug#69920; Package emacs. Full text available.

Message received at 69920 <at> debbugs.gnu.org:


Received: (at 69920) by debbugs.gnu.org; 18 Apr 2024 09:00:58 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Thu Apr 18 05:00:57 2024
Received: from localhost ([127.0.0.1]:51162 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rxNdW-00080O-JO
	for submit <at> debbugs.gnu.org; Thu, 18 Apr 2024 05:00:57 -0400
Received: from eggs.gnu.org ([2001:470:142:3::10]:41222)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <eliz@HIDDEN>) id 1rxNdR-0007z6-68
 for 69920 <at> debbugs.gnu.org; Thu, 18 Apr 2024 05:00:53 -0400
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@HIDDEN>)
 id 1rxNd6-0000Kw-FS; Thu, 18 Apr 2024 05:00:28 -0400
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org;
 s=fencepost-gnu-org; h=MIME-version:References:Subject:In-Reply-To:To:From:
 Date; bh=hZg/LdjXGEjuBj6t+ZM12uasA697sf2ngLNF23sNc9M=; b=ctVfqYRkv3V7iJVZy/z2
 CNPa2+oX3OdDZm7r7b+030+bGGBHLIW+jrkjfsqeiMxdB2KnYYqbKyLB9gyVRXa+g6DGueYvf+YeO
 4LYx/aF0T2ZPNtPRx+ACeSNv0LKmLczGf9IM2QVXV+iAxJRzVGDudYYbAeW0LlWCjbbWOos3vvkzN
 7ews+hqCNPf3XdD8cDbCDns7DEhytFNDvcajn/XqyY6Xyi8LFt+7ABL2koFACZvM/N2PvqFv78v1v
 eDZDGhp1yJS+oUsGqCD11SbaBXTQlIyWn3jPwlF/lyn4ZgNwi+aJbREMWDZ3WJF8SNKb4+M9feIbw
 xjwFKKEOXjCx0A==;
Date: Thu, 18 Apr 2024 12:00:21 +0300
Message-Id: <86bk67f3kq.fsf@HIDDEN>
From: Eli Zaretskii <eliz@HIDDEN>
To: eric@HIDDEN
In-Reply-To: <864jce50ju.fsf@HIDDEN> (message from Eli Zaretskii on Sat, 06
 Apr 2024 11:59:33 +0300)
Subject: Re: bug#69920: Proposed fix - Toggling MIME inline attachment previews
 adds superfluous newlines
References: <87wmpuouay.fsf@HIDDEN> <87y1a952gf.fsf@HIDDEN>
 <86sf0hzb5c.fsf@HIDDEN> <874jcxmfzh.fsf@HIDDEN> <86bk75z1un.fsf@HIDDEN>
 <864jce50ju.fsf@HIDDEN>
MIME-version: 1.0
Content-type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
X-Spam-Score: -2.3 (--)
X-Debbugs-Envelope-To: 69920
Cc: 69920 <at> debbugs.gnu.org, alcor@HIDDEN
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -3.3 (---)

Ping! Ping! Eric, please chime in.

> Cc: 69920 <at> debbugs.gnu.org, alcor@HIDDEN
> Date: Sat, 06 Apr 2024 11:59:33 +0300
> From: Eli Zaretskii <eliz@HIDDEN>
> 
> Ping! Eric, can you please chime in?
> 
> > Cc: 69920 <at> debbugs.gnu.org
> > Date: Sat, 23 Mar 2024 12:20:16 +0200
> > From: Eli Zaretskii <eliz@HIDDEN>
> > 
> > > From: Alcor <alcor@HIDDEN>
> > > Cc: 69920 <at> debbugs.gnu.org
> > > Date: Sat, 23 Mar 2024 10:53:22 +0100
> > > 
> > > Eli Zaretskii <eliz@HIDDEN> writes:
> > > 
> > > > Are you saying that the problem is with the function that
> > > > "un-displays" the inline image, in that it fails to remove the
> > > > inserted newline?  (AFAIU, the code before the above commit also had
> > > > the same issue.)  That wasn't clear from the description of the
> > > > problem, and the Subject is ambiguous wrt what newlines are deemed
> > > > "superfluous".  So please clarify what is the problem you are flagging
> > > > here.
> > > 
> > > The problem is that the undisplayer does not correctly undo what the
> > > displayer does.
> > > 
> > > The undisplayer (as it is right now) just removes the image. That can be
> > > confirmed by stepping through the code with edebug or by inspecting the
> > > "b" variable.
> > > 
> > > I think we can agree that (delete-region b (1+ b)) will always delete
> > > _exactly one_ character, and in that case that would be the propertized
> > > "x" with the image. That would leave a dangling newline.
> > > 
> > > There are two ways to solve this:
> > > 
> > > 1. Do not add the extra newline (this is what the patch does).
> > > 2. Remove the extra newline via (delete-region b (+ b 2)) – note that I
> > > have not tried this, but it would make sense to me.
> > > 
> > > I happen to prefer option #1 as the extra newline does not seem to have
> > > any meaningful function. But this is just my own preference (Emacs/gnus
> > > maintainers may wish to retain the extra newline if it serves a valid purpose).
> > 
> > I prefer #2.  Eric, WDYT?
> > 
> > > PS: I'm not sure the original code from
> > > before 14ff920dc885636a763d6ab7f256cc9981c24781 was correct either. It
> > > used to insert "x\n\n" (3 characters) on display (x being the
> > > propertized image) and removed via (delete-region b (+ b 2)) exactly 2
> > > characters. The new code after that revision inserted "x\n" (2
> > > characters, x being the propertized image) and removed via
> > > (delete-region b (1+ b)) exactly 1 character. So it might be possible
> > > that this off-by-one error in `mm-inline-image' has always existed.
> > 
> > Yes, I think so.
> > 
> > 
> > 
> > 
> 
> 
> 
> 




Information forwarded to bug-gnu-emacs@HIDDEN:
bug#69920; Package emacs. Full text available.

Message received at 69920 <at> debbugs.gnu.org:


Received: (at 69920) by debbugs.gnu.org; 6 Apr 2024 08:59:50 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Sat Apr 06 04:59:50 2024
Received: from localhost ([127.0.0.1]:38246 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rt1tt-00070i-MS
	for submit <at> debbugs.gnu.org; Sat, 06 Apr 2024 04:59:50 -0400
Received: from eggs.gnu.org ([2001:470:142:3::10]:57672)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <eliz@HIDDEN>) id 1rt1tq-0006zr-V6
 for 69920 <at> debbugs.gnu.org; Sat, 06 Apr 2024 04:59:48 -0400
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@HIDDEN>)
 id 1rt1tf-0007rK-GH; Sat, 06 Apr 2024 04:59:35 -0400
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org;
 s=fencepost-gnu-org; h=MIME-version:References:Subject:In-Reply-To:To:From:
 Date; bh=xYQNK2Y3PJqfbdA/Ci2rOuZSaT87b6+WQgK3uWkPbeQ=; b=evchYJYOKNd+c2Gjf6wd
 bG/iouJogA+yi7QdNZ4pRrq6lizYCFcq7nK2vr2P/XcoznfShbeSgoZJxkJoP9qhMlIWH7UfIYfud
 qNIt66tD5LuKnHi4BQiKDooGa7jerPcPsqL2/leyaWm2i5d1cJXy9dx47HQFr7EMZcnT9a6WOcPvO
 gegyy7xVe5bdHHwz85Sf3TAuY5j4lyQUOpNxAtuJ3axqx54RaDR0xF/Ai5wB3pMnhNb9vGzL5DKrK
 WjcHNl6yt2GcgfOHwO0SAaagqlr7QbHyWW47KkjOjj1r/K4UXD/9oUNF8dwge9V75xQcKYGr5vp2t
 16Q8orO6plMDTQ==;
Date: Sat, 06 Apr 2024 11:59:33 +0300
Message-Id: <864jce50ju.fsf@HIDDEN>
From: Eli Zaretskii <eliz@HIDDEN>
To: eric@HIDDEN
In-Reply-To: <86bk75z1un.fsf@HIDDEN> (message from Eli Zaretskii on Sat, 23
 Mar 2024 12:20:16 +0200)
Subject: Re: bug#69920: Proposed fix - Toggling MIME inline attachment previews
 adds superfluous newlines
References: <87wmpuouay.fsf@HIDDEN> <87y1a952gf.fsf@HIDDEN>
 <86sf0hzb5c.fsf@HIDDEN> <874jcxmfzh.fsf@HIDDEN> <86bk75z1un.fsf@HIDDEN>
MIME-version: 1.0
Content-type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
X-Spam-Score: -2.3 (--)
X-Debbugs-Envelope-To: 69920
Cc: 69920 <at> debbugs.gnu.org, alcor@HIDDEN
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -3.3 (---)

Ping! Eric, can you please chime in?

> Cc: 69920 <at> debbugs.gnu.org
> Date: Sat, 23 Mar 2024 12:20:16 +0200
> From: Eli Zaretskii <eliz@HIDDEN>
> 
> > From: Alcor <alcor@HIDDEN>
> > Cc: 69920 <at> debbugs.gnu.org
> > Date: Sat, 23 Mar 2024 10:53:22 +0100
> > 
> > Eli Zaretskii <eliz@HIDDEN> writes:
> > 
> > > Are you saying that the problem is with the function that
> > > "un-displays" the inline image, in that it fails to remove the
> > > inserted newline?  (AFAIU, the code before the above commit also had
> > > the same issue.)  That wasn't clear from the description of the
> > > problem, and the Subject is ambiguous wrt what newlines are deemed
> > > "superfluous".  So please clarify what is the problem you are flagging
> > > here.
> > 
> > The problem is that the undisplayer does not correctly undo what the
> > displayer does.
> > 
> > The undisplayer (as it is right now) just removes the image. That can be
> > confirmed by stepping through the code with edebug or by inspecting the
> > "b" variable.
> > 
> > I think we can agree that (delete-region b (1+ b)) will always delete
> > _exactly one_ character, and in that case that would be the propertized
> > "x" with the image. That would leave a dangling newline.
> > 
> > There are two ways to solve this:
> > 
> > 1. Do not add the extra newline (this is what the patch does).
> > 2. Remove the extra newline via (delete-region b (+ b 2)) – note that I
> > have not tried this, but it would make sense to me.
> > 
> > I happen to prefer option #1 as the extra newline does not seem to have
> > any meaningful function. But this is just my own preference (Emacs/gnus
> > maintainers may wish to retain the extra newline if it serves a valid purpose).
> 
> I prefer #2.  Eric, WDYT?
> 
> > PS: I'm not sure the original code from
> > before 14ff920dc885636a763d6ab7f256cc9981c24781 was correct either. It
> > used to insert "x\n\n" (3 characters) on display (x being the
> > propertized image) and removed via (delete-region b (+ b 2)) exactly 2
> > characters. The new code after that revision inserted "x\n" (2
> > characters, x being the propertized image) and removed via
> > (delete-region b (1+ b)) exactly 1 character. So it might be possible
> > that this off-by-one error in `mm-inline-image' has always existed.
> 
> Yes, I think so.
> 
> 
> 
> 




Information forwarded to bug-gnu-emacs@HIDDEN:
bug#69920; Package emacs. Full text available.

Message received at 69920 <at> debbugs.gnu.org:


Received: (at 69920) by debbugs.gnu.org; 23 Mar 2024 10:37:35 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Sat Mar 23 06:37:35 2024
Received: from localhost ([127.0.0.1]:45111 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rnyko-0003FU-LH
	for submit <at> debbugs.gnu.org; Sat, 23 Mar 2024 06:37:35 -0400
Received: from eggs.gnu.org ([209.51.188.92]:57772)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <eliz@HIDDEN>) id 1rnykm-0003FE-3L
 for 69920 <at> debbugs.gnu.org; Sat, 23 Mar 2024 06:37:33 -0400
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@HIDDEN>)
 id 1rnyU9-0005WI-UX; Sat, 23 Mar 2024 06:20:21 -0400
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org;
 s=fencepost-gnu-org; h=MIME-version:References:Subject:In-Reply-To:To:From:
 Date; bh=GEIPIsgdTa2B4NqqPIMOVeWqGZ79Hbbq1vzaOanw8Jo=; b=LYtFRzFG7BRp8NW6Gz+x
 a9eJvYIbjLqC2ye8/flWU2py5iumXeItT6fl37oCeTz6JgBt/Au60LhsqzsKBmFeOtMWvcqi4jFye
 ANWVeYeOUBjBlcoUGeNKyFNEMK0pi9WFI+qYQiBU8D+7q9GmwQgRO//SNJhD5siba7iMUdXgtz9/U
 +MFNZZdqQxd5SR+n9H+JBuHbpHQnB+9zSQjvCIGKQu+dFcRSNQ4o/9Nz+/pJX4VujAXtQ7zgfZp2G
 K3TbH70M+2t91lBlp99wJ5OPXvULRye+APFggtTcZXI+Z5bAr6i5JKMokBZd1kFig3ZZv+CWeyWbT
 uV5XahQ2vRpdww==;
Date: Sat, 23 Mar 2024 12:20:16 +0200
Message-Id: <86bk75z1un.fsf@HIDDEN>
From: Eli Zaretskii <eliz@HIDDEN>
To: Alcor <alcor@HIDDEN>, Eric Abrahamsen <eric@HIDDEN>
In-Reply-To: <874jcxmfzh.fsf@HIDDEN> (message from Alcor on Sat, 23 Mar
 2024 10:53:22 +0100)
Subject: Re: bug#69920: Proposed fix - Toggling MIME inline attachment
 previews adds superfluous newlines
References: <87wmpuouay.fsf@HIDDEN> <87y1a952gf.fsf@HIDDEN>
 <86sf0hzb5c.fsf@HIDDEN> <874jcxmfzh.fsf@HIDDEN>
MIME-version: 1.0
Content-type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
X-Spam-Score: -2.3 (--)
X-Debbugs-Envelope-To: 69920
Cc: 69920 <at> debbugs.gnu.org
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -3.3 (---)

> From: Alcor <alcor@HIDDEN>
> Cc: 69920 <at> debbugs.gnu.org
> Date: Sat, 23 Mar 2024 10:53:22 +0100
> 
> Eli Zaretskii <eliz@HIDDEN> writes:
> 
> > Are you saying that the problem is with the function that
> > "un-displays" the inline image, in that it fails to remove the
> > inserted newline?  (AFAIU, the code before the above commit also had
> > the same issue.)  That wasn't clear from the description of the
> > problem, and the Subject is ambiguous wrt what newlines are deemed
> > "superfluous".  So please clarify what is the problem you are flagging
> > here.
> 
> The problem is that the undisplayer does not correctly undo what the
> displayer does.
> 
> The undisplayer (as it is right now) just removes the image. That can be
> confirmed by stepping through the code with edebug or by inspecting the
> "b" variable.
> 
> I think we can agree that (delete-region b (1+ b)) will always delete
> _exactly one_ character, and in that case that would be the propertized
> "x" with the image. That would leave a dangling newline.
> 
> There are two ways to solve this:
> 
> 1. Do not add the extra newline (this is what the patch does).
> 2. Remove the extra newline via (delete-region b (+ b 2)) – note that I
> have not tried this, but it would make sense to me.
> 
> I happen to prefer option #1 as the extra newline does not seem to have
> any meaningful function. But this is just my own preference (Emacs/gnus
> maintainers may wish to retain the extra newline if it serves a valid purpose).

I prefer #2.  Eric, WDYT?

> PS: I'm not sure the original code from
> before 14ff920dc885636a763d6ab7f256cc9981c24781 was correct either. It
> used to insert "x\n\n" (3 characters) on display (x being the
> propertized image) and removed via (delete-region b (+ b 2)) exactly 2
> characters. The new code after that revision inserted "x\n" (2
> characters, x being the propertized image) and removed via
> (delete-region b (1+ b)) exactly 1 character. So it might be possible
> that this off-by-one error in `mm-inline-image' has always existed.

Yes, I think so.




Information forwarded to bug-gnu-emacs@HIDDEN:
bug#69920; Package emacs. Full text available.

Message received at 69920 <at> debbugs.gnu.org:


Received: (at 69920) by debbugs.gnu.org; 23 Mar 2024 09:55:25 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Sat Mar 23 05:55:25 2024
Received: from localhost ([127.0.0.1]:42183 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rny61-0001BL-2s
	for submit <at> debbugs.gnu.org; Sat, 23 Mar 2024 05:55:25 -0400
Received: from tilde.club ([142.44.150.184]:43224 ident=postfix)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <alcor@HIDDEN>) id 1rny5P-00019W-Lg
 for 69920 <at> debbugs.gnu.org; Sat, 23 Mar 2024 05:54:48 -0400
Received: from topazii.tilde.club (unknown
 [IPv6:2001:9e8:7d5:6500:2748:b86f:523a:558c])
 by tilde.club (Postfix) with ESMTPSA id DB034221B617F;
 Sat, 23 Mar 2024 09:53:34 +0000 (UTC)
DKIM-Filter: OpenDKIM Filter v2.11.0 tilde.club DB034221B617F
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tilde.club; s=mail;
 t=1711187615; bh=Wp1Kscnkemra0HHK4qchg1zEnbYxvsc7ld592YkFRDQ=;
 h=From:To:Cc:Subject:In-Reply-To:References:Date:From;
 b=ta5IwnWKLasHWXFrekHpV6Ybgd5AZgFNMZ3Q8kabtFEZpfpCAhY2HtDGJ7l3Uh4tO
 fK6sr3ohp8FSzZZpBoxsli94CgnRQbKC04cw5AnAdzwGyukg/mBsM5rlOpYGeJuFQI
 l7Rh2n24O5uYIMMOtg/ZJdox7JMC8icuGtkLnVXQ=
From: Alcor <alcor@HIDDEN>
To: Eli Zaretskii <eliz@HIDDEN>
Subject: Re: bug#69920: Proposed fix - Toggling MIME inline attachment
 previews adds superfluous newlines
In-Reply-To: <86sf0hzb5c.fsf@HIDDEN> (Eli Zaretskii's message of "Sat, 23 Mar
 2024 08:59:27 +0200")
References: <87wmpuouay.fsf@HIDDEN> <87y1a952gf.fsf@HIDDEN>
 <86sf0hzb5c.fsf@HIDDEN>
Date: Sat, 23 Mar 2024 10:53:22 +0100
Message-ID: <874jcxmfzh.fsf@HIDDEN>
User-Agent: Gnus/5.13 (Gnus v5.13)
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
X-Spam-Score: -0.0 (/)
X-Debbugs-Envelope-To: 69920
Cc: 69920 <at> debbugs.gnu.org
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -1.0 (-)

Eli Zaretskii <eliz@HIDDEN> writes:

> Are you saying that the problem is with the function that
> "un-displays" the inline image, in that it fails to remove the
> inserted newline?  (AFAIU, the code before the above commit also had
> the same issue.)  That wasn't clear from the description of the
> problem, and the Subject is ambiguous wrt what newlines are deemed
> "superfluous".  So please clarify what is the problem you are flagging
> here.

The problem is that the undisplayer does not correctly undo what the
displayer does.

The undisplayer (as it is right now) just removes the image. That can be
confirmed by stepping through the code with edebug or by inspecting the
"b" variable.

I think we can agree that (delete-region b (1+ b)) will always delete
_exactly one_ character, and in that case that would be the propertized
"x" with the image. That would leave a dangling newline.

There are two ways to solve this:

1. Do not add the extra newline (this is what the patch does).
2. Remove the extra newline via (delete-region b (+ b 2)) =E2=80=93 note th=
at I
have not tried this, but it would make sense to me.

I happen to prefer option #1 as the extra newline does not seem to have
any meaningful function. But this is just my own preference (Emacs/gnus
maintainers may wish to retain the extra newline if it serves a valid purpo=
se).

If it helps clarify things, I'm okay with renaming the bug report to
something like "MIME inline image preview undisplayer does not clean up
displayed image correctly" or something along these lines. When I filed
the report, I wasn't sure about the cause so I described the user-facing
issue instead of the actual technical problem.

Cheers,
-A.

PS: I'm not sure the original code from
before 14ff920dc885636a763d6ab7f256cc9981c24781 was correct either. It
used to insert "x\n\n" (3 characters) on display (x being the
propertized image) and removed via (delete-region b (+ b 2)) exactly 2
characters. The new code after that revision inserted "x\n" (2
characters, x being the propertized image) and removed via
(delete-region b (1+ b)) exactly 1 character. So it might be possible
that this off-by-one error in `mm-inline-image' has always existed.




Information forwarded to bug-gnu-emacs@HIDDEN:
bug#69920; Package emacs. Full text available.

Message received at 69920 <at> debbugs.gnu.org:


Received: (at 69920) by debbugs.gnu.org; 23 Mar 2024 07:09:18 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Sat Mar 23 03:09:18 2024
Received: from localhost ([127.0.0.1]:58873 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rnvVG-0001hr-0O
	for submit <at> debbugs.gnu.org; Sat, 23 Mar 2024 03:09:18 -0400
Received: from eggs.gnu.org ([209.51.188.92]:55116)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <eliz@HIDDEN>) id 1rnvVA-0001gh-Lo
 for 69920 <at> debbugs.gnu.org; Sat, 23 Mar 2024 03:09:12 -0400
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@HIDDEN>)
 id 1rnvFc-0005s5-Vd; Sat, 23 Mar 2024 02:53:10 -0400
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=fePJQX+xvK+XZ9WBHHFSvw+rJTGAdW9IFI0yV4dnP+w=; b=d5Lr33oKuaUB
 RR83NX2htvlTET4c0FoVvovY+cqPQnRfs1rMtOXNTKJDePza8RJjU1fySXcd5XLzidN0TbcMridEE
 n+LStL8L2vSsCFbPhO7fq6VXF9nuR+rTLR9tdCDMa86ZlLBzuB8OF9hDf993RNYUmSBsFefs4YSF5
 tfG9ZRDX0+ytNbFzYeGoIeByhkgSJiH61sYQWVjY0e1iqjERmEzGJI6+BHf2E/oazjOrZUH2f7yTE
 yKG7mkWt13DKcKybPbR6QPbctYEexji8LS1UrBD+mYd+IkhsrMByEdlKQX1TBuOBDhN43DoKa7UcE
 Pd/pYcRv0jmnx88/imeiAA==;
Date: Sat, 23 Mar 2024 08:52:56 +0200
Message-Id: <86ttkxzbg7.fsf@HIDDEN>
From: Eli Zaretskii <eliz@HIDDEN>
To: Alcor <alcor@HIDDEN>, Eric Abrahamsen <eric@HIDDEN>
In-Reply-To: <87wmpuouay.fsf@HIDDEN> (bug-gnu-emacs@HIDDEN)
Subject: Re: bug#69920: Proposed fix - Toggling MIME inline attachment previews
 adds superfluous newlines
References: <87bk78pywn.fsf@HIDDEN> <87wmpuouay.fsf@HIDDEN>
X-Spam-Score: -2.3 (--)
X-Debbugs-Envelope-To: 69920
Cc: 69920 <at> debbugs.gnu.org
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -3.3 (---)

> Date: Fri, 22 Mar 2024 22:01:09 +0100
> From:  Alcor via "Bug reports for GNU Emacs,
>  the Swiss army knife of text editors" <bug-gnu-emacs@HIDDEN>
> 
> After some investigation, I noticed that `mm-inline-image' adds a
> newline via (insert "\n") directly after `insert-image' for unclear
> reasons.

I'm guessing this is to make sure text following the image starts on a
new line, rather than after the image.

Why is the current code a problem?  IOW, when does the extra newline
cause any harm?  You talk about "the issue", but I didn't see any
explanation what that issue is.

Eric, any comments?




Information forwarded to bug-gnu-emacs@HIDDEN:
bug#69920; Package emacs. Full text available.

Message received at 69920 <at> debbugs.gnu.org:


Received: (at 69920) by debbugs.gnu.org; 23 Mar 2024 07:09:18 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Sat Mar 23 03:09:17 2024
Received: from localhost ([127.0.0.1]:58870 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rnvVF-0001hi-HF
	for submit <at> debbugs.gnu.org; Sat, 23 Mar 2024 03:09:17 -0400
Received: from eggs.gnu.org ([209.51.188.92]:55116)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <eliz@HIDDEN>) id 1rnvV9-0001gh-Nf
 for 69920 <at> debbugs.gnu.org; Sat, 23 Mar 2024 03:09:12 -0400
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@HIDDEN>)
 id 1rnvLm-0006jy-UN; Sat, 23 Mar 2024 02:59:30 -0400
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org;
 s=fencepost-gnu-org; h=MIME-version:References:Subject:In-Reply-To:To:From:
 Date; bh=qJ2GmKxjvRxda/1x6PcPjMnnmSkaYZyBrExiiLfZAqc=; b=QhtePOtv0/MRJCKTsRse
 +JvAx2oeL0HcH705Fy16Oj5+12zYywjTNgBIXkQ4FFHGd249jmMhelwXLXcQXw/aniDc+SF3H+9gZ
 qOWbH8hXlFTwsOsEsV6Gjd3r4JNbFCH/gqAWdu9xN19NjOTOPCGf9W1YYG9QPB5x5yfay4oWbC2nc
 yxsJ6f2OG9ExQbDqZT0b5dvDxJySQBruf+zGfaUzuaBjUKn845kZcuG87Wtxt+YtVFHDhMQzRPr2x
 Y0ga62EfoPv7DBDvvSXdb4/Hj+yNdPjCLUZm3z29IW55qWMumrJsShTMVyNjVdoezCSYHO/b8DdYI
 Wi/051oxnl4PAQ==;
Date: Sat, 23 Mar 2024 08:59:27 +0200
Message-Id: <86sf0hzb5c.fsf@HIDDEN>
From: Eli Zaretskii <eliz@HIDDEN>
To: Alcor <alcor@HIDDEN>
In-Reply-To: <87y1a952gf.fsf@HIDDEN> (bug-gnu-emacs@HIDDEN)
Subject: Re: bug#69920: Proposed fix - Toggling MIME inline attachment previews
 adds superfluous newlines
References: <87wmpuouay.fsf@HIDDEN> <87y1a952gf.fsf@HIDDEN>
MIME-version: 1.0
Content-type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
X-Spam-Score: -2.3 (--)
X-Debbugs-Envelope-To: 69920
Cc: 69920 <at> debbugs.gnu.org
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -3.3 (---)

> Date: Fri, 22 Mar 2024 23:25:20 +0100
> From:  Alcor via "Bug reports for GNU Emacs,
>  the Swiss army knife of text editors" <bug-gnu-emacs@HIDDEN>
> 
> git blame'ing the line in question on yields a decade-old
> commit. Apparently, this used to be (insert "\n\n") … (delete-region b
> (+ b 2)) about ~10 years ago, and the last relevant change turned that
> into (insert "\n") … (delete-region b (1+ b)), but it seems lots of
> other things on that codepath changed in the meantime.

That commit explains the rationale:

  commit 14ff920dc885636a763d6ab7f256cc9981c24781
  Author:     Katsumi Yamaoka <yamaoka@HIDDEN>
  AuthorDate: Fri May 2 09:44:34 2014 +0000
  Commit:     Katsumi Yamaoka <yamaoka@HIDDEN>
  CommitDate: Fri May 2 09:44:34 2014 +0000

      Gnus: Make gnus-mime-inline-part and gnus-mime-inline-part work similarly

> FWIW, (insert-image … "x") (insert "\n") inserts two characters,
> i.e. the image itself "x" and "\n"

The code inserts the character "x" with a 'display' property on it
(which causes the image to be shown instead of "x"), followed by the
newline.

> so it should either be (insert "\n") … (delete-region b (+ 2 b)) or,
> alternatively just (delete-region b (1+ b)) (without any `insert'
> call).

Are you saying that the problem is with the function that
"un-displays" the inline image, in that it fails to remove the
inserted newline?  (AFAIU, the code before the above commit also had
the same issue.)  That wasn't clear from the description of the
problem, and the Subject is ambiguous wrt what newlines are deemed
"superfluous".  So please clarify what is the problem you are flagging
here.

Thanks.




Information forwarded to bug-gnu-emacs@HIDDEN:
bug#69920; Package emacs. Full text available.

Message received at 69920 <at> debbugs.gnu.org:


Received: (at 69920) by debbugs.gnu.org; 22 Mar 2024 23:40:20 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Fri Mar 22 19:40:20 2024
Received: from localhost ([127.0.0.1]:58190 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rnoUl-0005ER-8C
	for submit <at> debbugs.gnu.org; Fri, 22 Mar 2024 19:40:20 -0400
Received: from tilde.club ([142.44.150.184]:50012 ident=postfix)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <alcor@HIDDEN>) id 1rnnur-0003Gd-Mx
 for 69920 <at> debbugs.gnu.org; Fri, 22 Mar 2024 19:03:14 -0400
Received: from topazii.tilde.club (unknown
 [IPv6:2001:9e8:7cd:5200:bbe8:2b24:626f:ac56])
 by tilde.club (Postfix) with ESMTPSA id 3D07D2213F7A6
 for <69920 <at> debbugs.gnu.org>; Fri, 22 Mar 2024 22:25:35 +0000 (UTC)
DKIM-Filter: OpenDKIM Filter v2.11.0 tilde.club 3D07D2213F7A6
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tilde.club; s=mail;
 t=1711146335; bh=dkBXo7j14ZmK1gYfT+zCTMBeXK0ppGOQXejSJffctso=;
 h=From:To:Subject:In-Reply-To:References:Date:From;
 b=efcg99h4i6hD4OZx0aHy74Z9AvP5/ChvNUU7jMIcIY7V/NF5v3ZgcuwyzInDmBcEU
 uMCop1uNEYK4Tjc6AEwE+mLg4UKr3YtLjFuWe8V8gWc6h9nbeD5eOb7H0P8zoqCbHh
 uyY8fzzwLm00qY8H/JeSMbQpLEjM9g0Zn+LHVqEo=
From: Alcor <alcor@HIDDEN>
To: 69920 <at> debbugs.gnu.org
Subject: Re: Proposed fix - Toggling MIME inline attachment previews adds
 superfluous newlines
In-Reply-To: <87wmpuouay.fsf@HIDDEN> (alcor@HIDDEN's message of "Fri, 
 22 Mar 2024 22:01:09 +0100")
References: <87wmpuouay.fsf@HIDDEN>
Date: Fri, 22 Mar 2024 23:25:20 +0100
Message-ID: <87y1a952gf.fsf@HIDDEN>
User-Agent: Gnus/5.13 (Gnus v5.13)
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
X-Spam-Score: -0.0 (/)
X-Debbugs-Envelope-To: 69920
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -1.0 (-)

Tags: patch

Add patch available mark to bug (forgot to do that in the previous
message).

git blame'ing the line in question on yields a decade-old
commit. Apparently, this used to be (insert "\n\n") =E2=80=A6 (delete-regio=
n b
(+ b 2)) about ~10 years ago, and the last relevant change turned that
into (insert "\n") =E2=80=A6 (delete-region b (1+ b)), but it seems lots of
other things on that codepath changed in the meantime.

FWIW, (insert-image =E2=80=A6 "x") (insert "\n") inserts two characters,
i.e. the image itself "x" and "\n", so it should either be (insert "\n")
=E2=80=A6 (delete-region b (+ 2 b)) or, alternatively just (delete-region b=
 (1+
b)) (without any `insert' call).




Information forwarded to bug-gnu-emacs@HIDDEN:
bug#69920; Package emacs. Full text available.

Message received at 69920 <at> debbugs.gnu.org:


Received: (at 69920) by debbugs.gnu.org; 22 Mar 2024 21:25:20 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Fri Mar 22 17:25:20 2024
Received: from localhost ([127.0.0.1]:50355 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rnmO7-0001jx-7h
	for submit <at> debbugs.gnu.org; Fri, 22 Mar 2024 17:25:20 -0400
Received: from tilde.club ([142.44.150.184]:44514 ident=postfix)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <alcor@HIDDEN>) id 1rnm1g-0000hr-2b
 for 69920 <at> debbugs.gnu.org; Fri, 22 Mar 2024 17:02:08 -0400
Received: from topazii.tilde.club (unknown
 [IPv6:2001:9e8:7cd:5200:bbe8:2b24:626f:ac56])
 by tilde.club (Postfix) with ESMTPSA id DF07E2213F7A6
 for <69920 <at> debbugs.gnu.org>; Fri, 22 Mar 2024 21:01:25 +0000 (UTC)
DKIM-Filter: OpenDKIM Filter v2.11.0 tilde.club DF07E2213F7A6
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tilde.club; s=mail;
 t=1711141286; bh=jFEcWCoi+atxHRzhjmo03aHTNFbiqEXrgwjZOKkVefA=;
 h=From:To:Subject:Date:From;
 b=hlxDnygvgvvqnT/+05piLjOoP7V1BHnd5mfajvhrbwbBNxSsEu7j757dUqe0qHmI+
 scy+t+j8ICH5GrHbm6WoX0I60rGVMxizWcKI72SJmsi4sPAwPyafCY8gLa0zhBwxiJ
 VIQZnbKbXTXerv8KnQJL+1LbZfQNfVUdFunHY6/U=
From: Alcor <alcor@HIDDEN>
To: 69920 <at> debbugs.gnu.org
Subject: Proposed fix - Toggling MIME inline attachment previews adds
 superfluous newlines
Date: Fri, 22 Mar 2024 22:01:09 +0100
Message-ID: <87wmpuouay.fsf@HIDDEN>
User-Agent: Gnus/5.13 (Gnus v5.13)
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=-=-="
X-Spam-Score: -0.0 (/)
X-Debbugs-Envelope-To: 69920
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -1.0 (-)

--=-=-=
Content-Type: text/plain

Hi Emacs maintainers,

After some investigation, I noticed that `mm-inline-image' adds a
newline via (insert "\n") directly after `insert-image' for unclear
reasons.

Removing the (insert "\n") form seems to resolve the issue (along with
the other issues mentioned under "Additionally, the following can be
observed") in the initial bug report message.

A patch is included illustrating the change made, but I'm unfamiliar
with the gnus codebase. It would be great if somebody more experienced
took a look.

Cheers,
-A.


--=-=-=
Content-Type: text/x-diff
Content-Disposition: attachment;
 filename=0001-Do-not-add-a-superfluous-newline-after-insert-image.patch

From f04178b25010f1a91094bea6b4a0fcbc590b14b0 Mon Sep 17 00:00:00 2001
From: "F. Moukayed" <smfadi+emacs@HIDDEN>
Date: Fri, 22 Mar 2024 20:46:21 +0000
Subject: [PATCH] Do not add a superfluous newline after `insert-image'.

* lisp/gnus/mm-view.el (mm-inline-image): Remove unnecessary (insert "\n") to resolve issues with toggling inline attachment previews (bug#69920).
---
 lisp/gnus/mm-view.el | 1 -
 1 file changed, 1 deletion(-)

diff --git a/lisp/gnus/mm-view.el b/lisp/gnus/mm-view.el
index 3a71920..50636de 100644
--- a/lisp/gnus/mm-view.el
+++ b/lisp/gnus/mm-view.el
@@ -99,7 +99,6 @@ This is only used if `mm-inline-large-images' is set to
 				 (- (nth 3 edges) (nth 1 edges)))))))
          image))
      "x")
-    (insert "\n")
     (mm-handle-set-undisplayer
      handle
      (lambda ()
-- 
2.34.1


--=-=-=--




Information forwarded to bug-gnu-emacs@HIDDEN:
bug#69920; Package emacs. Full text available.

Message received at submit <at> debbugs.gnu.org:


Received: (at submit) by debbugs.gnu.org; 20 Mar 2024 18:10:28 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Wed Mar 20 14:10:28 2024
Received: from localhost ([127.0.0.1]:54115 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rn0OL-0005sV-FL
	for submit <at> debbugs.gnu.org; Wed, 20 Mar 2024 14:10:28 -0400
Received: from lists.gnu.org ([209.51.188.17]:38998)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <alcor@HIDDEN>) id 1rn0Mw-0005lE-SF
 for submit <at> debbugs.gnu.org; Wed, 20 Mar 2024 14:08:56 -0400
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 <alcor@HIDDEN>) id 1rn0E5-0002Yl-Gi
 for bug-gnu-emacs@HIDDEN; Wed, 20 Mar 2024 13:59:45 -0400
Received: from tilde.club ([142.44.150.184])
 by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
 (Exim 4.90_1) (envelope-from <alcor@HIDDEN>) id 1rn0E2-0006u1-W3
 for bug-gnu-emacs@HIDDEN; Wed, 20 Mar 2024 13:59:45 -0400
Received: from topazii.tilde.club (unknown
 [IPv6:2001:9e8:7fc:7100:4aaf:e9fb:50b0:611e])
 by tilde.club (Postfix) with ESMTPSA id 6E38F22111981
 for <bug-gnu-emacs@HIDDEN>; Wed, 20 Mar 2024 17:59:40 +0000 (UTC)
DKIM-Filter: OpenDKIM Filter v2.11.0 tilde.club 6E38F22111981
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tilde.club; s=mail;
 t=1710957580; bh=XAcDsiVvwr23p+AjPokzP3SUoKs3E4HtUMCoJPghCvQ=;
 h=From:To:Subject:Date:From;
 b=i49p87iJUYA2X8e6Emr+Fi5NuipDd+wXkKrYulLG1rRntKsaa05rhIyO2tfiXHF9X
 UHNZbU6M1vAL8mw9CvO5wn7CYi806/tEmjhWW1IQaRe0FJSM1K1n4SE2LvoTZdkMOs
 YYGditVzJmVZo1vm/CxhVrjDuUB14zCcJSTo29dU=
From: Alcor <alcor@HIDDEN>
To: bug-gnu-emacs@HIDDEN
Subject: 29.2; gnus: article-mode: Toggling MIME inline attachment previews
 adds superfluous newlines
Date: Wed, 20 Mar 2024 18:59:36 +0100
Message-ID: <87bk78pywn.fsf@HIDDEN>
User-Agent: Gnus/5.13 (Gnus v5.13)
MIME-Version: 1.0
Content-Type: text/plain
Received-SPF: pass client-ip=142.44.150.184; envelope-from=alcor@HIDDEN;
 helo=tilde.club
X-Spam_score_int: -20
X-Spam_score: -2.1
X-Spam_bar: --
X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1,
 DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_HELO_PASS=-0.001,
 SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no
X-Spam_action: no action
X-Spam-Score: -1.4 (-)
X-Debbugs-Envelope-To: submit
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -2.4 (--)


1. Receive an E-Mail message with at least two image attachments (I
tried with two image/png files, sent via SMTP and read via IMAP).

2. Go into the gnus summary buffer and view the received message/article.

3. In the "Attachments: ..." line, click on the attachment to toggle the
inline preview.

4. Click on the attachment button again to hide the inline preview.

5. Repeat steps 3-4 a number of times (e.g. 10-20 times).

6. Observe as newlines keep getting inserted and accumulate under the
preview.

Additionally, the following can be observed:
- Overlays are added to the attachment button, but are never cleaned up
(can be checked with `describe-char' with point on the button).

- If there is more than 1 inline-previewable attachment, clicking on the
button for attachment #1, then #2, then #1, and #2 in that order breaks
the preview state (the inline previews can no longer be hidden). This
may be unrelated, however (please advise if a separate bug should be
opened for this).


In GNU Emacs 29.2 (build 2, x86_64-pc-linux-gnu, GTK+ Version 3.24.20,
 cairo version 1.16.0) of 2024-03-13 built on lcy02-amd64-051
Repository revision: 900dc1b4bf3011e685a1ec7d7ce4dcd0262ec880
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12201001
System Description: Ubuntu 22.04.4 LTS

Configured using:
 'configure --prefix=/snap/emacs/current/usr --with-x-toolkit=gtk3
 --without-xaw3d --with-modules --with-cairo
 --with-native-compilation=aot --without-pgtk --with-xinput2
 --with-tree-sitter --with-json
 'CFLAGS=-isystem/build/emacs/parts/emacs/install/usr/include
 -isystem/build/emacs/parts/emacs/install/usr/include/x86_64-linux-gnu
 -isystem/build/emacs/stage/usr/include -O2'
 'CPPFLAGS=-isystem/build/emacs/parts/emacs/install/usr/include
 -isystem/build/emacs/parts/emacs/install/usr/include/x86_64-linux-gnu
 -isystem/build/emacs/stage/usr/include'
 'LDFLAGS=-L/build/emacs/parts/emacs/install/lib
 -L/build/emacs/parts/emacs/install/usr/lib
 -L/build/emacs/parts/emacs/install/lib/x86_64-linux-gnu
 -L/build/emacs/parts/emacs/install/usr/lib/x86_64-linux-gnu
 -L/build/emacs/stage/usr/lib''

Configured features:
ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GPM GSETTINGS HARFBUZZ JPEG
JSON LCMS2 LIBOTF LIBSELINUX LIBSYSTEMD LIBXML2 M17N_FLT MODULES
NATIVE_COMP NOTIFY INOTIFY PDUMPER PNG RSVG SECCOMP SOUND SQLITE3
THREADS TIFF TOOLKIT_SCROLL_BARS TREE_SITTER X11 XDBE XIM XINPUT2 XPM
GTK3 ZLIB

Important settings:
  value of $LANG: de_DE.UTF-8
  value of $XMODIFIERS: @im=ibus
  locale-coding-system: utf-8-unix

Major mode: Article

Minor modes in effect:
  shell-dirtrack-mode: t
  erc-ring-mode: t
  erc-notifications-mode: t
  erc-netsplit-mode: t
  erc-menu-mode: t
  erc-list-mode: t
  erc-irccontrols-mode: t
  erc-keep-place-mode: t
  erc-move-to-prompt-mode: t
  erc-readonly-mode: t
  erc-scrolltobottom-mode: t
  erc-imenu-mode: t
  erc-button-mode: t
  erc-fill-mode: t
  erc-stamp-mode: t
  erc-bufbar-mode: t
  erc-track-mode: t
  erc-match-mode: t
  erc-autojoin-mode: t
  erc-autoaway-mode: t
  recentf-mode: t
  pixel-scroll-precision-mode: t
  minibuffer-depth-indicate-mode: t
  global-whitespace-mode: t
  global-goto-address-mode: t
  goto-address-mode: t
  global-auto-revert-mode: t
  fido-vertical-mode: t
  icomplete-vertical-mode: t
  icomplete-mode: t
  fido-mode: t
  erc-networks-mode: t
  desktop-save-mode: t
  windmove-mode: t
  tooltip-mode: t
  global-eldoc-mode: t
  show-paren-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  buffer-read-only: t
  column-number-mode: t
  line-number-mode: t
  indent-tabs-mode: t
  transient-mark-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t

Features:
(shadow emacsbug misearch multi-isearch qp descr-text elec-pair
mh-buffers mh-loaddefs gnus-registry registry eieio-base rmailsum rmail
gnus-salt gnus-topic gnus-uu yenc gnus-vm shortdoc help-fns radix-tree
mailalias smtpmail smerge-mode diff diff-mode sort smiley gnus-cite
mm-archive mail-extr textsec uni-scripts idna-mapping ucs-normalize
uni-confusable textsec-check gnus-async gnus-bcklg gnus-ml nndraft nnmh
nnfolder utf-7 rfc2104 network-stream nsm gnus-agent gnus-srvr
gnus-score score-mode nnvirtual gnus-msg nntp gnus-cache
display-line-numbers org-element org-persist org-id org-refile avl-tree
generator oc-basic ol-eww eww url-queue mm-url ol-rmail ol-mhe ol-irc
ol-info ol-gnus nnselect gnus-art mm-uu mml2015 mm-view mml-smime smime
gnutls dig gnus-sum shr pixel-fill kinsoku url-file svg dom gnus-group
gnus-undo gnus-start gnus-dbus gnus-cloud nnimap nnmail mail-source utf7
nnoo gnus-spec gnus-int gnus-range message yank-media puny rfc822 mml
mml-sec epa epg rfc6068 epg-config mm-decode mm-bodies mm-encode
mail-parse rfc2231 mailabbrev gmm-utils mailheader gnus-win ol-docview
doc-view jka-compr image-mode exif dired dired-loaddefs ol-bibtex bibtex
ol-bbdb ol-w3m ol-doi org-link-doi org ob ob-tangle ob-ref ob-lob
ob-table ob-exp org-macro org-src ob-comint org-pcomplete org-list
org-footnote org-faces org-entities noutline outline ob-emacs-lisp
ob-core ob-eval org-cycle org-table ol org-fold org-fold-core org-keys
oc org-loaddefs find-func cal-menu calendar cal-loaddefs org-version
org-compat org-macs tramp-cache time-stamp tramp-sh tramp tramp-loaddefs
trampver tramp-integration files-x tramp-compat shell parse-time iso8601
disp-table erc-ring erc-desktop-notifications notifications dbus xml
erc-nicks color erc-netsplit erc-menu erc-list erc-goodies erc-imenu
imenu erc-pcomplete pcomplete comint ansi-osc ansi-color erc-button
erc-fill erc-stamp erc-status-sidebar erc-track erc-match erc-join
erc-autoaway leuven-dark-theme recentf tree-widget pixel-scroll cua-base
ring mb-depth whitespace goto-addr thingatpt autorevert filenotify
icomplete erc derived format-spec erc-backend erc-networks easy-mmode
erc-common inline erc-compat pcase compat erc-loaddefs desktop frameset
sendmail rfc2047 rfc2045 ietf-drums gnus nnheader gnus-util
text-property-search time-date mail-utils range mm-util mail-prsvr
cus-edit pp cus-load wid-edit windmove xdg site-start comp comp-cstr
warnings icons rx cl-extra help-mode erc-autoloads info compat-autoloads
markdown-mode-autoloads package browse-url url url-proxy url-privacy
url-expand url-methods url-history url-cookie generate-lisp-file
url-domsuf url-util mailcap url-handlers url-parse auth-source cl-seq
eieio eieio-core cl-macs password-cache json subr-x map byte-opt gv
bytecomp byte-compile url-vars cl-loaddefs cl-lib rmc iso-transl tooltip
cconv eldoc paren electric uniquify ediff-hook vc-hooks lisp-float-type
elisp-mode mwheel term/x-win x-win term/common-win x-dnd tool-bar dnd
fontset image regexp-opt fringe tabulated-list replace newcomment
text-mode lisp-mode prog-mode register page tab-bar menu-bar rfn-eshadow
isearch easymenu timer select scroll-bar mouse jit-lock font-lock syntax
font-core term/tty-colors frame minibuffer nadvice seq simple cl-generic
indonesian philippine cham georgian utf-8-lang misc-lang vietnamese
tibetan thai tai-viet lao korean japanese eucjp-ms cp51932 hebrew greek
romanian slovak czech european ethiopic indian cyrillic chinese
composite emoji-zwj charscript charprop case-table epa-hook
jka-cmpr-hook help abbrev obarray oclosure cl-preloaded button loaddefs
theme-loaddefs faces cus-face macroexp files window text-properties
overlay sha1 md5 base64 format env code-pages mule custom widget keymap
hashtable-print-readable backquote threads dbusbind inotify lcms2
dynamic-setting system-font-setting font-render-setting cairo
move-toolbar gtk x-toolkit xinput2 x multi-tty make-network-process
native-compile emacs)

Memory information:
((conses 16 735396 84127)
 (symbols 48 46838 24)
 (strings 32 190158 4181)
 (string-bytes 1 5259546)
 (vectors 16 82583)
 (vector-slots 8 2226749 106897)
 (floats 8 679 1292)
 (intervals 56 3126 636)
 (buffers 984 33))




Acknowledgement sent to Alcor <alcor@HIDDEN>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs@HIDDEN. Full text available.
Report forwarded to bug-gnu-emacs@HIDDEN:
bug#69920; Package emacs. Full text available.
Please note: This is a static page, with minimal formatting, updated once a day.
Click here to see this page with the latest information and nicer formatting.
Last modified: Mon, 22 Apr 2024 12:45:02 UTC

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997 nCipher Corporation Ltd, 1994-97 Ian Jackson.