VB -> C#: Duplicate break in switch statement with Exit Select

[mrmonday]

When using Exit Select in VB, the converter generates duplicate break statements in C#.

Input code

Class A
    Public Function Add(ByVal x As Integer) As Integer
        Select Case x
            Case 1
                Exit Select
        End Select
        Return 3
    End Function
End Class

Erroneous output

class A
{
    public int Add(int x)
    {
        switch (x)
        {
            case 1:
                {
                    break;
                    break;
                }
        }
        return 3;
    }
}

Expected output

class A
{
    public int Add(int x)
    {
        switch (x)
        {
            case 1:
                {
                    break;
                }
        }
        return 3;
    }
}

Details

• Version in use: 7.2.0

warning CS0162: Unreachable code detected

[GrahamTheCoder]

Yep I'm guessing it just always sticks a break at the end, just linking #432 for reference

[mrmonday]

Interestingly it doesn't - I had to add the if statement in #432 to trigger it - if you just have Select Case x Case 1 Return 3 the break isn't added.