2.2.0 warnings from Redim ... As

[SystemsModelling]

ReDim Result(StringLen) As String

gets me several warnings:
Variable 'Result' is used but not assigned
Local variable 'Result' is not declared.

This gives no warnings:

Dim Result() As String
ReDim Result(StringLen)

[Vogel612]

Could you add the extended version information from the logs, please? Thank You :)

[SystemsModelling]

RubberduckLog.txt

Code:

  Option Explicit
  
  Sub testReDim()
  Const length As Long = 2
  ReDim Result(length) As String
  End Sub
  
  Sub testDimReDim()
  Const length As Long = 2
  Dim Result() As String
  ReDim Result(length)
  End Sub

[retailcoder]

That's a tough one. The resolver currently isn't treating ReDim statements as declarations, because it literally can't know whether the identifier is or isn't Dim'd elsewhere (e.g. a global array declared in another module) in scope.

VBA ReDim statements are dual-purposed: they serve as executable declaration statements, and as a statement that resizes a dynamic array.

Rubberduck can't treat them as executable declaration statements, so "variable 'Results' is not declared" isn't entirely wrong, but it's not per language specs either. So yes, it's a bug, but I don't see how we can fix it without introducing an entire parse pass dedicated to finding what array identifiers are available in global scope - which isn't impossible, but feels overkill, when adding an explicit Dim statement removes it.

"Variable not assigned" inspection needs to ignore arrays though.

[ChrisBrackett]

This might be a duplicate of #2522.