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.

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

Update syn related dependencies to 1.0 #379

Merged
merged 1 commit into from Aug 19, 2019
Merged

Conversation

Eijebong
Copy link
Contributor

No description provided.

Cargo.lock Show resolved Hide resolved
@emilio
Copy link
Collaborator

emilio commented Aug 16, 2019

@dtolnay is serde-rs/serde#1591 expected to land soon-ish? If so we may as well wait for that.

@dtolnay
Copy link

dtolnay commented Aug 16, 2019

Done

@Eijebong Eijebong force-pushed the syn-1.0 branch 2 times, most recently from 4d9a486 to 2875929 Compare August 16, 2019 19:00
@emilio
Copy link
Collaborator

emilio commented Aug 17, 2019

Thanks @dtolnay!

@Eijebong travis is complaining, mind taking a look?

@Eijebong
Copy link
Contributor Author

Travis is always complaining 😄
Also looks like I failed at rebasing somehow. I'll fix it later today or tomorrow

@Eijebong
Copy link
Contributor Author

There, should be fixed

src/bindgen/ir/cfg.rs Outdated Show resolved Hide resolved
src/bindgen/ir/constant.rs Outdated Show resolved Hide resolved
src/bindgen/parser.rs Outdated Show resolved Hide resolved
src/bindgen/utilities.rs Outdated Show resolved Hide resolved
syn::Item::TraitAlias(ref item) => (|$i: &syn::ItemTraitAlias| $a)(item),
syn::Item::Verbatim(_) => (|| $b)(),
_ => panic!("Unhandled syn::Item: {:?}", $s),
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'd prefer to keep handling the others.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

@emilio emilio left a comment

Choose a reason for hiding this comment

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

r=me with a rustfmt pass (travis is mad because of it) and this warning fixed:

warning: unused import: `std::mem`
 --> src/bindgen/ir/constant.rs:8:5

Thanks!

@emilio
Copy link
Collaborator

emilio commented Aug 18, 2019

Oh, also, mind not doing the Cargo.toml version bump? I hope to merge #377 soonish, though I may as well do two releases so no big deal I guess.

@Eijebong
Copy link
Contributor Author

Sure, I'll remove it

@Eijebong Eijebong changed the title Update syn related dependencies to 1.0 and bump version Update syn related dependencies to 1.0 Aug 18, 2019
Copy link
Collaborator

@emilio emilio left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@emilio emilio merged commit 25e51f6 into mozilla:master Aug 19, 2019
emilio added a commit that referenced this pull request Aug 25, 2019
 * Various improvements to comment output. #370 / #375.
 * Fixed expand when ran from build.rs. #371
 * More debugging output for expansion. #383
 * New option to add a default private constructor in C++ tagged enums. #377
 * Syn and related dependencies updated to 1.0. #379
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Sep 6, 2019
Changelog:
## 0.9.1
     * Various improvements to comment output. mozilla/cbindgen#370 / mozilla/cbindgen#375.
     * Fixed expand when ran from build.rs. mozilla/cbindgen#371
     * More debugging output for expansion. mozilla/cbindgen#383
     * New option to add a default private constructor in C++ tagged enums. mozilla/cbindgen#377
     * Syn and related dependencies updated to 1.0. mozilla/cbindgen#379
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.

None yet

3 participants