emmenlau commented on PR #2745: URL: https://github.com/apache/thrift/pull/2745#issuecomment-1385668209
@cfriedt , I guess this is beyond your current scope, but for the sake of clean code, would it not be better to bind the `std::shared_ptr` to a custom destructor that calls `::THRIFT_CLOSESOCKET` on the socket upon destruction? And then again, why is the operating system not doing that in the first place? Please let me know if I'm completely on the wrong track here... -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
