Skip to content

fix(lib/webidl2): disallow null on constants#284

Merged
saschanaz merged 3 commits into
gh-pagesfrom
no-const-null
Mar 15, 2019
Merged

fix(lib/webidl2): disallow null on constants#284
saschanaz merged 3 commits into
gh-pagesfrom
no-const-null

Conversation

@saschanaz
Copy link
Copy Markdown
Member

Following whatwg/webidl#686.

@saschanaz saschanaz requested a review from marcoscaceres March 15, 2019 12:26
Comment thread test/invalid/baseline/const-nullable.txt Outdated
Comment thread lib/webidl2.js
}
idlType = Object.assign({ type: "const-type" }, EMPTY_IDLTYPE, idlType);
if (probe("?")) {
error("Unexpected nullable constant type");
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😍

@saschanaz saschanaz merged commit 927328d into gh-pages Mar 15, 2019
@saschanaz saschanaz deleted the no-const-null branch March 15, 2019 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants