Re: [heycam/webidl] Update method creation for operations (#155)

> @@ -12005,12 +11827,51 @@ interface
>                      Try running the following steps:
>                      <ol>
>                        <li>
> +                        Let <var>O</var> be <span class='esvalue'>null</span>.
> +                      </li>
> +
> +                      <li>
> +                        If <var>target</var> is not a <a class='dfnref'

It's not obvious at this point that `target` can only be a namespace or interface.  So I think it would be clearer to make this check "If `target` is an interface".  And perhaps assert at the top of the algorithm that it's one or the other.

But also, calling this thing `target` is a bit weird.  It's more like "source" or something....

-- 
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/pull/155/files/5f3f6f66bb2d8cfbf1c7197e64f2b54f9cdd2845#r76880306