- From: Timothy Gu <notifications@github.com>
- Date: Wed, 25 Mar 2020 12:38:57 -0700
- To: heycam/webidl <webidl@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <heycam/webidl/pull/859/review/381476304@github.com>
@TimothyGu commented on this pull request. > @@ -4779,19 +4779,26 @@ from and all of |E|’s [=inherited dictionaries=]. [=Dictionary members=] can be specified as <dfn id="required-dictionary-member" export for="dictionary member">required</dfn>, meaning that -converting a language-specific value to a dictionary requires providing a value for that member. -They can also be specified as having a +converting a language-specific value to a dictionary requires providing a value for that member. Any +dictionary member that is not [=dictionary member/required=] is +<dfn export for="dictionary member">optional</dfn>. + +Note that specifying [=dictionary members=] as [=dictionary member/required=] only has +an observable effect when converting other representations of dictionaries (like an ECMAScript value +supplied as an argument to an [=operation=]) to an IDL dictionary. Specification authors should +leave the members [=dictionary member/optional=] in all other cases, including when a dictionary +type is used soley as the [=return type=] of [=operations=]. ```suggestion type is used solely as the [=return type=] of [=operations=]. ``` -- 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/859#pullrequestreview-381476304
Received on Wednesday, 25 March 2020 19:39:10 UTC