search

janet-lang / janet

A dynamic language and bytecode vm
https://janet-lang.org
MIT License
3.38k stars 217 forks source link

net/ev: Cleaned up unused NetStateConnect, fixed janet_async_end() ev refcount #1325

Closed zevv closed 8 months ago

zevv commented 8 months ago

Not sure if this makes proper sense, needs some review.

This cleans up the unused NetStateConnect state in the connect state machine, but not allocating any state to pass to janet_async_start() hung my janet at cleanup time; I believe the refcount dec should not depend on state of evdata

bakpakin commented 8 months ago

LGTM as it has passed CI, so it is working on windows