Re: [heycam/webidl] Make the class strings for interface prototype objects and platform objects match. (#577)

TimothyGu commented on this pull request.



> @@ -1076,6 +1076,20 @@ with a [{{NoInterfaceObject}}] [=extended attribute=],
 and [=callback interfaces=] which declare [=constants=]
 must be annotated with an [{{Exposed}}] [=extended attribute=].
 
+<div algorithm>
+
+The <dfn>qualified name</dfn> of an [=interface=] |I| is defined as follows:
+
+1.  Let |interface| be the [=identifier=] of |I|.

I'd encourage _not_ using any one-letter variables any more. How about "Let *identifier* be the identifier of *interface*"?

> @@ -1076,6 +1076,20 @@ with a [{{NoInterfaceObject}}] [=extended attribute=],
 and [=callback interfaces=] which declare [=constants=]
 must be annotated with an [{{Exposed}}] [=extended attribute=].
 
+<div algorithm>
+
+The <dfn>qualified name</dfn> of an [=interface=] |I| is defined as follows:
+
+1.  Let |interface| be the [=identifier=] of |I|.
+1.  If |I| has a [{{LegacyNamespace}}] [=extended attribute=], then:
+    1.  Let |namespace| be the identifier argument of the [{{LegacyNamespace}}]
+        [=extended attribute=].
+    1.  Return the [=concatenation=] of « |namespace|, |interface| » with
+        separator ".".

"`.`"

-- 
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/577#pullrequestreview-144062222