T
twostars
Server: Split client socket management (#786)
* Separate client socket logic from server socket logic
* Rename SocketManager -> TcpSocketManager since we're exclusively dealing with TCP sockets.
* Strip "Server*" from member naming; these are now shared.
* Remove duplicate OnPost* events and make them virtual for overriding.
* Move server-specific implementation code into TcpServerSocketManager
* Move client socket logic into TcpClientSocketManager
* Remove outdated comment explaining TcpSocketManager
* Remove unnecessary virtual methods
* Manage sockets as shared pointers
* Update shared-server/CMakeLists.txt
* Tweak asio post callback signatures and explicitly copy the shared pointer
We can also move Close() back to TcpSocket. The only difference is the log message,
but we should fix this separately.
* Attach socket & manager class names to log entries
Github Commit Görüntüle...
See Commints...
..
* Separate client socket logic from server socket logic
* Rename SocketManager -> TcpSocketManager since we're exclusively dealing with TCP sockets.
* Strip "Server*" from member naming; these are now shared.
* Remove duplicate OnPost* events and make them virtual for overriding.
* Move server-specific implementation code into TcpServerSocketManager
* Move client socket logic into TcpClientSocketManager
* Remove outdated comment explaining TcpSocketManager
* Remove unnecessary virtual methods
* Manage sockets as shared pointers
* Update shared-server/CMakeLists.txt
* Tweak asio post callback signatures and explicitly copy the shared pointer
We can also move Close() back to TcpSocket. The only difference is the log message,
but we should fix this separately.
* Attach socket & manager class names to log entries
Github Commit Görüntüle...
See Commints...
..