From ff0b764af198e94616bb1d4811083c6c724b2be0 Mon Sep 17 00:00:00 2001 From: Skydev0h Date: Thu, 4 Jun 2015 12:29:59 +0300 Subject: [PATCH] Fix ResponseContexts not being returned to pull Fix FinishReceiving not being called in normal flow of OnReceived Tweak FinishReceiving to not do ReturnTransport (it pollutes transport pool) --- .../Rpc/Client/Protocols/ClientTransport.cs | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/MsgPack.Rpc.Client/Rpc/Client/Protocols/ClientTransport.cs b/src/MsgPack.Rpc.Client/Rpc/Client/Protocols/ClientTransport.cs index f1d2693..30a2b72 100644 --- a/src/MsgPack.Rpc.Client/Rpc/Client/Protocols/ClientTransport.cs +++ b/src/MsgPack.Rpc.Client/Rpc/Client/Protocols/ClientTransport.cs @@ -1080,9 +1080,11 @@ protected virtual void OnReceived( ClientResponseContext context ) return; } - this.FinishReceiving( context ); - return; + //this.FinishReceiving( context ); + //return; } + + this.FinishReceiving( context ); } private void FinishReceiving( ClientResponseContext context ) @@ -1090,7 +1092,7 @@ private void FinishReceiving( ClientResponseContext context ) context.StopWatchTimeout(); context.Timeout -= this.OnReceiveTimeout; this.Manager.ReturnResponseContext( context ); - this.Manager.ReturnTransport( this ); + //this.Manager.ReturnTransport( this ); } private static int? TryDetectMessageId( ClientResponseContext context )