- From: Sameer via GitHub <sysbot+gh@w3.org>
- Date: Fri, 12 Jan 2024 11:34:25 +0000
- To: public-webrtc-logs@w3.org
> Overall I don't mind this API, so I'm approving it for the sake of progress. > > However, I worry there are some API races here that we still need to sort out. We appear to have two async methods, select and remove basically, which appear as if they can race. We should clarify what's supposed to happen then. > > For instance, what happens if someone calls both methods at once with the same argument? > > My recommendation of adding an internal slot in a follow-up might help clarify this. Thank you! I think the internal slot and validations should address a lot of the race conditions. I would certainly want to fix any remaining issues once the proposed changes are submitted. I am happy to continue discussing and iterating. -- GitHub Notification of comment by sam-vi Please view or discuss this issue at https://github.com/w3c/webrtc-extensions/pull/175#issuecomment-1888933908 using your GitHub account -- Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config
Received on Friday, 12 January 2024 11:34:27 UTC