Merge pull request #3554 from Vogel612/jagged-arrays

Stop crashing resolver on jagged arrays.

Author: retailcoder

Rubberduck.Parsing/Binding/IndexDefaultBinding.cs

@@ -127,7 +127,7 @@ with a parameter list that cannot accept any parameters and an <argument-list> t
             bool isVariable = lExpression.Classification == ExpressionClassification.Variable;
             bool propertyWithParameters = lExpression.Classification == ExpressionClassification.Property && !((IParameterizedDeclaration)lExpression.ReferencedDeclaration).Parameters.Any();
             bool functionWithParameters = lExpression.Classification == ExpressionClassification.Function && !((IParameterizedDeclaration)lExpression.ReferencedDeclaration).Parameters.Any();
-            if (isVariable || ((propertyWithParameters || functionWithParameters) && _argumentList.HasArguments))
+            if (lExpression.ReferencedDeclaration != null && (isVariable || ((propertyWithParameters || functionWithParameters) && _argumentList.HasArguments)))
             {
                 IBoundExpression boundExpression = null;
                 var asTypeName = lExpression.ReferencedDeclaration.AsTypeName;

RubberduckTests/Grammar/ResolverTests.cs

@@ -112,6 +112,32 @@ End Function
             }
         }
 
+        [TestCategory("Resolver")]
+        [TestMethod]
+        public void JaggedArrayReference_DoesNotBlowUp()
+        {
+            // see https://github.com/rubberduck-vba/Rubberduck/issues/3098
+            var code = @"Option Explicit
+
+Public Sub Test()
+    Dim varTemp() As Variant
+    
+    ReDim varTemp(0)
+    
+    varTemp(0) = Array(0)
+    varTemp(0)(0) = Array(0)
+    
+    Debug.Print varTemp(0)(0)
+End Sub
+";
+
+            using (var state = Resolve(code))
+            {
+                var declaration = state.AllUserDeclarations.Single(item => item.DeclarationType == DeclarationType.Variable && item.IdentifierName == "varTemp");
+                Assert.IsTrue(declaration.IsArray);
+            }
+        }
+
         [TestCategory("Resolver")]
         [TestMethod]
         public void OptionalParameterDefaultConstValue_IsReferenceToDeclaredConst()