Removed unused/redundant PRCExtension methods

Author: BZngr

Rubberduck.Parsing/ParserRuleContextExtensions.cs

@@ -164,109 +164,13 @@ public static IEnumerable<TContext> GetDescendents<TContext>(this ParserRuleCont
             return listener.Matches;
         }
 
-        //public static IEnumerable<T> GetChildren<T>(this RuleContext context)
-        //{
-        //    if (context == null)
-        //    {
-        //        yield break;
-        //    }
-
-        //    for (var index = 0; index < context.ChildCount; index++)
-        //    {
-        //        var child = context.GetChild(index);
-        //        if (child is T)
-        //        {
-        //            yield return (T)child;
-        //        }
-        //    }
-        //}
-
-        //public static bool HasParent(this RuleContext context, RuleContext parent)
-        //{
-        //    if (context == null)
-        //    {
-        //        return false;
-        //    }
-        //    if (context == parent)
-        //    {
-        //        return true;
-        //    }
-        //    return HasParent(context.Parent, parent);
-        //}
-
-        //public static TContext FindChild<TContext>(this ParserRuleContext context) where TContext : ParserRuleContext
-        //{
-        //    if (context == null)
-        //    {
-        //        return default;
-        //    }
-
-        //    for (var index = 0; index < context.ChildCount; index++)
-        //    {
-        //        var child = context.GetChild(index);
-        //        if (context.GetChild(index) is TContext)
-        //        {
-        //            return (TContext)child;
-        //        }
-        //    }
-        //    return default;
-        //}
-
-        //public static bool HasChildToken(this IParseTree context, string token)
-        //{
-        //    for (var index = 0; index < context.ChildCount; index++)
-        //    {
-        //        var child = context.GetChild(index);
-        //        if (context.GetChild(index).GetText().Equals(token))
-        //        {
-        //            return true;
-        //        }
-        //    }
-        //    return false;
-        //}
-
-        public static T GetDescendent<T>(this IParseTree context)
-        {
-            if (context == null)
-            {
-                return default;
-            }
-
-            for (var index = 0; index < context.ChildCount; index++)
-            {
-                var child = context.GetChild(index);
-                if (context.GetChild(index) is T)
-                {
-                    return (T)child;
-                }
-
-                var descendent = child.GetDescendent<T>();
-                if (descendent != null)
-                {
-                    return descendent;
-                }
-            }
-
-            return default;
-        }
-
-        public static IEnumerable<IParseTree> GetDescendents(this IParseTree context)
+        /// <summary>
+        /// Try to get the first child of the generic context type.
+        /// </summary>
+        public static bool TryGetChildContext<TContext>(this ParserRuleContext ctxt, out TContext opCtxt) where TContext : ParserRuleContext
         {
-            if (context == null)
-            {
-                yield break;
-            }
-
-            for (var index = 0; index < context.ChildCount; index++)
-            {
-                var child = context.GetChild(index);
-                yield return child;
-
-                foreach (var node in child.GetDescendents())
-                {
-                    yield return node;
-                }
-            }
+            opCtxt = ctxt.GetChild<TContext>();
+            return opCtxt != null;
         }
 
         private class ChildNodeListener<TContext> : VBAParserBaseListener where TContext : ParserRuleContext
@@ -283,11 +187,5 @@ public override void EnterEveryRule(ParserRuleContext context)
                 }
             }
         }
-
-        public static bool TryGetChildContext<TContext>(this ParserRuleContext ctxt, out TContext opCtxt) where TContext : ParserRuleContext
-        {
-            opCtxt = ctxt.GetChild<TContext>();
-            return opCtxt != null;
-        }
     }
 }

RubberduckTests/Inspections/UnreachableCaseInspectionTests.cs

@@ -1,4 +1,5 @@
-using NUnit.Framework;
+using Antlr4.Runtime;
+using NUnit.Framework;
 using Rubberduck.Inspections.Concrete.UnreachableCaseInspection;
 using Rubberduck.Parsing;
 using Rubberduck.Parsing.Grammar;
@@ -2187,7 +2188,8 @@ private IUCIValueResults GetParseTreeValueResults(string inputCode, out VBAParse
             var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out var _);
             using (var state = MockParser.CreateAndParse(vbe.Object))
             {
-                selectStmt = state.ParseTrees.First().Value.GetDescendent<VBAParser.SelectCaseStmtContext>();
+                var firstParserRuleContext = (ParserRuleContext)state.ParseTrees.Where(pt => pt.Value is ParserRuleContext).First().Value;
+                selectStmt = firstParserRuleContext.GetDescendent<VBAParser.SelectCaseStmtContext>();
                 var visitor = ValueVisitorFactory.Create(state, ValueFactory);
                 valueResults = selectStmt.Accept(visitor);
             }