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

wgl: add support for pixel buffer surfaces #1701

Merged
merged 10 commits into from
Sep 22, 2024

Conversation

SK83RJOSH
Copy link
Contributor

  • Tested on all platforms changed
  • Added an entry to CHANGELOG.md if knowledge of this change could be valuable to users
  • Updated documentation to reflect any user-facing changes, including notes of platform-specific behavior
  • Created or updated an example program if it would help users understand this functionality

I didn't create an example, since this is standard functionality that was previously unexposed.

glutin/src/api/wgl/surface.rs Outdated Show resolved Hide resolved
glutin/src/api/wgl/surface.rs Outdated Show resolved Hide resolved
glutin/src/api/wgl/surface.rs Outdated Show resolved Hide resolved
glutin/src/api/wgl/surface.rs Outdated Show resolved Hide resolved
glutin/src/api/wgl/surface.rs Outdated Show resolved Hide resolved
glutin/src/api/wgl/surface.rs Outdated Show resolved Hide resolved
glutin/src/api/wgl/surface.rs Outdated Show resolved Hide resolved
glutin/src/api/wgl/surface.rs Outdated Show resolved Hide resolved
glutin/src/api/wgl/surface.rs Outdated Show resolved Hide resolved
glutin/src/api/wgl/surface.rs Outdated Show resolved Hide resolved
@SK83RJOSH SK83RJOSH requested a review from kchibisov September 22, 2024 04:36
glutin/src/api/wgl/surface.rs Outdated Show resolved Hide resolved
glutin/src/api/wgl/surface.rs Outdated Show resolved Hide resolved
glutin/src/api/wgl/context.rs Outdated Show resolved Hide resolved
@SK83RJOSH SK83RJOSH requested a review from kchibisov September 22, 2024 12:42
Copy link
Member

@kchibisov kchibisov left a comment

Choose a reason for hiding this comment

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

Could you update doc here https://github.com/rust-windowing/glutin/blob/master/glutin/src/surface.rs#L522 with the pbuffer that could be present there?

Other than that looks good.

@SK83RJOSH
Copy link
Contributor Author

Done! Thank you for the thorough review, and my apologies for the sorry state of this initially. Have a great rest of your weekend. 🙂

glutin/src/surface.rs Outdated Show resolved Hide resolved
glutin/src/surface.rs Outdated Show resolved Hide resolved
@kchibisov kchibisov merged commit 23e4ca1 into rust-windowing:master Sep 22, 2024
43 checks passed
@SK83RJOSH SK83RJOSH deleted the wgl-pbuffers branch September 22, 2024 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants