Re: [heycam/webidl] Can [Replaceable] be used on namespace members? (#572)

Hmm.  So namespaces only allow readonly attributes.  But `[Replaceable]` could in theory still make sense.

That said, in practice it doesn't make sense to me; `[Replaceable]` is basically a backwards-compat hack, and there's no need for that on namespaces....

There is definitely a  spec bug here, though.  https://heycam.github.io/webidl/#define-the-attributes can have a namespace as _definition_ and in step 3 does https://heycam.github.io/webidl/#dfn-attribute-setter with that namespace as _target_ if that happens, but https://heycam.github.io/webidl/#dfn-attribute-setter assumes that _target_ is not a namespace.  We need to fix that.  Probably by just having step 1 of https://heycam.github.io/webidl/#dfn-attribute-setter return **undefined** if _target_ is a namespace.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/heycam/webidl/issues/572#issuecomment-405685553

Received on Tuesday, 17 July 2018 18:40:41 UTC