Merge pull request #136 from Cysharp/feature/FixUnnecessaryReleaseHandles

mayuki · web-flow · commit d6df6e692845 · 2025-05-14T18:01:24.000+09:00
Remove unnecessary handle release
diff --git a/src/YetAnotherHttpHandler/RequestContext.cs b/src/YetAnotherHttpHandler/RequestContext.cs
@@ -129,7 +129,6 @@ private async Task RunReadRequestLoopAsync(CancellationToken cancellationToken)
             finally
             {
                 if (YahaEventSource.Log.IsEnabled()) YahaEventSource.Log.Trace($"[ReqSeq:{_requestSequence}:State:0x{Handle:X}] Completing RunReadRequestLoopAsync");
-                TryReleaseNativeHandles();
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,6 @@ private async Task RunReadRequestLoopAsync(CancellationToken cancellationToken)`
`129`	`129`	`finally`
`130`	`130`	`{`
`131`	`131`	`if (YahaEventSource.Log.IsEnabled()) YahaEventSource.Log.Trace($"[ReqSeq:{_requestSequence}:State:0x{Handle:X}] Completing RunReadRequestLoopAsync");`
`132`		`- TryReleaseNativeHandles();`
`133`	`132`	`}`
`134`	`133`	`}`
`135`	`134`