Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

IDL processing pipeline is not recognizing Partial(Dictionary|Namespace) #35

Open
mdittmer opened this issue Jan 19, 2017 · 1 comment
Open

IDL processing pipeline is not recognizing Partial(Dictionary|Namespace) #35

mdittmer opened this issue Jan 19, 2017 · 1 comment
Assignees
@mdittmer
Labels
bug

Comments

@mdittmer
Copy link
Owner

Non-Interface partials do not have custom constructors, so

web-apis/lib/idl/IDLProcessMemo.es6.js

Line 118 in dc46875

return `${parse.constructor.name}:${key}`;
does not categorize partials properly.
@mdittmer mdittmer added the bug label Jan 19, 2017
@mdittmer mdittmer self-assigned this Jan 19, 2017
@mdittmer
Copy link
Owner Author

@LoonyBean FYI. This appears to be related to some false positives you've been seeing.

This was referenced Jan 19, 2017
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mdittmer mdittmer

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mdittmer