Skip to content

Fix #9511. Export webidl enums using "pub use". #10152

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.

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

Merged
merged 1 commit into from
Mar 24, 2016

Conversation

jdm
Copy link
Member

@jdm jdm commented Mar 23, 2016

Change typedef codegen to export the underlying enum itself, rather
than an alias. Works around rust-lang/rust#31355

Rebase of #9691.


This change is Reviewable

Change typedef codegen to export the underlying enum itself, rather
than an alias. Works around rust-lang/rust#31355
@highfive highfive added the S-awaiting-review There is new code that needs to be reviewed. label Mar 23, 2016
@highfive
Copy link

warning Warning warning

  • These commits modify script code, but no tests are modified. Please consider adding a test!

@jdm jdm mentioned this pull request Mar 23, 2016
@jdm jdm assigned nox and unassigned glennw Mar 23, 2016
@jdm
Copy link
Member Author

jdm commented Mar 23, 2016

@bors-servo: r=nox

@bors-servo
Copy link
Contributor

📌 Commit 2477f6a has been approved by nox

@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-awaiting-review There is new code that needs to be reviewed. labels Mar 23, 2016
@bors-servo
Copy link
Contributor

⌛ Testing commit 2477f6a with merge edcf24f...

bors-servo pushed a commit that referenced this pull request Mar 24, 2016
Fix #9511. Export webidl enums using "pub use".

Change typedef codegen to export the underlying enum itself, rather
than an alias. Works around rust-lang/rust#31355

Rebase of #9691.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10152)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - android, arm32, gonk, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, status-appveyor

@bors-servo bors-servo merged commit 2477f6a into servo:master Mar 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants