apigear-io / objectlink-core-cpp

A object link protocol for C++ 14
MIT License
0 stars 3 forks source link

Consider removing client registry #12

Closed dorotaphanSiili closed 1 year ago

dorotaphanSiili commented 1 year ago

Motivation: on the client side the client registry adds

*we keep it as raw pointers in registry, we could be safer changing to weak_ptrs but that requires change for creating objects, they need to be created only with factory method as shared ptrs, where for clientNode I think it is ok, for objectSink, which is remoteInterface that feels wrong.

w4bremer commented 1 year ago

Has been discussed and decided not to implement for now. Can be re-opened later if necessary.