[LyX/master] Export ' (straight) as ’ (curly) in DocBook

[Thibaut Cuvelier]

commit d4095dc0e65d292c3ae0772a429d1db662573f14
Author: Thibaut Cuvelier <tcuvelier at lyx.org>
Date:   Fri Dec 9 01:25:34 2022 +0100

    Export ' (straight) as ’ (curly) in DocBook
    
    This is similar to what LaTeX does in its output.
    
    See the (long) discussion in ticket #11244. Port of ad3e6c69b2ce for DocBook.
    
    This patch requires delaying entire strings instead of just characters, so that the DocBook code path can be as similar to the XHTML one as possible.
---
 src/Paragraph.cpp |   30 ++++++++++++++++++++----------
 1 files changed, 20 insertions(+), 10 deletions(-)

diff --git a/src/Paragraph.cpp b/src/Paragraph.cpp
index 6b156b9..3f1f814 100644
--- a/src/Paragraph.cpp
+++ b/src/Paragraph.cpp
@@ -3552,7 +3552,7 @@ std::tuple<std::vector<docstring>, std::vector<docstring>, std::vector<docstring
     // State variables for the main loop.
     auto xs = new XMLStream(os); // XMLStream has no copy constructor: to create a new object, the only solution
     // is to hold a pointer to the XMLStream (xs = XMLStream(os) is not allowed once the first object is built).
-    std::vector<char_type> delayedChars; // When a font tag ends with a space, output it after the closing font tag.
+    std::vector<docstring> delayedChars; // When a font tag ends with a space, output it after the closing font tag.
     // This requires to store delayed characters at some point.
 
     DocBookFontState fs; // Track whether we have opened font tags
@@ -3610,8 +3610,8 @@ std::tuple<std::vector<docstring>, std::vector<docstring>, std::vector<docstring
 
         // Deal with the delayed characters *after* closing font tags.
         if (!delayedChars.empty()) {
-            for (char_type c: delayedChars)
-                *xs << c;
+            for (const docstring& c: delayedChars)
+                *xs << XMLStream::ESCAPE_NONE << c;
             delayedChars.clear();
         }
 
@@ -3642,10 +3642,18 @@ std::tuple<std::vector<docstring>, std::vector<docstring>, std::vector<docstring
 			}
 		} else {
 			char_type c = getUChar(buf.masterBuffer()->params(), rp, i);
-			if (lyx::isSpace(c) && !ignore_fonts)
-				delayedChars.push_back(c);
-			else
-				*xs << c;
+			if (lyx::isSpace(c) && !ignore_fonts) { // Delay spaces *after* the font-tag closure for cleaner output.
+				if (c == ' ' && (style.free_spacing || rp.free_spacing)) {
+					delayedChars.push_back(from_ascii(" "));
+				} else {
+					delayedChars.emplace_back(1, c);
+				}
+			} else { // No need to delay the character.
+				if (c == '\'')
+					*xs << XMLStream::ESCAPE_NONE << "’";
+				else
+					*xs << c;
+			}
 		}
 		font_old = font.fontInfo();
 	}
@@ -3657,9 +3665,11 @@ std::tuple<std::vector<docstring>, std::vector<docstring>, std::vector<docstring
 		xs->closeFontTags();
 
 	// Deal with the delayed characters *after* closing font tags.
-	if (!delayedChars.empty())
-		for (char_type c: delayedChars)
-			*xs << c;
+	if (!delayedChars.empty()) {
+		for (const docstring &c: delayedChars)
+			*xs << XMLStream::ESCAPE_NONE << c;
+		delayedChars.clear();
+	}
 
 	// In listings, new lines (i.e. \n characters in the output) are very important. Avoid generating one for the
 	// last line to get a clean output.