rsocket / rsocket-cpp

C++ implementation of RSocket
http://rsocket.io
Apache License 2.0
253 stars 99 forks source link

Don't use ConnectionSet in RSocketClient #815

Closed alexmalyshev closed 7 years ago

alexmalyshev commented 7 years ago

It only has one connection, why does it need a set of them?

Makes ~RSocketClient destroy the RSocketStateMachine synchronously, where previously this was async. I'm worried about users destroying their RSocketClient objects, and then having their EventBases accidentally access more state later via stream subscribers.