Stream: git-wasmtime

Topic: wasmtime / PR #7423 wasi-sockets: Sync up UDP implementat...


view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2023 at 16:14):

badeend opened PR #7423 from badeend:udp-tests to bytecodealliance:main:

No major functional changes:

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2023 at 16:14):

badeend requested alexcrichton for a review on PR #7423.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2023 at 16:14):

badeend requested wasmtime-core-reviewers for a review on PR #7423.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2023 at 16:33):

alexcrichton submitted PR review:

Thanks!

Sort of a side-question as it's not directly related to this PR itself, where's all this error-handling code coming from? This seems extensive enough that the tests probably aren't exercising all the various cases here and I'm curious if you're porting from another library for example or something like that.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2023 at 16:33):

alexcrichton submitted PR review:

Thanks!

Sort of a side-question as it's not directly related to this PR itself, where's all this error-handling code coming from? This seems extensive enough that the tests probably aren't exercising all the various cases here and I'm curious if you're porting from another library for example or something like that.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2023 at 16:33):

alexcrichton created PR review comment:

Can this fail if the port is already bound? We've had issues with tests running concurrently in the past and I think the TCP test has a case where it handles failure to bind here.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2023 at 19:13):

badeend submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2023 at 19:13):

badeend created PR review comment:

Ah right. Hadn't noticed the TCP test changed recently. Yes, this UDP test needs the same update.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 04 2023 at 19:39):

badeend updated PR #7423.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 04 2023 at 19:40):

badeend submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 04 2023 at 19:40):

badeend created PR review comment:

I've updated the test with the same workaround that's in the TCP test.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 05 2023 at 22:15):

alexcrichton merged PR #7423.


Last updated: Dec 23 2024 at 13:07 UTC