-
Notifications
You must be signed in to change notification settings - Fork 50
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
v0.6.0 #132
v0.6.0 #132
Conversation
We'd likely need an update downstream, so we can sort of migrate the ecosystem, or at least offer the |
Do you mean something like rust-windowing/winit#2943? I've made similar PRs for Glutin and Softbuffer. |
Yes, but I also meant to say, that we should offer raw-window-handle based on the features in winit. I know that we have patches around, but the thing is, that they still need review and work. And I also know that we can expose |
But I wasn't paying close attention to what actually will be included and what the scope will be in the next release, so it's up to @Lokathor , I guess. |
Honestly I haven't looked closely at any of this in a while. I was mostly letting @notgull take the lead, and just providing a point of occasional double checking. Just offhand, I'm not sure we need to update the outside world too much before we do a release. It seems okay to release 0.6 when it's ready and then if This might be too simple a view, so we could do stronger coordination with other crates if you think it's necessary @kchibisov . |
11960d9
to
7af597e
Compare
I don't think we need strong coordination. I'm just saying that if it's time to release, we should probably do it. Winit will support both I've came here only because the winit release will be relatively soon and I'd like to get reviewing patches wrt raw-window-handle. |
I've updated the CHANGELOG here to describe the set of changes included in this new version more accurately. Before merging and releasing I think we should ping stakeholders in
Like @Lokathor said above, I don't think ACKs from everyone are necessary, but we should wait at least a week to let people know that this is coming. |
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.
Thanks for pinging @notgull!
LGTM!
I can update |
I'll continue to wait for others to have a look and merge/release next week if objections haven't come up since then. |
@Lokathor Its been more than a week, I think we're good to release. |
Roger. I'll get to this after work. |
I like the new lifetime since our |
NonNull
where appropriate, to avoid null pointer dereferences.empty
methods tonew
, and take parameters in most of those, to better match normal Rust semantics.HasRaw(Display/Window)Handle::raw_(display/window)_handle
returns a result indicating if fetching the window handle failed (Make raw_*_handle return a result #122).Active/ActiveHandle
types from the public API (Remove Active from the API #126).AppKitWindowHandle::ns_window
andUiKitWindowHandle::ui_window
since they can be retrieved from the view (Remove window handles from Apple handles #129).Copy
derive fromRawWindowHandle
andWindowHandle
(RemoveCopy
forRawWindowHandle
/WindowHandle
#140).PartialEq
,Eq
andHash
forWindowHandle
too. (Make sure traits are uniform across WindowHandle and DisplayHandle #128)&mut T where T: <trait>
. (Incorrect changelog PR number #130)wasm-bindgen
v0.2. They are locked behind thewasm-bindgen-0-2
feature. (Add wasm-bindgen v0.2 handles for web_sys::HtmlCanvasElement and web_sys::OffscreenCanvas #134)