GNU bug report logs - #52302
28.0.50; [PATCH] Overlay strings should not increment vpos

Previous Next

Package: emacs;

Reported by: dick.r.chiang <at> gmail.com

Date: Sun, 5 Dec 2021 18:09:02 UTC

Severity: normal

Tags: patch

Found in version 28.0.50

Fixed in version 28.1

Done: dick <dick.r.chiang <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: dick <dick.r.chiang <at> gmail.com>
To: 52302 <at> debbugs.gnu.org
Subject: bug#52302: 28.0.50; [PATCH] Overlay strings should not increment vpos
Date: Mon, 06 Dec 2021 14:47:45 -0500
[Message part 1 (text/plain, inline)]
A more dangerous change like the below would break the cycle of fixes
that minimize impact while maximizing obfuscation.

[0001-A-small-change-with-disastrous-potential.patch (text/x-diff, inline)]
From 5f734c1d50953836dda444b8371642c6f20d4065 Mon Sep 17 00:00:00 2001
From: dickmao <dick.r.chiang <at> gmail.com>
Date: Mon, 6 Dec 2021 14:30:17 -0500
Subject: [PATCH] A small change with disastrous potential

* src/xdisp.c (move_it_in_display_line_to): How this function has
managed to get by without a notion of visibility is a real mystery.
---
 src/xdisp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/xdisp.c b/src/xdisp.c
index 0ff6286af74..1522c6b3193 100644
--- a/src/xdisp.c
+++ b/src/xdisp.c
@@ -9561,7 +9561,7 @@ #define IT_RESET_X_ASCENT_DESCENT(IT)			\
 
       PRODUCE_GLYPHS (it);
 
-      if (it->area != TEXT_AREA)
+      if (it->area != TEXT_AREA || it->method == GET_FROM_STRING)
 	{
 	  prev_method = it->method;
 	  if (it->method == GET_FROM_BUFFER)
-- 
2.26.2

[Message part 3 (text/plain, inline)]
I'll explore this more deeply in my longlines rewrite.  In the meantime, closing.

This bug report was last modified 3 years and 172 days ago.

Previous Next


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