Closed
Bug 1530273
Opened 6 years ago
Closed 6 years ago
Support "funcref" as alias for "anyfuc"
Categories
(Core :: JavaScript: WebAssembly, enhancement, P3)
Core
JavaScript: WebAssembly
Tracking
()
RESOLVED
FIXED
mozilla67
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: lth, Assigned: lth, Mentored)
References
Details
(Keywords: good-first-bug)
Attachments
(1 file)
Wabt just landed a change that deprecated "anyfunc" and introduced / strengthened the use of "funcref", so we should at least accept the latter as an alias for the former.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → lhansen
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•6 years ago
|
||
Wabt is now supporting funcref exclusively, and with the reftypes proposal I think
there's broad agreement that we will stop using anyfunc. So let's accept funcref
both in the text format and in the table creation dictionary, and let's use this
name as the canonical name in error messages and similar.
But let's also continue to accept anyfunc, since there may be in-flight tests and
other content that uses it. This includes a couple of emscripten-compiled benchmarks
currently in the repo; I chose not to change those.
Assignee | ||
Updated•6 years ago
|
Keywords: dev-doc-needed
Pushed by lhansen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5c4daba349dc
Make 'funcref' the canonical name. r=bbouvier
Comment 3•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Updated•4 years ago
|
Keywords: dev-doc-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•