Make all concrete inspection listeners private

MDoerner · MDoerner · commit af2b026bd175 · 2020-03-07T00:46:17.000+01:00
diff --git a/Rubberduck.CodeAnalysis/Inspections/Abstract/DeclarationInspectionBaseBase.cs b/Rubberduck.CodeAnalysis/Inspections/Abstract/DeclarationInspectionBaseBase.cs
@@ -8,6 +8,9 @@
 
 namespace Rubberduck.Inspections.Abstract
 {
+    /// <summary>
+    /// This is a base class for the other declaration inspection base classes. It should not be implemented directly by concrete inspections.
+    /// </summary>
     public abstract class DeclarationInspectionBaseBase : InspectionBase
     {
         private readonly DeclarationType[] _relevantDeclarationTypes;
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/BooleanAssignedInIfElseInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/BooleanAssignedInIfElseInspection.cs
@@ -59,7 +59,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.IfStmtCon
                 literalText);
         }
 
-        public class BooleanAssignedInIfElseListener : InspectionListenerBase<VBAParser.IfStmtContext>
+        private class BooleanAssignedInIfElseListener : InspectionListenerBase<VBAParser.IfStmtContext>
         {
             public override void ExitIfStmt(VBAParser.IfStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/DefTypeStatementInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/DefTypeStatementInspection.cs
@@ -68,7 +68,7 @@ private string GetTypeOfDefType(string defType)
             { "DefVar", "Variant" }
         };
 
-        public class DefTypeStatementInspectionListener : InspectionListenerBase<VBAParser.DefTypeContext>
+        private class DefTypeStatementInspectionListener : InspectionListenerBase<VBAParser.DefTypeContext>
         {
             public override void ExitDefType([NotNull] VBAParser.DefTypeContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyBlockInspectionListenerBase.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyBlockInspectionListenerBase.cs
@@ -7,7 +7,7 @@
 
 namespace Rubberduck.Inspections.Concrete
 {
-    public class EmptyBlockInspectionListenerBase<TContext> : InspectionListenerBase<TContext>
+    internal class EmptyBlockInspectionListenerBase<TContext> : InspectionListenerBase<TContext>
         where TContext : ParserRuleContext
     {
         public void InspectBlockForExecutableStatements<T>(VBAParser.BlockContext block, T context) where T : TContext
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyCaseBlockInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyCaseBlockInspection.cs
@@ -55,7 +55,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.CaseClaus
             return InspectionResults.EmptyCaseBlockInspection;
         }
 
-        public class EmptyCaseBlockListener : EmptyBlockInspectionListenerBase<VBAParser.CaseClauseContext>
+        private class EmptyCaseBlockListener : EmptyBlockInspectionListenerBase<VBAParser.CaseClauseContext>
         {
             public override void EnterCaseClause([NotNull] VBAParser.CaseClauseContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyDoWhileBlockInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyDoWhileBlockInspection.cs
@@ -50,7 +50,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.DoLoopStm
             return InspectionResults.EmptyDoWhileBlockInspection;
         }
 
-        public class EmptyDoWhileBlockListener : EmptyBlockInspectionListenerBase<VBAParser.DoLoopStmtContext>
+        private class EmptyDoWhileBlockListener : EmptyBlockInspectionListenerBase<VBAParser.DoLoopStmtContext>
         {
             public override void EnterDoLoopStmt([NotNull] VBAParser.DoLoopStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyElseBlockInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyElseBlockInspection.cs
@@ -52,7 +52,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.ElseBlock
             return InspectionResults.EmptyElseBlockInspection;
         }
 
-        public class EmptyElseBlockListener : EmptyBlockInspectionListenerBase<VBAParser.ElseBlockContext>
+        private class EmptyElseBlockListener : EmptyBlockInspectionListenerBase<VBAParser.ElseBlockContext>
         {
             public override void EnterElseBlock([NotNull] VBAParser.ElseBlockContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyForEachBlockInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyForEachBlockInspection.cs
@@ -52,7 +52,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.ForEachSt
             return InspectionResults.EmptyForEachBlockInspection;
         }
 
-        public class EmptyForEachBlockListener : EmptyBlockInspectionListenerBase<VBAParser.ForEachStmtContext>
+        private class EmptyForEachBlockListener : EmptyBlockInspectionListenerBase<VBAParser.ForEachStmtContext>
         {
             public override void EnterForEachStmt([NotNull] VBAParser.ForEachStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyForLoopBlockInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyForLoopBlockInspection.cs
@@ -52,7 +52,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.ForNextSt
             return InspectionResults.EmptyForLoopBlockInspection;
         }
 
-        public class EmptyForLoopBlockListener : EmptyBlockInspectionListenerBase<VBAParser.ForNextStmtContext>
+        private class EmptyForLoopBlockListener : EmptyBlockInspectionListenerBase<VBAParser.ForNextStmtContext>
         {
             public override void EnterForNextStmt([NotNull] VBAParser.ForNextStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyIfBlockInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyIfBlockInspection.cs
@@ -49,7 +49,7 @@ protected override string ResultDescription(QualifiedContext<ParserRuleContext>
 
         protected override IInspectionListener<ParserRuleContext> ContextListener { get; }
 
-        public class EmptyIfBlockListener : EmptyBlockInspectionListenerBase<ParserRuleContext>
+        private class EmptyIfBlockListener : EmptyBlockInspectionListenerBase<ParserRuleContext>
         {
             public override void EnterIfStmt([NotNull] VBAParser.IfStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyStringLiteralInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyStringLiteralInspection.cs
@@ -48,7 +48,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.LiteralEx
             return InspectionResults.EmptyStringLiteralInspection;
         }
 
-        public class EmptyStringLiteralListener : InspectionListenerBase<VBAParser.LiteralExpressionContext>
+        private class EmptyStringLiteralListener : InspectionListenerBase<VBAParser.LiteralExpressionContext>
         {
             public override void ExitLiteralExpression(VBAParser.LiteralExpressionContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyWhileWendBlockInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/EmptyWhileWendBlockInspection.cs
@@ -50,7 +50,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.WhileWend
             return InspectionResults.EmptyWhileWendBlockInspection;
         }
 
-        public class EmptyWhileWendBlockListener : EmptyBlockInspectionListenerBase<VBAParser.WhileWendStmtContext>
+        private class EmptyWhileWendBlockListener : EmptyBlockInspectionListenerBase<VBAParser.WhileWendStmtContext>
         {
             public override void EnterWhileWendStmt([NotNull] VBAParser.WhileWendStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ModuleScopeDimKeywordInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ModuleScopeDimKeywordInspection.cs
@@ -47,7 +47,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.VariableS
                 identifierName);
         }
 
-        public class ModuleScopedDimListener : InspectionListenerBase<VBAParser.VariableSubStmtContext>
+        private class ModuleScopedDimListener : InspectionListenerBase<VBAParser.VariableSubStmtContext>
         {
             public override void ExitVariableStmt([NotNull] VBAParser.VariableStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/MultilineParameterInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/MultilineParameterInspection.cs
@@ -50,7 +50,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.ArgContex
                 parameterText);
         }
 
-        public class ParameterListener : InspectionListenerBase<VBAParser.ArgContext>
+        private class ParameterListener : InspectionListenerBase<VBAParser.ArgContext>
         {
             public override void ExitArg([NotNull] VBAParser.ArgContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/MultipleDeclarationsInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/MultipleDeclarationsInspection.cs
@@ -41,7 +41,7 @@ protected override string ResultDescription(QualifiedContext<ParserRuleContext>
             return InspectionResults.MultipleDeclarationsInspection;
         }
 
-        public class ParameterListListener : InspectionListenerBase<ParserRuleContext>
+        private class ParameterListListener : InspectionListenerBase<ParserRuleContext>
         {
             public override void ExitVariableListStmt([NotNull] VBAParser.VariableListStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteCallStatementInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteCallStatementInspection.cs
@@ -68,7 +68,7 @@ protected override bool IsResultContext(QualifiedContext<VBAParser.CallStmtConte
             return false;
         }
 
-        public class ObsoleteCallStatementListener : InspectionListenerBase<VBAParser.CallStmtContext>
+        private class ObsoleteCallStatementListener : InspectionListenerBase<VBAParser.CallStmtContext>
         {
             public override void ExitCallStmt(VBAParser.CallStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteCallingConventionInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteCallingConventionInspection.cs
@@ -48,7 +48,7 @@ protected override bool IsResultContext(QualifiedContext<VBAParser.DeclareStmtCo
             return ((VBAParser.DeclareStmtContext)context.Context).CDECL() != null;
         }
 
-        public class ObsoleteCallingConventionListener : InspectionListenerBase<VBAParser.DeclareStmtContext>
+        private class ObsoleteCallingConventionListener : InspectionListenerBase<VBAParser.DeclareStmtContext>
         {
             public override void ExitDeclareStmt(VBAParser.DeclareStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteCommentSyntaxInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteCommentSyntaxInspection.cs
@@ -42,7 +42,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.RemCommen
             return InspectionResults.ObsoleteCommentSyntaxInspection;
         }
 
-        public class ObsoleteCommentSyntaxListener : InspectionListenerBase<VBAParser.RemCommentContext>
+        private class ObsoleteCommentSyntaxListener : InspectionListenerBase<VBAParser.RemCommentContext>
         {
             public override void ExitRemComment(VBAParser.RemCommentContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteErrorSyntaxInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteErrorSyntaxInspection.cs
@@ -42,7 +42,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.ErrorStmt
             return InspectionResults.ObsoleteErrorSyntaxInspection;
         }
 
-        public class ObsoleteErrorSyntaxListener : InspectionListenerBase<VBAParser.ErrorStmtContext>
+        private class ObsoleteErrorSyntaxListener : InspectionListenerBase<VBAParser.ErrorStmtContext>
         {
             public override void ExitErrorStmt(VBAParser.ErrorStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteLetStatementInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteLetStatementInspection.cs
@@ -44,7 +44,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.LetStmtCo
             return InspectionResults.ObsoleteLetStatementInspection;
         }
 
-        public class ObsoleteLetStatementListener : InspectionListenerBase<VBAParser.LetStmtContext>
+        private class ObsoleteLetStatementListener : InspectionListenerBase<VBAParser.LetStmtContext>
         {
             public override void ExitLetStmt(VBAParser.LetStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteWhileWendStatementInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteWhileWendStatementInspection.cs
@@ -47,7 +47,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.WhileWend
             return InspectionResults.ObsoleteWhileWendStatementInspection;
         }
 
-        public class ObsoleteWhileWendStatementListener : InspectionListenerBase<VBAParser.WhileWendStmtContext>
+        private class ObsoleteWhileWendStatementListener : InspectionListenerBase<VBAParser.WhileWendStmtContext>
         {
             public override void ExitWhileWendStmt(VBAParser.WhileWendStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/OnLocalErrorInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/OnLocalErrorInspection.cs
@@ -51,7 +51,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.OnErrorSt
             return InspectionResults.OnLocalErrorInspection;
         }
 
-        public class OnLocalErrorListener : InspectionListenerBase<VBAParser.OnErrorStmtContext>
+        private class OnLocalErrorListener : InspectionListenerBase<VBAParser.OnErrorStmtContext>
         {
             public override void ExitOnErrorStmt([NotNull] VBAParser.OnErrorStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/OptionBaseInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/OptionBaseInspection.cs
@@ -54,7 +54,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.OptionBas
                 moduleName);
         }
 
-        public class OptionBaseStatementListener : InspectionListenerBase<VBAParser.OptionBaseStmtContext>
+        private class OptionBaseStatementListener : InspectionListenerBase<VBAParser.OptionBaseStmtContext>
         {
             public override void ExitOptionBaseStmt(VBAParser.OptionBaseStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/OptionExplicitInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/OptionExplicitInspection.cs
@@ -60,7 +60,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.ModuleCon
                 moduleName);
         }
 
-        public class MissingOptionExplicitListener : InspectionListenerBase<VBAParser.ModuleContext>
+        private class MissingOptionExplicitListener : InspectionListenerBase<VBAParser.ModuleContext>
         {
             private readonly IDictionary<QualifiedModuleName, bool> _hasOptionExplicit = new Dictionary<QualifiedModuleName, bool>();
 
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/RedundantOptionInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/RedundantOptionInspection.cs
@@ -50,7 +50,7 @@ protected override string ResultDescription(QualifiedContext<ParserRuleContext>
                 context.Context.GetText());
         }
 
-        public class RedundantModuleOptionListener : InspectionListenerBase<ParserRuleContext>
+        private class RedundantModuleOptionListener : InspectionListenerBase<ParserRuleContext>
         {
             public override void ExitOptionBaseStmt(VBAParser.OptionBaseStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/StepIsNotSpecifiedInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/StepIsNotSpecifiedInspection.cs
@@ -43,23 +43,23 @@ public StepIsNotSpecifiedInspection(IDeclarationFinderProvider declarationFinder
             ContextListener = new StepIsNotSpecifiedListener();
         }
 
+        protected override IInspectionListener<VBAParser.ForNextStmtContext> ContextListener { get; }
+
         protected override string ResultDescription(QualifiedContext<VBAParser.ForNextStmtContext> context)
         {
             return InspectionResults.StepIsNotSpecifiedInspection;
         }
 
-        protected override IInspectionListener<VBAParser.ForNextStmtContext> ContextListener { get; } 
-    }
-
-    public class StepIsNotSpecifiedListener : InspectionListenerBase<VBAParser.ForNextStmtContext>
-    {
-        public override void EnterForNextStmt([NotNull] VBAParser.ForNextStmtContext context)
+        private class StepIsNotSpecifiedListener : InspectionListenerBase<VBAParser.ForNextStmtContext>
         {
-            var stepStatement = context.stepStmt();
-
-            if (stepStatement == null)
+            public override void EnterForNextStmt([NotNull] VBAParser.ForNextStmtContext context)
             {
-                SaveContext(context);
+                var stepStatement = context.stepStmt();
+
+                if (stepStatement == null)
+                {
+                    SaveContext(context);
+                }
             }
         }
     }
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/StepOneIsRedundantInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/StepOneIsRedundantInspection.cs
@@ -49,24 +49,24 @@ protected override string ResultDescription(QualifiedContext<VBAParser.StepStmtC
         {
             return InspectionResults.StepOneIsRedundantInspection;
         }
-    }
 
-    public class StepOneIsRedundantListener : InspectionListenerBase<VBAParser.StepStmtContext>
-    {
-        public override void EnterForNextStmt([NotNull] VBAParser.ForNextStmtContext context)
+        private class StepOneIsRedundantListener : InspectionListenerBase<VBAParser.StepStmtContext>
         {
-            var stepStatement = context.stepStmt();
-
-            if (stepStatement == null)
+            public override void EnterForNextStmt([NotNull] VBAParser.ForNextStmtContext context)
             {
-                return;
-            }
+                var stepStatement = context.stepStmt();
 
-            var stepText = stepStatement.expression().GetText();
+                if (stepStatement == null)
+                {
+                    return;
+                }
 
-            if(stepText == "1")
-            {
-                SaveContext(stepStatement);
+                var stepText = stepStatement.expression().GetText();
+
+                if (stepText == "1")
+                {
+                    SaveContext(stepStatement);
+                }
             }
         }
     }
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/StopKeywordInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/StopKeywordInspection.cs
@@ -47,7 +47,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.StopStmtC
             return InspectionResults.StopKeywordInspection;
         }
 
-        public class StopKeywordListener : InspectionListenerBase<VBAParser.StopStmtContext>
+        private class StopKeywordListener : InspectionListenerBase<VBAParser.StopStmtContext>
         {
             public override void ExitStopStmt([NotNull] VBAParser.StopStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ThunderCode/LineContinuationBetweenKeywordsInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ThunderCode/LineContinuationBetweenKeywordsInspection.cs
@@ -36,7 +36,7 @@ protected override string ResultDescription(QualifiedContext<ParserRuleContext>
             return InspectionResults.LineContinuationBetweenKeywordsInspection.ThunderCodeFormat();
         }
 
-        public class LineContinuationBetweenKeywordsListener : InspectionListenerBase<ParserRuleContext>
+        private class LineContinuationBetweenKeywordsListener : InspectionListenerBase<ParserRuleContext>
         {
             public override void EnterSubStmt(VBAParser.SubStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ThunderCode/NegativeLineNumberInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ThunderCode/NegativeLineNumberInspection.cs
@@ -32,7 +32,7 @@ protected override string ResultDescription(QualifiedContext<ParserRuleContext>
             return InspectionResults.NegativeLineNumberInspection.ThunderCodeFormat();
         }
 
-        public class NegativeLineNumberKeywordsListener : InspectionListenerBase<ParserRuleContext>
+        private class NegativeLineNumberKeywordsListener : InspectionListenerBase<ParserRuleContext>
         {
             public override void EnterOnErrorStmt(VBAParser.OnErrorStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ThunderCode/OnErrorGoToMinusOneInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ThunderCode/OnErrorGoToMinusOneInspection.cs
@@ -31,7 +31,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.OnErrorSt
             return InspectionResults.OnErrorGoToMinusOneInspection.ThunderCodeFormat();
         }
 
-        public class OnErrorGoToMinusOneListener : InspectionListenerBase<VBAParser.OnErrorStmtContext>
+        private class OnErrorGoToMinusOneListener : InspectionListenerBase<VBAParser.OnErrorStmtContext>
         {
             public override void EnterOnErrorStmt(VBAParser.OnErrorStmtContext context)
             {
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/UnhandledOnErrorResumeNextInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/UnhandledOnErrorResumeNextInspection.cs

Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,9 @@`
`8`	`8`
`9`	`9`	`namespace Rubberduck.Inspections.Abstract`
`10`	`10`	`{`
	`11`	`+ /// <summary>`
	`12`	`+ /// This is a base class for the other declaration inspection base classes. It should not be implemented directly by concrete inspections.`
	`13`	`+ /// </summary>`
`11`	`14`	`public abstract class DeclarationInspectionBaseBase : InspectionBase`
`12`	`15`	`{`
`13`	`16`	`private readonly DeclarationType[] _relevantDeclarationTypes;`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.IfStmtCon`
`59`	`59`	`literalText);`
`60`	`60`	`}`
`61`	`61`
`62`		`- public class BooleanAssignedInIfElseListener : InspectionListenerBase<VBAParser.IfStmtContext>`
	`62`	`+ private class BooleanAssignedInIfElseListener : InspectionListenerBase<VBAParser.IfStmtContext>`
`63`	`63`	`{`
`64`	`64`	`public override void ExitIfStmt(VBAParser.IfStmtContext context)`
`65`	`65`	`{`
Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ private string GetTypeOfDefType(string defType)`
`68`	`68`	`{ "DefVar", "Variant" }`
`69`	`69`	`};`
`70`	`70`
`71`		`- public class DefTypeStatementInspectionListener : InspectionListenerBase<VBAParser.DefTypeContext>`
	`71`	`+ private class DefTypeStatementInspectionListener : InspectionListenerBase<VBAParser.DefTypeContext>`
`72`	`72`	`{`
`73`	`73`	`public override void ExitDefType([NotNull] VBAParser.DefTypeContext context)`
`74`	`74`	`{`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`
`8`	`8`	`namespace Rubberduck.Inspections.Concrete`
`9`	`9`	`{`
`10`		`- public class EmptyBlockInspectionListenerBase<TContext> : InspectionListenerBase<TContext>`
	`10`	`+ internal class EmptyBlockInspectionListenerBase<TContext> : InspectionListenerBase<TContext>`
`11`	`11`	`where TContext : ParserRuleContext`
`12`	`12`	`{`
`13`	`13`	`public void InspectBlockForExecutableStatements<T>(VBAParser.BlockContext block, T context) where T : TContext`
Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.CaseClaus`
`55`	`55`	`return InspectionResults.EmptyCaseBlockInspection;`
`56`	`56`	`}`
`57`	`57`
`58`		`- public class EmptyCaseBlockListener : EmptyBlockInspectionListenerBase<VBAParser.CaseClauseContext>`
	`58`	`+ private class EmptyCaseBlockListener : EmptyBlockInspectionListenerBase<VBAParser.CaseClauseContext>`
`59`	`59`	`{`
`60`	`60`	`public override void EnterCaseClause([NotNull] VBAParser.CaseClauseContext context)`
`61`	`61`	`{`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.DoLoopStm`
`50`	`50`	`return InspectionResults.EmptyDoWhileBlockInspection;`
`51`	`51`	`}`
`52`	`52`
`53`		`- public class EmptyDoWhileBlockListener : EmptyBlockInspectionListenerBase<VBAParser.DoLoopStmtContext>`
	`53`	`+ private class EmptyDoWhileBlockListener : EmptyBlockInspectionListenerBase<VBAParser.DoLoopStmtContext>`
`54`	`54`	`{`
`55`	`55`	`public override void EnterDoLoopStmt([NotNull] VBAParser.DoLoopStmtContext context)`
`56`	`56`	`{`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ protected override string ResultDescription(QualifiedContext<VBAParser.ElseBlock`
`52`	`52`	`return InspectionResults.EmptyElseBlockInspection;`
`53`	`53`	`}`
`54`	`54`
`55`		`- public class EmptyElseBlockListener : EmptyBlockInspectionListenerBase<VBAParser.ElseBlockContext>`
	`55`	`+ private class EmptyElseBlockListener : EmptyBlockInspectionListenerBase<VBAParser.ElseBlockContext>`
`56`	`56`	`{`
`57`	`57`	`public override void EnterElseBlock([NotNull] VBAParser.ElseBlockContext context)`
`58`	`58`	`{`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ protected override string ResultDescription(QualifiedContext<ParserRuleContext>`
`49`	`49`
`50`	`50`	`protected override IInspectionListener<ParserRuleContext> ContextListener { get; }`
`51`	`51`
`52`		`- public class EmptyIfBlockListener : EmptyBlockInspectionListenerBase<ParserRuleContext>`
	`52`	`+ private class EmptyIfBlockListener : EmptyBlockInspectionListenerBase<ParserRuleContext>`
`53`	`53`	`{`
`54`	`54`	`public override void EnterIfStmt([NotNull] VBAParser.IfStmtContext context)`
`55`	`55`	`{`