search

tc39 / proposal-mass-proxy-revocation

Proposal for revoking proxies en masse.
MIT License
5 stars 1 forks source link

Minor spec bug: options.signal might not exist #16

Closed ajvincent closed 1 year ago

ajvincent commented 1 year ago

Section 10.5.14 in spec.emu, step 4.c requires the options dictionary to have a signal property. It's thus not really an option. We need to allow options.signal to be undefined / not present.

This is intended for forwards compatibility, to allow other features to be added later.

ajvincent commented 1 year ago

https://github.com/ajvincent/proposal-mass-proxy-revocation/pull/17

Branch name: options-signal-optional . I figured we could do both changes on the same branch.

This was my fault. I forgot the "?" character, and the spec matches the TypeScript definition. Sorry about that!