Switch not Handling Goto Case XXX

[jcdickinson]

The switch/case logic is not handling goto case ...; instead either complicated ifs are generated or a bunch of labels and gotos are generated after the switch.

Set of ifs:

string test = "hello";

switch (test)
{
    case "hello":
        Console.WriteLine("Foo");
        goto case "bar";
    case "bar":
        Console.WriteLine("Bar");
        break;
    case "baz":
        Console.WriteLine("Bar");
        break;
    case "frob":
        Console.WriteLine("Bar");
        goto case "bar";
}

We get (which is likely 'correct'):

string test = "hello";
string text = test;
if (text != null)
{
    if (!(text == "hello"))
    {
        if (!(text == "bar"))
        {
            if (text == "baz")
            {
                Console.WriteLine("Bar");
                return;
            }
            if (!(text == "frob"))
            {
                return;
            }
            Console.WriteLine("Bar");
        }
    }
    else
    {
        Console.WriteLine("Foo");
    }
    Console.WriteLine("Bar");
}

For the goto example decompile System.Web.Mvc.Html.InputExtensions::InputHelper from System.Web.Mvc (Version 3.0.0.0).

[jcdickinson]

d'oh. I am using 2.0.0.1564 from 2012/02/19.

[dgrunwald]

These are actually two separate features missing from the decompiler:

1. We do not support detecting switch statements that get compiled to a series of if (we only detect switch statements using the switch IL opcode).
2. We never use goto case and use goto+label instead.

[Need Help] How can i remove "goto" after using ILSpy? It's too much :'(

[siegfriedpammer]

We just merged the switch branch, which adds goto case support.