Subject: Re: bug#52302: 28.0.50; [PATCH] Overlay strings should not increment vpos

From: dick <[email-masked]>
Date: Mon, 06 Dec 2021 14:47:45 -0500

A more dangerous change like the below would break the cycle of fixes
that minimize impact while maximizing obfuscation.

>From 5f734c1d50953836dda444b8371642c6f20d4065 Mon Sep 17 00:00:00 2001
From: dickmao <[email-masked]>
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

I'll explore this more deeply in my longlines rewrite.  In the meantime, closing.