Support inverted parentheses in mathtext.

anntzer · anntzer · commit 10b33c7f97f5 · 2022-06-11T15:02:26.000+02:00
TeX is perfectly happy to start with a closing parenthesis and end with
an opening one; no need to distinguish them.
diff --git a/lib/matplotlib/_mathtext.py b/lib/matplotlib/_mathtext.py
@@ -1685,13 +1685,12 @@ class _MathStyle(enum.Enum):
       liminf sin cos exp limsup sinh cosh gcd ln sup cot hom log tan
       coth inf max tanh""".split())
 
-    _ambi_delim = set(r"""
+    _ambi_delims = set(r"""
       | \| / \backslash \uparrow \downarrow \updownarrow \Uparrow
       \Downarrow \Updownarrow . \vert \Vert \\|""".split())
-
-    _left_delim = set(r"( [ \{ < \lfloor \langle \lceil".split())
-
-    _right_delim = set(r") ] \} > \rfloor \rangle \rceil".split())
+    _left_delims = set(r"( [ \{ < \lfloor \langle \lceil".split())
+    _right_delims = set(r") ] \} > \rfloor \rangle \rceil".split())
+    _delims = _left_delims | _right_delims | _ambi_delims
 
     def __init__(self):
         p = types.SimpleNamespace()
@@ -1726,9 +1725,7 @@ def set_names_and_parse_actions():
             Optional(r"\math" + oneOf(self._fontnames)("font")) + "{")
         p.end_group      = Literal("}")
 
-        p.ambi_delim     = oneOf(self._ambi_delim)
-        p.left_delim     = oneOf(self._left_delim)
-        p.right_delim    = oneOf(self._right_delim)
+        p.delim          = oneOf(self._delims)
 
         set_names_and_parse_actions()  # for root definitions.
 
@@ -1788,8 +1785,8 @@ def set_names_and_parse_actions():
             | Error(r"Expected \binom{num}{den}"))
 
         p.genfrac <<= r"\genfrac" - (
-            "{" + Optional(p.ambi_delim | p.left_delim)("ldelim") + "}"
-            + "{" + Optional(p.ambi_delim | p.right_delim)("rdelim") + "}"
+            "{" + Optional(p.delim)("ldelim") + "}"
+            + "{" + Optional(p.delim)("rdelim") + "}"
             + "{" + p.float_literal("rulesize") + "}"
             + p.simple_group("style")
             + p.required_group("num")
@@ -1854,13 +1851,9 @@ def set_names_and_parse_actions():
         )
 
         p.auto_delim    <<= (
-            r"\left"
-            - ((p.left_delim | p.ambi_delim)("left")
-               | Error("Expected a delimiter"))
+            r"\left" - (p.delim("left") | Error("Expected a delimiter"))
             + ZeroOrMore(p.simple | p.auto_delim)("mid")
-            + r"\right"
-            - ((p.right_delim | p.ambi_delim)("right")
-               | Error("Expected a delimiter"))
+            + r"\right" - (p.delim("right") | Error("Expected a delimiter"))
         )
 
         # Leaf definitions.
@@ -1982,7 +1975,7 @@ def symbol(self, s, loc, toks):
             # Binary operators at start of string should not be spaced
             if (c in self._binary_operators and
                     (len(s[:loc].split()) == 0 or prev_char == '{' or
-                     prev_char in self._left_delim)):
+                     prev_char in self._left_delims)):
                 return [char]
             else:
                 return [Hlist([self._make_space(0.2),
@@ -2087,8 +2080,7 @@ def operatorname(self, s, loc, toks):
         if isinstance(name, ParseResults):
             next_char_loc += len('operatorname{}')
         next_char = next((c for c in s[next_char_loc:] if c != ' '), '')
-        delimiters = self._left_delim | self._ambi_delim | self._right_delim
-        delimiters |= {'^', '_'}
+        delimiters = self._delims | {'^', '_'}
         if (next_char not in delimiters and
                 name not in self._overunder_functions):
             # Add thin space except when followed by parenthesis, bracket, etc.
diff --git a/lib/matplotlib/tests/test_mathtext.py b/lib/matplotlib/tests/test_mathtext.py
@@ -344,6 +344,12 @@ def test_operator_space(fig_test, fig_ref):
     fig_ref.text(0.1, 0.8, r"$\mathrm{log}_2$")
 
 
+@check_figures_equal(extensions=["png"])
+def test_inverted_delimiters(fig_test, fig_ref):
+    fig_test.text(.5, .5, r"$\left)\right($", math_fontfamily="dejavusans")
+    fig_ref.text(.5, .5, r"$)($", math_fontfamily="dejavusans")
+
+
 def test_mathtext_fallback_valid():
     for fallback in ['cm', 'stix', 'stixsans', 'None']:
         mpl.rcParams['mathtext.fallback'] = fallback
