Merge branch 'rubberduck-vba/next' into ConflictDetectionSession

rubberduck-vba · Aug 6, 2020 · c525a8d · c525a8d
2 parents 8db77b7 + 9ed85e0
commit c525a8d
Show file tree

Hide file tree

Showing 216 changed files with 1,736 additions and 810 deletions.
diff --git a/.gitignore b/.gitignore
@@ -190,3 +190,4 @@ Rubberduck.CodeAnalysis.xml
 #Gradle 
 /.gradle/
 /Rubberduck.CodeAnalysis/Rubberduck.CodeAnalysis.xml
+/Rubberduck.Parsing/Rubberduck.Parsing.xml
diff --git a/...ApplicationWorksheetFunctionInspection.cs → ...ApplicationWorksheetFunctionInspection.cs b/...ApplicationWorksheetFunctionInspection.cs → ...ApplicationWorksheetFunctionInspection.cs
@@ -8,7 +8,7 @@
 using Rubberduck.Parsing.VBA.DeclarationCaching;
 using Rubberduck.Resources.Inspections;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.Excel
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Warns about late-bound WorksheetFunction calls made against the extended interface of the Application object.

diff --git a/...te/EmptyBlock/EmptyCaseBlockInspection.cs → ...ions/Concrete/EmptyCaseBlockInspection.cs b/...te/EmptyBlock/EmptyCaseBlockInspection.cs → ...ions/Concrete/EmptyCaseBlockInspection.cs
@@ -5,7 +5,7 @@
 using Rubberduck.Parsing.VBA;
 using Rubberduck.Resources.Inspections;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.EmptyBlock
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Identifies empty 'Case' blocks that can be safely removed.

diff --git a/...EmptyBlock/EmptyDoWhileBlockInspection.cs → ...s/Concrete/EmptyDoWhileBlockInspection.cs b/...EmptyBlock/EmptyDoWhileBlockInspection.cs → ...s/Concrete/EmptyDoWhileBlockInspection.cs
@@ -5,7 +5,7 @@
 using Rubberduck.Parsing.VBA;
 using Rubberduck.Resources.Inspections;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.EmptyBlock
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Identifies empty 'Do...Loop While' blocks that can be safely removed.

diff --git a/...te/EmptyBlock/EmptyElseBlockInspection.cs → ...ions/Concrete/EmptyElseBlockInspection.cs b/...te/EmptyBlock/EmptyElseBlockInspection.cs → ...ions/Concrete/EmptyElseBlockInspection.cs
@@ -5,7 +5,7 @@
 using Rubberduck.Parsing.VBA;
 using Rubberduck.Resources.Inspections;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.EmptyBlock
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Identifies empty 'Else' blocks that can be safely removed.

diff --git a/...EmptyBlock/EmptyForEachBlockInspection.cs → ...s/Concrete/EmptyForEachBlockInspection.cs b/...EmptyBlock/EmptyForEachBlockInspection.cs → ...s/Concrete/EmptyForEachBlockInspection.cs
@@ -5,7 +5,7 @@
 using Rubberduck.Parsing.VBA;
 using Rubberduck.Resources.Inspections;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.EmptyBlock
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Identifies empty 'For Each...Next' blocks that can be safely removed.

diff --git a/...EmptyBlock/EmptyForLoopBlockInspection.cs → ...s/Concrete/EmptyForLoopBlockInspection.cs b/...EmptyBlock/EmptyForLoopBlockInspection.cs → ...s/Concrete/EmptyForLoopBlockInspection.cs
@@ -5,7 +5,7 @@
 using Rubberduck.Parsing.VBA;
 using Rubberduck.Resources.Inspections;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.EmptyBlock
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Identifies empty 'For...Next' blocks that can be safely removed.

diff --git a/...rete/EmptyBlock/EmptyIfBlockInspection.cs → ...ctions/Concrete/EmptyIfBlockInspection.cs b/...rete/EmptyBlock/EmptyIfBlockInspection.cs → ...ctions/Concrete/EmptyIfBlockInspection.cs
@@ -6,7 +6,7 @@
 using Rubberduck.Parsing.VBA;
 using Rubberduck.Resources.Inspections;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.EmptyBlock
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Identifies empty 'If' blocks.

diff --git a/...ptyBlock/EmptyWhileWendBlockInspection.cs → ...Concrete/EmptyWhileWendBlockInspection.cs b/...ptyBlock/EmptyWhileWendBlockInspection.cs → ...Concrete/EmptyWhileWendBlockInspection.cs
@@ -5,7 +5,7 @@
 using Rubberduck.Parsing.VBA;
 using Rubberduck.Resources.Inspections;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.EmptyBlock
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Identifies empty 'While...Wend' blocks that can be safely removed.

diff --git a/.../ExcelMemberMayReturnNothingInspection.cs → .../ExcelMemberMayReturnNothingInspection.cs b/.../ExcelMemberMayReturnNothingInspection.cs → .../ExcelMemberMayReturnNothingInspection.cs
@@ -7,7 +7,7 @@
 using Rubberduck.Parsing.VBA.DeclarationCaching;
 using Rubberduck.Resources.Inspections;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.Excel
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>Locates instances of member calls made against the result of a Range.Find/FindNext/FindPrevious method, without prior validation.</summary>
     /// <reference name="Excel" />

diff --git a/...lUdfNameIsValidCellReferenceInspection.cs → ...lUdfNameIsValidCellReferenceInspection.cs b/...lUdfNameIsValidCellReferenceInspection.cs → ...lUdfNameIsValidCellReferenceInspection.cs
@@ -11,7 +11,7 @@
 using Rubberduck.VBEditor;
 using Rubberduck.VBEditor.SafeComWrappers;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.Excel
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Locates public User-Defined Function procedures accidentally named after a cell reference.

diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ImplementedInterfaceMemberInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ImplementedInterfaceMemberInspection.cs
@@ -2,7 +2,7 @@
 using Rubberduck.CodeAnalysis.Inspections.Abstract;
 using Rubberduck.CodeAnalysis.Inspections.Extensions;
 using Rubberduck.Common;
-using Rubberduck.Parsing.Annotations;
+using Rubberduck.Parsing.Annotations.Concrete;
 using Rubberduck.Parsing.Symbols;
 using Rubberduck.Parsing.VBA;
 using Rubberduck.Parsing.VBA.DeclarationCaching;

diff --git a/...ImplicitActiveSheetReferenceInspection.cs → ...ImplicitActiveSheetReferenceInspection.cs b/...ImplicitActiveSheetReferenceInspection.cs → ...ImplicitActiveSheetReferenceInspection.cs
@@ -7,7 +7,7 @@
 using Rubberduck.Parsing.VBA.DeclarationCaching;
 using Rubberduck.Resources.Inspections;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.Excel
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Locates unqualified Worksheet.Range/Cells/Columns/Rows member calls that implicitly refer to ActiveSheet.

diff --git a/...licitActiveWorkbookReferenceInspection.cs → ...licitActiveWorkbookReferenceInspection.cs b/...licitActiveWorkbookReferenceInspection.cs → ...licitActiveWorkbookReferenceInspection.cs
@@ -7,7 +7,7 @@
 using Rubberduck.Parsing.VBA.DeclarationCaching;
 using Rubberduck.Resources.Inspections;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.Excel
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Locates unqualified Workbook.Worksheets/Sheets/Names member calls that implicitly refer to ActiveWorkbook.

diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ModuleWithoutFolderInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ModuleWithoutFolderInspection.cs
@@ -1,6 +1,6 @@
 using System.Linq;
 using Rubberduck.CodeAnalysis.Inspections.Abstract;
-using Rubberduck.Parsing.Annotations;
+using Rubberduck.Parsing.Annotations.Concrete;
 using Rubberduck.Parsing.Symbols;
 using Rubberduck.Parsing.VBA;
 using Rubberduck.Parsing.VBA.DeclarationCaching;

diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/README.md b/Rubberduck.CodeAnalysis/Inspections/Concrete/README.md
@@ -0,0 +1,18 @@
+## Rubberduck.CodeAnalysis.Inspections.Concrete
+
+All concrete code inspection implementations (classes) must be in their own .cs source file **in a namespace that corresponds to its folder location**, under this namespace.
+
+The xml-doc content in this namespace is automatically downloaded, processed, and ultimately served on the rubberduckvba.com website feature pages.
+
+Each inspection can have as many examples using as many modules of as many types as necessary. The following string values are recognized as module types:
+
+- "Standard Module"
+- "Class Module"
+- "Predeclared Class"
+- "Interface Module"
+- "UserForm Module"
+- "Document Module"
+
+The "edit this page" link on each page generated from xml-doc content in this namespace, links to `https://github.com/rubberduck-vba/Rubberduck/edit/next/{namespace}/{inspection-name}.cs`; it is imperative that the files' folder location corresponds to their namespace, lest we generate broken links on the website.
+
+The content generated from xml-doc in this namespace (and any concrete inspections in a namespace under it) is accessible at `https://rubberduckvba.com/inspections/details/{inspection-name}`.
diff --git a/...cel/SheetAccessedUsingStringInspection.cs → ...ete/SheetAccessedUsingStringInspection.cs b/...cel/SheetAccessedUsingStringInspection.cs → ...ete/SheetAccessedUsingStringInspection.cs
@@ -12,7 +12,7 @@
 using Rubberduck.VBEditor.SafeComWrappers;
 using Rubberduck.VBEditor.SafeComWrappers.Abstract;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.Excel
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Locates ThisWorkbook.Worksheets and ThisWorkbook.Sheets calls that appear to be dereferencing a worksheet that is already accessible at compile-time with a global-scope identifier.

diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ThunderCode/OnErrorGoToMinusOneInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ThunderCode/OnErrorGoToMinusOneInspection.cs
@@ -13,7 +13,7 @@ namespace Rubberduck.CodeAnalysis.Inspections.Concrete.ThunderCode
     /// <why>
     /// This inpection is flagging code we dubbed "ThunderCode", 
     /// code our friend Andrew Jackson would have written to confuse Rubberduck's parser and/or resolver. 
-    /// 'On Error GoTo -1' is poorly documented and uselessly complicates error handling.
+    /// 'On Error GoTo -1' is poorly documented and uselessly complicates error handling. Consider using 'On Error GoTo 0' instead.
     /// </why>
     internal sealed class OnErrorGoToMinusOneInspection : ParseTreeInspectionBase<VBAParser.OnErrorStmtContext>
     {

diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ThunderCode/README.md b/Rubberduck.CodeAnalysis/Inspections/Concrete/ThunderCode/README.md
@@ -0,0 +1,5 @@
+## Rubberduck.CodeAnalysis.Inspections.Concrete.ThunderCode
+
+Inspections in this namespace should have a *hidden* attribute on the summary tag, e.g. `<summary hidden="true">`. When the xml-doc is processed, this attribute toggles this content to be displayed:
+
+> *This feature is hidden. It could be an Easter egg, or a problematic feature that is likely (hopefully?) disabled by default.*
diff --git a/...ableCaseInspection/ComparableDateValue.cs → ...ableCaseEvaluation/ComparableDateValue.cs b/...ableCaseInspection/ComparableDateValue.cs → ...ableCaseEvaluation/ComparableDateValue.cs
@@ -5,7 +5,7 @@
 using Antlr4.Runtime;
 using Rubberduck.Parsing.PreProcessing;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal class ComparableDateValue : IValue, IComparable<ComparableDateValue>
     {

diff --git a/...achableCaseInspection/ExpressionFilter.cs → ...achableCaseEvaluation/ExpressionFilter.cs b/...achableCaseInspection/ExpressionFilter.cs → ...achableCaseEvaluation/ExpressionFilter.cs
@@ -5,7 +5,7 @@
 using System.Text;
 using Rubberduck.Parsing.Grammar;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal enum VariableClauseTypes
     {

diff --git a/...CaseInspection/ExpressionFilterBoolean.cs → ...CaseEvaluation/ExpressionFilterBoolean.cs b/...CaseInspection/ExpressionFilterBoolean.cs → ...CaseEvaluation/ExpressionFilterBoolean.cs
@@ -1,6 +1,6 @@
 using Rubberduck.Parsing.Grammar;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal class ExpressionFilterBoolean : ExpressionFilter<bool>
     {

diff --git a/...bleCaseInspection/ExpressionFilterDate.cs → ...bleCaseEvaluation/ExpressionFilterDate.cs b/...bleCaseInspection/ExpressionFilterDate.cs → ...bleCaseEvaluation/ExpressionFilterDate.cs
@@ -2,7 +2,7 @@
 using System.Linq;
 using Rubberduck.Parsing.Grammar;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal class ExpressionFilterDate : ExpressionFilter<ComparableDateValue>
     {

diff --git a/...CaseInspection/ExpressionFilterFactory.cs → ...CaseEvaluation/ExpressionFilterFactory.cs b/...CaseInspection/ExpressionFilterFactory.cs → ...CaseEvaluation/ExpressionFilterFactory.cs
@@ -3,8 +3,8 @@
 using System.Linq;
 using Rubberduck.Parsing.Grammar;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection{
-
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
+{ 
     internal static class ExpressionFilterFactory
     {
         private static readonly Dictionary<string, (long typeMin, long typeMax)> IntegralNumberExtents = new Dictionary<string, (long typeMin, long typeMax)>()

diff --git a/...aseInspection/ExpressionFilterIntegral.cs → ...aseEvaluation/ExpressionFilterIntegral.cs b/...aseInspection/ExpressionFilterIntegral.cs → ...aseEvaluation/ExpressionFilterIntegral.cs
@@ -2,7 +2,7 @@
 using System.Collections.Generic;
 using System.Linq;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal class ExpressionFilterIntegral : ExpressionFilter<long>
     {

diff --git a/...UnreachableCaseInspection/FilterLimits.cs → ...UnreachableCaseEvaluation/FilterLimits.cs b/...UnreachableCaseInspection/FilterLimits.cs → ...UnreachableCaseEvaluation/FilterLimits.cs
@@ -1,6 +1,6 @@
 using System;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal struct Limit<T> where T : IComparable<T>
     {

diff --git a/...te/UnreachableCaseInspection/LetCoerce.cs → ...te/UnreachableCaseEvaluation/LetCoerce.cs b/...te/UnreachableCaseInspection/LetCoerce.cs → ...te/UnreachableCaseEvaluation/LetCoerce.cs
@@ -4,7 +4,7 @@
 using Rubberduck.Parsing.Grammar;
 using Rubberduck.Parsing.PreProcessing;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal struct VBACurrency
     {

diff --git a/...leCaseInspection/OperatorTypesProvider.cs → ...leCaseEvaluation/OperatorTypesProvider.cs b/...leCaseInspection/OperatorTypesProvider.cs → ...leCaseEvaluation/OperatorTypesProvider.cs
@@ -3,7 +3,7 @@
 using System.Linq;
 using Rubberduck.Parsing.Grammar;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal class OperatorTypesProvider
     {

diff --git a/...nspection/ParseTreeExpressionEvaluator.cs → ...valuation/ParseTreeExpressionEvaluator.cs b/...nspection/ParseTreeExpressionEvaluator.cs → ...valuation/ParseTreeExpressionEvaluator.cs
@@ -6,7 +6,7 @@
 using Rubberduck.Parsing.Grammar;
 using Rubberduck.Parsing.PreProcessing;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal interface IParseTreeExpressionEvaluator
     {

diff --git a/...reachableCaseInspection/ParseTreeValue.cs → ...reachableCaseEvaluation/ParseTreeValue.cs b/...reachableCaseInspection/ParseTreeValue.cs → ...reachableCaseEvaluation/ParseTreeValue.cs
@@ -4,7 +4,7 @@
 using Rubberduck.Parsing.Grammar;
 using Rubberduck.Parsing.PreProcessing;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal interface IParseTreeValue
     {

diff --git a/...leCaseInspection/ParseTreeValueFactory.cs → ...leCaseEvaluation/ParseTreeValueFactory.cs b/...leCaseInspection/ParseTreeValueFactory.cs → ...leCaseEvaluation/ParseTreeValueFactory.cs
@@ -3,7 +3,7 @@
 using Rubberduck.Parsing;
 using Rubberduck.Parsing.Grammar;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal interface IParseTreeValueFactory
     {

diff --git a/...leCaseInspection/ParseTreeValueVisitor.cs → ...leCaseEvaluation/ParseTreeValueVisitor.cs b/...leCaseInspection/ParseTreeValueVisitor.cs → ...leCaseEvaluation/ParseTreeValueVisitor.cs
@@ -9,7 +9,7 @@
 using Rubberduck.Parsing.VBA.DeclarationCaching;
 using Rubberduck.VBEditor;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal interface IParseTreeValueVisitor
     {

diff --git a/...CaseInspection/ParseTreeVisitorResults.cs → ...CaseEvaluation/ParseTreeVisitorResults.cs b/...CaseInspection/ParseTreeVisitorResults.cs → ...CaseEvaluation/ParseTreeVisitorResults.cs
@@ -4,7 +4,7 @@
 using Antlr4.Runtime;
 using Rubberduck.Parsing.Grammar;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal interface IParseTreeVisitorResults
     {

diff --git a/...leCaseInspection/RangeClauseExpression.cs → ...leCaseEvaluation/RangeClauseExpression.cs b/...leCaseInspection/RangeClauseExpression.cs → ...leCaseEvaluation/RangeClauseExpression.cs
@@ -1,7 +1,7 @@
 using System.Collections.Generic;
 using Rubberduck.Parsing.Grammar;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal interface IRangeClauseExpression
     {

diff --git a/...nreachableCaseInspection/TypeTokenPair.cs → ...nreachableCaseEvaluation/TypeTokenPair.cs b/...nreachableCaseInspection/TypeTokenPair.cs → ...nreachableCaseEvaluation/TypeTokenPair.cs
@@ -4,7 +4,7 @@
 using System.Linq;
 using Rubberduck.Parsing.Grammar;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal struct TypeTokenPair
     {

diff --git a/...aseInspection/UnreachableCaseInspector.cs → ...aseEvaluation/UnreachableCaseInspector.cs b/...aseInspection/UnreachableCaseInspector.cs → ...aseEvaluation/UnreachableCaseInspector.cs
@@ -8,7 +8,7 @@
 using Rubberduck.Parsing.VBA.DeclarationCaching;
 using Rubberduck.VBEditor;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation
 {
     internal interface IUnreachableCaseInspector
     {

diff --git a/...seInspection/UnreachableCaseInspection.cs → ...ons/Concrete/UnreachableCaseInspection.cs b/...seInspection/UnreachableCaseInspection.cs → ...ons/Concrete/UnreachableCaseInspection.cs
@@ -14,8 +14,9 @@
 using Rubberduck.Parsing.VBA.Parsing;
 using Rubberduck.Resources.Inspections;
 using Rubberduck.VBEditor;
+using Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseEvaluation;
 
-namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
+namespace Rubberduck.CodeAnalysis.Inspections.Concrete
 {
     /// <summary>
     /// Flags 'Case' blocks that will never execute.
@@ -104,7 +105,7 @@ namespace Rubberduck.CodeAnalysis.Inspections.Concrete.UnreachableCaseInspection
     /// <module name="MyModule" type="Standard Module">
     /// <![CDATA[
     /// 
-    /// 'The inspecion flags Range Clauses that are not of the required form:
+    /// 'The inspection flags Range Clauses that are not of the required form:
     /// '[x] To [y] where [x] less than or equal to [y]
     /// 
     /// Private Sub ExampleInvalidRangeExpression(ByVal value As String)

diff --git a/Rubberduck.CodeAnalysis/Inspections/Extensions/IgnoreRelatedExtensions.cs b/Rubberduck.CodeAnalysis/Inspections/Extensions/IgnoreRelatedExtensions.cs
@@ -1,7 +1,7 @@
 using System.Linq;
 using Rubberduck.CodeAnalysis.Inspections.Results;
 using Rubberduck.Parsing;
-using Rubberduck.Parsing.Annotations;
+using Rubberduck.Parsing.Annotations.Concrete;
 using Rubberduck.Parsing.Symbols;
 using Rubberduck.Parsing.VBA.DeclarationCaching;
 using Rubberduck.VBEditor;

diff --git a/Rubberduck.CodeAnalysis/QuickFixes/Concrete/AccessSheetUsingCodeNameQuickFix.cs b/Rubberduck.CodeAnalysis/QuickFixes/Concrete/AccessSheetUsingCodeNameQuickFix.cs
@@ -1,7 +1,7 @@
 using System.Linq;
 using Antlr4.Runtime;
 using Rubberduck.CodeAnalysis.Inspections;
-using Rubberduck.CodeAnalysis.Inspections.Concrete.Excel;
+using Rubberduck.CodeAnalysis.Inspections.Concrete;
 using Rubberduck.CodeAnalysis.Inspections.Results;
 using Rubberduck.CodeAnalysis.QuickFixes.Abstract;
 using Rubberduck.Parsing;

diff --git a/Rubberduck.CodeAnalysis/QuickFixes/Concrete/ApplicationWorksheetFunctionQuickFix.cs b/Rubberduck.CodeAnalysis/QuickFixes/Concrete/ApplicationWorksheetFunctionQuickFix.cs
@@ -1,5 +1,5 @@
 using Rubberduck.CodeAnalysis.Inspections;
-using Rubberduck.CodeAnalysis.Inspections.Concrete.Excel;
+using Rubberduck.CodeAnalysis.Inspections.Concrete;
 using Rubberduck.CodeAnalysis.QuickFixes.Abstract;
 using Rubberduck.Parsing.Rewriter;
 

diff --git a/Rubberduck.CodeAnalysis/QuickFixes/Concrete/IgnoreInModuleQuickFix.cs b/Rubberduck.CodeAnalysis/QuickFixes/Concrete/IgnoreInModuleQuickFix.cs
@@ -3,6 +3,7 @@
 using Rubberduck.CodeAnalysis.Inspections;
 using Rubberduck.CodeAnalysis.Inspections.Attributes;
 using Rubberduck.CodeAnalysis.QuickFixes.Abstract;
+using Rubberduck.Parsing.Annotations.Concrete;
 using Rubberduck.Parsing.Annotations;
 using Rubberduck.Parsing.Rewriter;
 using Rubberduck.Parsing.Symbols;

diff --git a/Rubberduck.CodeAnalysis/QuickFixes/Concrete/IgnoreOnceQuickFix.cs b/Rubberduck.CodeAnalysis/QuickFixes/Concrete/IgnoreOnceQuickFix.cs
@@ -4,7 +4,7 @@
 using Rubberduck.CodeAnalysis.Inspections.Attributes;
 using Rubberduck.CodeAnalysis.QuickFixes.Abstract;
 using Rubberduck.Parsing;
-using Rubberduck.Parsing.Annotations;
+using Rubberduck.Parsing.Annotations.Concrete;
 using Rubberduck.Parsing.Rewriter;
 using Rubberduck.Parsing.Symbols;
 using Rubberduck.Parsing.VBA;