search

jeikabu / nng.NETCore

MIT License
40 stars 22 forks source link

Update to nng 1.3 #79

Closed tomekki closed 4 years ago

tomekki commented 4 years ago

Our team recently run into a dispose bug, which is fixed by now: https://github.com/nanomsg/nng/commit/c726f93041b8dcd085c88144f2c13a2ac6fe58ad For this bug alone It would be good to update the wrapper.

jeikabu commented 4 years ago

I did some of the initial work in #76 but there were a few test failures I didn't have time to look into. I'll try to wrap that up.

jeikabu commented 4 years ago

Our tests are unfortunately generally unreliable, but the behaviour of nng_msg_clear definitely changed. Will see what the response to https://github.com/nanomsg/nng/issues/1252 is, adjust the test accordingly, and then commit #76 .

jeikabu commented 4 years ago

We can live with that as a "known issue".