Remove gotos

Author: benaadams

src/Servers/Kestrel/Core/src/Internal/Http/HttpParser.cs

@@ -378,20 +378,20 @@ private static bool TryTakeSingleHeader(TRequestHandler handler, ReadOnlySpan<by
             if ((uint)nameEnd >= (uint)headerLine.Length)
             {
                 // Colon not found.
-                goto Reject;
+                return false;
             }
 
             // Early memory read to hide latency
             var expectedColon = headerLine[nameEnd];
             if (nameEnd == 0)
             {
                 // Header name is empty.
-                goto Reject;
+                return false;
             }
             if (expectedColon != ByteColon)
             {
                 // Header name space or tab.
-                goto Reject;
+                return false;
             }
 
             // Skip colon to get to the value start.
@@ -457,11 +457,6 @@ private static bool TryTakeSingleHeader(TRequestHandler handler, ReadOnlySpan<by
             handler.OnHeader(name: headerLine[..nameEnd], value: headerLine[valueStart..valueEnd]);
 
             return true;
-
-        Reject:
-            // Reject is a conditional jump forward as we expect most headers to be accepted,
-            // so we want it to be unpredicted by an unprimied branch predictor.
-            return false;
         }
 
         [MethodImpl(MethodImplOptions.NoInlining)]