"else" not needed after "throw new ArgumentNullException"

[letiemble]

Hi,

I have noticed in a few places that an "else" is shown after argument checks (for nullity or anything else).

For example, in System.Collections.ArrayList :

public static ArrayList Adapter(IList list)
{
    if (list == null)
    {
        throw new ArgumentNullException("list");
    }
    else
    {
        return new ArrayList.IListWrapper(list);
    }
}

The else is not necessary.

[dsrbecky]

Should be fixed in e229194

[letiemble]

Great !!! Works like a charm even with multiple argument checks. Thanks.