capnproto / pycapnp

Cap'n Proto serialization/RPC system - Python bindings
BSD 2-Clause "Simplified" License
458 stars 125 forks source link

Properly join list of methods in _DynamicCapabilityClient #345

Closed fabiorossetto closed 8 months ago

fabiorossetto commented 8 months ago

This was already fixed in c9bea05f44, but the fix does not seem to work. This commit uses a set union, which should be more robust. It also adds a couple of assertions to verify that it indeed works.

fabiorossetto commented 8 months ago

@lewinb-corp FYI

haata commented 8 months ago

Thanks!