Converting "variable scope" blocks.

[paul1956]

Converting "variable scope" blocks.

[paul1956]

@DualBrain I can't detect that, the problem is the second using should just be "Using parser as new TextFieldParser(path))" and that I might be able to fix.

[paul1956]

I fix the specific issue and other places where I should have used

Dim X As New Y

Fix in New in 5.0.0.17/5.0.0.8

[DualBrain]

You might have misunderstood what I was trying to convey.

public void SomeSuch()
{

    {
        int x = 1;
    }
    // ...
    {
        int x = 1;
    }
}

There are two different instances of x in their own individual variable scopes due to the curly brace "blocks". In VB this can be done using:

Public Sub SomeSuch

    If True Then
        Dim x = 1
    End If
    If True Then
        Dim x = 1
    End If

 End Sub

The question I was asking was related to the curly brace sections so that they could be detected (if possible) and turned into the "do nothing" (except create variable scoping) If / End If blocks.

[DualBrain]

Also, right now... if I attempt to convert this:

Public Sub SomeSuch()
    Dim x As Integer = 1
    ' ...
     Dim x As Integer = 1
    ' ...
End Sub

Which is obviously not a valid conversion. ;-)

[paul1956]

@DualBrain Fixed in 5.0.0.18/5.0.0.10, please review

[DualBrain]

Excellent work!

[paul1956]

@DualBrain thanks. Very odd the original code had a test identical to your your simplified example. Which I had to fix for some reason whoever wrote original code that it was OK to define a variable twice.