Fix punctuation (#128)

* Fix punctuation

* Wait longer since we want to ensure a caret toggle

* Try again

Author: Happypig375

CSharpMath.CoreTests/LaTeXParserTest.cs

@@ -25,7 +25,7 @@ public static MathList ParseLaTeX(string latex) {
     [InlineData("(", new[] { typeof(Open) }, "(")]
     [InlineData(")", new[] { typeof(Close) }, ")")]
     [InlineData(",", new[] { typeof(Punctuation) }, ",")]
-    [InlineData("?!", new[] { typeof(Close), typeof(Close) }, "?!")]
+    [InlineData("?!", new[] { typeof(Punctuation), typeof(Punctuation) }, "?!")]
     [InlineData("=", new[] { typeof(Relation) }, "=")]
     [InlineData("x+2", new[] { typeof(Variable), typeof(BinaryOperator), typeof(Number) }, "x+2")]
     [InlineData("(2.3 * 8)", new[] { typeof(Open), typeof(Number), typeof(Number), typeof(Number), typeof(BinaryOperator), typeof(Number), typeof(Close) }, "(2.3*8)")]

CSharpMath.CoreTests/MathListTest.cs

@@ -5,7 +5,7 @@
 using Xunit;
 
 namespace CSharpMath.CoreTests {
-  using Range = CSharpMath.Atom.Range;
+  using Range = Atom.Range;
   public class MathListTest {
     internal static void CheckClone(MathAtom? original, MathAtom? clone) {
       Assert.Equal(original, clone);
@@ -197,7 +197,7 @@ static void CheckListContents(MathList? list) {
         );
         var radical = Assert.IsType<Radical>(Assert.Single(integral.Superscript));
         Assert.Collection(radical.Degree,
-          CheckAtomNucleusAndRange<Close>("!", 0, 1),
+          CheckAtomNucleusAndRange<Punctuation>("!", 0, 1),
           CheckAtomNucleusAndRange<Ordinary>(" ", 1, 1)
         );
         Assert.Collection(radical.Radicand,

CSharpMath.Editor.Tests/CaretTests.cs

@@ -146,7 +146,7 @@ public async Task Test() {
       await Task.Delay((int)MathKeyboard<TestFont, char>.DefaultBlinkMilliseconds - CaretBlinks.MillisecondBuffer);
 
       Assert.Equal(MathKeyboardCaretState.Shown, keyboard.CaretState);
-      await Task.Delay((int)MathKeyboard<TestFont, char>.DefaultBlinkMilliseconds - CaretBlinks.MillisecondBuffer);
+      await Task.Delay(2 * CaretBlinks.MillisecondBuffer);
       Assert.Equal(MathKeyboardCaretState.TemporarilyHidden, keyboard.CaretState);
     }
   }

CSharpMath.Editor/MathKeyboard.cs

@@ -165,7 +165,7 @@ void HandleSlashButton() {
             case (null, _): throw new InvalidCodePathException("Invalid _insertionIndex");
             // Stop looking behind upon encountering these atoms unparenthesized
             case (Atoms.Open _, _) when --parenDepth < 0: goto stop;
-            case (Atoms.Close { HasCorrespondingOpen: true } a, _): parenDepth++; numerator.Push(a); break;
+            case (Atoms.Close a, _): parenDepth++; numerator.Push(a); break;
             case (Atoms.UnaryOperator _, 0): goto stop;
             case (Atoms.BinaryOperator _, 0): goto stop;
             case (Atoms.Relation _, 0): goto stop;

CSharpMath.Evaluation/Evaluation.cs

@@ -501,7 +501,7 @@ _ when LaTeXSettings.CommandForAtom(atom) is string s => MathS.Var(s + subscript
                   if (levelsDeep == 0) open = i;
                   levelsDeep++;
                   break;
-                case Atoms.Close { HasCorrespondingOpen: true } close:
+                case Atoms.Close close:
                   levelsDeep--;
                   if (levelsDeep == 0) {
                     if (open == -1) return "Missing argument for " + atom.Nucleus;

CSharpMath/Atom/Atoms/BinaryOperator.cs

@@ -1,5 +1,5 @@
 namespace CSharpMath.Atom.Atoms {
-  /// <summary>A binary operator</summary>
+  /// <summary>AMSMath class 2: Binary Operator (conjunction), e.g. +, \cup, \wedge</summary>
   public sealed class BinaryOperator : MathAtom {
     public BinaryOperator(string nucleus) : base(nucleus) { }
     public override bool ScriptsAllowed => true;

CSharpMath/Atom/Atoms/Close.cs

@@ -1,9 +1,7 @@
 namespace CSharpMath.Atom.Atoms {
-  /// <summary>Close brackets</summary>
+  /// <summary>AMSMath class 5: Right/closing delimiter, e.g. ), ], }, \rangle</summary>
   public sealed class Close : MathAtom {
-    public Close(string nucleus, bool hasCorrespondingOpen = true) : base(nucleus) =>
-      HasCorrespondingOpen = hasCorrespondingOpen;
-    public bool HasCorrespondingOpen { get; }
+    public Close(string nucleus) : base(nucleus) { }
     public override bool ScriptsAllowed => true;
     public new Close Clone(bool finalize) => (Close)base.Clone(finalize);
     protected override MathAtom CloneInside(bool finalize) => new Close(Nucleus);

CSharpMath/Atom/Atoms/LargeOperator.cs

@@ -1,5 +1,5 @@
-namespace CSharpMath.Atom.Atoms {
-  /// <summary>A large operator such as sin/cos, integral, etc.</summary>
+namespace CSharpMath.Atom.Atoms {
+  /// <summary>AMSMath class 1: Prefix operator, e.g. \sin, \cos, \sum, \prod, \int</summary>
   public sealed class LargeOperator : MathAtom {
     bool? _limits;
     /// <summary>

CSharpMath/Atom/Atoms/Number.cs

@@ -1,4 +1,5 @@
 namespace CSharpMath.Atom.Atoms {
+  /// <summary>Class 0: ordinary for Arabic numerals 0–9 and the decimal dot .</summary>
   public sealed class Number : MathAtom {
     public Number(string number) : base(number) { }
     public override bool ScriptsAllowed => true;

CSharpMath/Atom/Atoms/Open.cs

@@ -1,5 +1,5 @@
 namespace CSharpMath.Atom.Atoms {
-  /// <summary>Open brackets</summary>
+  /// <summary>AMSMath class 4: Left/Opening delimiter, e.g. (, [, {, \langle</summary>
   public sealed class Open : MathAtom {
     public Open(string nucleus) : base(nucleus) { }
     public override bool ScriptsAllowed => true;