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

[dick <dick.r.chiang@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 122 bytes --]

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


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-A-small-change-with-disastrous-potential.patch --]
[-- Type: text/x-diff, Size: 808 bytes --]

From 5f734c1d50953836dda444b8371642c6f20d4065 Mon Sep 17 00:00:00 2001
From: dickmao <dick.r.chiang@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


[-- Attachment #3: Type: text/plain, Size: 83 bytes --]


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