cleaning

b3b00 · Oct 11, 2024 · ec0a5a9 · ec0a5a9
1 parent f2591eb
commit ec0a5a9
Show file tree

Hide file tree

Showing 7 changed files with 11 additions and 72 deletions.
diff --git a/src/sly/lexer/attributes/ModesAttribute.cs b/src/sly/lexer/attributes/ModesAttribute.cs
diff --git a/src/sly/parser/generator/EBNFParserBuilder.cs b/src/sly/parser/generator/EBNFParserBuilder.cs
@@ -111,25 +111,10 @@ protected override ISyntaxParser<IN, OUT> BuildSyntaxParser(ParserConfiguration<
             string rootRule)
         {
             ISyntaxParser<IN, OUT> parser = null;
-            switch (parserType)
+            if (parserType == ParserType.EBNF_LL_RECURSIVE_DESCENT)
             {
-                case ParserType.LL_RECURSIVE_DESCENT:
-                {
-                    parser = new RecursiveDescentSyntaxParser<IN, OUT>(conf, rootRule, I18N);
-                    break;
-                }
-                case ParserType.EBNF_LL_RECURSIVE_DESCENT:
-                {
-                    parser = new EBNFRecursiveDescentSyntaxParser<IN, OUT>(conf, rootRule, I18N);
-                    break;
-                }
-                default:
-                {
-                    parser = null;
-                    break;
-                }
+                parser = new EBNFRecursiveDescentSyntaxParser<IN, OUT>(conf, rootRule, I18N);
             }
-
             return parser;
         }
 

diff --git a/src/sly/parser/generator/ParserBuilder.cs b/src/sly/parser/generator/ParserBuilder.cs
@@ -14,9 +14,6 @@
 
 namespace sly.parser.generator
 {
-    public delegate BuildResult<Parser<IN, OUT>> ParserChecker<IN, OUT>(BuildResult<Parser<IN, OUT>> result,
-        NonTerminal<IN> nonterminal) where IN : struct;
-
     /// <summary>
     ///     this class provides API to build parser
     /// </summary>
@@ -280,7 +277,7 @@ private Rule<IN> BuildNonTerminal(Tuple<string, string> ntAndRule)
 
         private BuildResult<Parser<IN, OUT>> CheckParser(BuildResult<Parser<IN, OUT>> result)
         {
-            var checkers = new List<ParserChecker<IN, OUT>>
+            var checkers = new List<Func<BuildResult<Parser<IN, OUT>>,NonTerminal<IN>,BuildResult<Parser<IN, OUT>>>>
             {
                 CheckUnreachable,
                 CheckNotFound,

diff --git a/src/sly/parser/generator/visitor/mermaid/IMermaid.cs b/src/sly/parser/generator/visitor/mermaid/IMermaid.cs
@@ -1,4 +1,4 @@
-namespace sly.parser.generator.visitor.dotgraph
+namespace sly.parser.generator.visitor.mermaid
 {
     public interface IMermaid
     {

diff --git a/src/sly/parser/generator/visitor/mermaid/MermaidNode.cs b/src/sly/parser/generator/visitor/mermaid/MermaidNode.cs
@@ -1,6 +1,7 @@
 using System.Text;
+using sly.parser.generator.visitor.dotgraph;
 
-namespace sly.parser.generator.visitor.dotgraph {
+namespace sly.parser.generator.visitor.mermaid {
 
     public enum MermaidNodeShape
     {

diff --git a/src/sly/parser/syntax/grammar/TerminalClause.cs b/src/sly/parser/syntax/grammar/TerminalClause.cs
@@ -73,6 +73,7 @@ public string Dump()
         }
 
 
+        [ExcludeFromCodeCoverage]
         public bool Equals(IClause<T> clause)
         {
             if (clause is TerminalClause<T> other)
@@ -82,6 +83,8 @@ public bool Equals(IClause<T> clause)
 
             return false;
         }
+
+        [ExcludeFromCodeCoverage]
         protected bool Equals(TerminalClause<T> other)
         {
             if (IsExplicitToken)
@@ -110,11 +113,8 @@ public IndentTerminalClause(IndentationType expectedIndentation, bool discard) :
             ExpectedIndentation = expectedIndentation;
             Discarded = discard;
         }
-
-        public override  bool MayBeEmpty()
-        {
-            return false;
-        }
+
+        public override bool MayBeEmpty() => false;
 
         public override bool Check(Token<T> nextToken)
         {

diff --git a/src/sly/parser/syntax/tree/EmptyNode.cs b/src/sly/parser/syntax/tree/EmptyNode.cs