-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
stylo bindings update #14492
stylo bindings update #14492
Conversation
This allows us to incrementally remove both bindgen hacks and stylo hacks.
Heads up! This PR modifies the following files:
|
@@ -106,6 +106,7 @@ | |||
"AnonymousContent", | |||
"AudioContext", | |||
"CapturingContentInfo", | |||
"ConsumeStyleBehavior", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These should be noops, right? They're in the mozilla namespace above. Anyway, happy to clean it up afterwards.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some reason without adding these namespace-less names here, the types didn't get output.
extern "C" { | ||
#[link_name = "_ZN7mozilla20ThreadSafeAutoRefCnt12isThreadSafeE"] | ||
pub static ThreadSafeAutoRefCnt_isThreadSafe: bool; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, you're definitely using clang 3.8, but if you're not building with --features llvm_stable
, so I expect stuff to break, like this, and potentially the JSWhyMagic
thing.
I'm actually surprised it works so decently to be fair, but that's good :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am using --features llvm_stable
. Does it make sense to make regen.sh require to a newer clang?
@bors-servo r+ |
📌 Commit 3d881f3 has been approved by |
stylo bindings update <!-- Please describe your changes on the following line: --> For the latest m-c stylo sync, on top of #14472. r? @emilio <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/14492) <!-- Reviewable:end -->
☀️ Test successful - arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css, mac-rel-wpt1, mac-rel-wpt2, windows-dev |
On Thu, Dec 08, 2016 at 11:51:16AM -0800, Cameron McCormack wrote:
I am using `--features llvm_stable`. Does it make sense to make regen.sh require to a newer clang?
I guess eventually, yes, but for now it's fine.
|
For the latest m-c stylo sync, on top of #14472.
r? @emilio
This change is