Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

js api: improve definition of error objects #810

Closed
Ms2ger opened this issue May 24, 2018 · 3 comments · Fixed by #1024
Closed

js api: improve definition of error objects #810

Ms2ger opened this issue May 24, 2018 · 3 comments · Fixed by #1024

Comments

@Ms2ger
Copy link
Collaborator

Ms2ger commented May 24, 2018

There's [LegacyNamespace=WebAssembly] interface CompileError { }; and then a bunch of prose requirements on the objects that are constructed because of that IDL. AFAICT the only thing that the IDL is supposed to do is define that the namespace object has properties called FooError. Let's just do that manually and avoid the conflict?

@ericprud
Copy link

@Ms2ger , can you see if this has been addressed and close this issue if not?

@Ms2ger
Copy link
Collaborator Author

Ms2ger commented May 17, 2019

There has been no attempt to address this.

@Ms2ger
Copy link
Collaborator Author

Ms2ger commented May 17, 2019

I'm working on it.

Ms2ger added a commit to Ms2ger/spec that referenced this issue May 22, 2019
This at least gets somewhat close to defining what's actually intended.

Fixes WebAssembly#810.
Ms2ger added a commit to Ms2ger/spec that referenced this issue May 27, 2019
This at least gets somewhat close to defining what's actually intended.

Fixes WebAssembly#810.
littledan pushed a commit that referenced this issue Sep 11, 2019
This at least gets somewhat close to defining what's actually intended.

Fixes #810.
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants