code coverage and dead code cleaning

b3b00 · May 2, 2024 · 911c802 · 911c802
1 parent fe66990
commit 911c802
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 13 deletions.
diff --git a/src/sly/lexer/LexerResult.cs b/src/sly/lexer/LexerResult.cs
@@ -1,4 +1,6 @@
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
+
 namespace sly.lexer
 {
     public class LexerResult<IN> where IN : struct
@@ -29,6 +31,7 @@ public LexerResult(LexicalError error, List<Token<IN>> tokens)
             SetTokens(tokens);
         }
 
+        [ExcludeFromCodeCoverage]
         public override string ToString()
         {
             if (IsOk)

diff --git a/src/sly/lexer/TokenChannels.cs b/src/sly/lexer/TokenChannels.cs
@@ -1,5 +1,6 @@
 using System.Collections;
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.Linq;
 
 namespace sly.lexer
@@ -116,6 +117,7 @@ IEnumerator IEnumerable.GetEnumerator()
             return Tokens.GetEnumerator();
         }
 
+        [ExcludeFromCodeCoverage]
         public override string ToString()
         {
             return string.Join("\n", _tokenChannels.Values.Select(x => x.ToString()).ToArray());

diff --git a/src/sly/parser/syntax/grammar/Rule.cs b/src/sly/parser/syntax/grammar/Rule.cs
@@ -1,4 +1,5 @@
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.Linq;
 using System.Reflection;
 using System.Text;
@@ -33,19 +34,6 @@ public Rule()
 
         public string RuleString { get; set;  }
 
-        public string Key
-        {
-            get
-            {
-                var key = string.Join("_", Clauses.Select(c => c.ToString()));
-
-                if (Clauses.Count == 1) 
-                    key += "_";
-
-                return IsExpressionRule ? key.Replace(" | ", "_") : key;
-            }
-        }
-
         public List<IClause<IN>> Clauses { get; set; }
         public List<LeadingToken<IN>> PossibleLeadingTokens { get; set; }
 
@@ -171,6 +159,7 @@ public bool Match<OUT>(IList<Token<IN>> tokens, int position, ParserConfiguratio
             return PossibleLeadingTokens.Exists(x => x.Match(tokens[position])) || MayBeEmpty;
         }
 
+        [ExcludeFromCodeCoverage]
         public string Dump()
         {
             StringBuilder builder = new StringBuilder();