Closed Bug 1530273 Opened 6 years ago Closed 6 years ago

Support "funcref" as alias for "anyfuc"

Categories

(Core :: JavaScript: WebAssembly, enhancement, P3)

enhancement

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: nobody → lhansen
Status: NEW → ASSIGNED

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.

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
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: