the zx_handle_t type

35 views
Skip to first unread message

George Kulakowski

unread,
Oct 3, 2017, 7:24:59 PM10/3/17
to rust-f...@fuchsia.com
Very shortly the zx_handle_t type is being changed from signed to unsigned. Rust publishes its own definition of this type in https://fuchsia.googlesource.com/garnet/+/master/public/rust/crates/fuchsia-zircon/fuchsia-zircon-sys/src/lib.rs#12

This is at least in theory an ABI breaking change. Do I have to do anything re crates publication etc.?

George Kulakowski

unread,
Oct 3, 2017, 7:26:01 PM10/3/17
to rust-f...@fuchsia.com
Relatedly, I'm noticing that the definition of zx_futex_t is wrong. It should be 32 bits, not isize-sized.

Taylor Cramer

unread,
Oct 3, 2017, 7:31:52 PM10/3/17
to George Kulakowski, rust-f...@fuchsia.com
Since nothing is depending on the crates.io version at the moment (I think?), I believe you should just be able to make the change in place. When you do, can you bump the version numbers of fuchsia-zircon-sys and fuchsia-zircon to 0.3?

--
You received this message because you are subscribed to the Google Groups "rust-fuchsia" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rust-fuchsia+unsubscribe@fuchsia.com.
To post to this group, send email to rust-f...@fuchsia.com.
To view this discussion on the web visit https://groups.google.com/a/fuchsia.com/d/msgid/rust-fuchsia/CAKP51mb4CfMcSN17sNN4SekFM3c5n5qNxM2wu3JU6y-RyUtbaQ%40mail.gmail.com.

George Kulakowski

unread,
Oct 3, 2017, 7:41:21 PM10/3/17
to Taylor Cramer, rust-f...@fuchsia.com

James Tucker

unread,
Oct 4, 2017, 4:07:11 PM10/4/17
to George Kulakowski, Taylor Cramer, rust-fuchsia
We also have to update rust libstd, I am preparing a patch for that right now.

George Kulakowski

unread,
Oct 4, 2017, 4:11:33 PM10/4/17
to James Tucker, Taylor Cramer, rust-fuchsia
I already did.

George Kulakowski

unread,
Oct 4, 2017, 4:12:30 PM10/4/17
to James Tucker, Taylor Cramer, rust-fuchsia
Oh, I misread. Carry on!

James Tucker

unread,
Oct 4, 2017, 6:13:16 PM10/4/17
to George Kulakowski, Taylor Cramer, rust-fuchsia
Reply all
Reply to author
Forward
0 new messages