Fields in HttpMethods cannot be used in switch

[vanillajonathan]

The fields are declared as public static readonly string hence cant be used in a switch statement.

https://github.com/aspnet/HttpAbstractions/blob/dev/src/Microsoft.AspNetCore.Http.Abstractions/HttpMethods.cs

Why aren't these declared as constants instead of static readonly?

[benaadams]

Can't do the very fast ReferenceEquals on a constant

[khellang]

Can't do the very fast ReferenceEquals on a constant

Is this true? I thought string constants are interned?

[benaadams]

Not across assembly boundaries

[khellang]

Anyway, this has been discussed time and time again, and is definitely a duplicate; #795

[khellang]

Not across assembly boundaries

@halter73 says otherwise... #712 (comment) 😉

[benaadams]

https://github.com/dotnet/coreclr/issues/14208#issuecomment-332586251

FYI, literal strings are OFTEN interned, but are not guaranteed to be interned.

[khellang]

It's hard to argue with Vance 😂

[benaadams]

As a guess you could use a pattern matching switch to use a switch anyway?

[muratg]

Dup of a few bugs as @khellang correctly points out :)