Fields in HttpMethods cannot be used in switch

[Jonathan]

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?

[Ben Adams]

Can't do the very fast ReferenceEquals on a constant

[Kristian Hellang]

Can't do the very fast ReferenceEquals on a constant

Is this true? I thought string constants are interned?

[Ben Adams]

Not across assembly boundaries

[Kristian Hellang]

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

[Kristian Hellang]

Not across assembly boundaries

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

[Ben Adams]

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

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

[Kristian Hellang]

It's hard to argue with Vance 😂

[Ben Adams]

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

[Murat Girgin]

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