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

Make size/position types generic over pixel type #1277

Merged
merged 12 commits into from Nov 14, 2019

Conversation

Osspial
Copy link
Contributor

@Osspial Osspial commented Nov 14, 2019

  • Tested on all platforms changed
  • Compilation warnings were addressed
  • cargo fmt has been run on this branch
  • cargo doc builds successfully
  • 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
  • Updated feature matrix, if new features were added or implemented

Implements the changes suggested in #1135 (comment). I haven't gotten around to updating the type signatures on non-Windows platforms but I'll go through the CI logs after they fail and update them where necessary.

@Osspial Osspial changed the title Dpi generics Make size/position types generic over pixel type Nov 14, 2019
@goddessfreya goddessfreya added C - waiting on author Waiting for a response or another PR S - api Design and usability C - in progress Implementation is proceeding smoothly and removed C - waiting on author Waiting for a response or another PR labels Nov 14, 2019
src/dpi.rs Outdated Show resolved Hide resolved
@Osspial Osspial merged commit 7043703 into rust-windowing:dpi-overhaul Nov 14, 2019
@tangmi tangmi mentioned this pull request Dec 23, 2019
7 tasks
Osspial added a commit that referenced this pull request Jan 4, 2020
* Begin implementing DPI generics

* Fix multithreaded example

* Format

* Fix serde test

* hopefully fix most of the errors

* Fix dpi module errors

* More error fixings

* Format

* fix macos errors

* Another error pass

* Replace bad type signatures

* more fixins
Osspial added a commit that referenced this pull request Jan 4, 2020
* Begin implementing DPI generics

* Fix multithreaded example

* Format

* Fix serde test

* hopefully fix most of the errors

* Fix dpi module errors

* More error fixings

* Format

* fix macos errors

* Another error pass

* Replace bad type signatures

* more fixins
Osspial added a commit that referenced this pull request Jan 4, 2020
* Begin implementing DPI generics

* Fix multithreaded example

* Format

* Fix serde test

* hopefully fix most of the errors

* Fix dpi module errors

* More error fixings

* Format

* fix macos errors

* Another error pass

* Replace bad type signatures

* more fixins
Osspial added a commit that referenced this pull request Jan 5, 2020
* Begin implementing DPI generics

* Fix multithreaded example

* Format

* Fix serde test

* hopefully fix most of the errors

* Fix dpi module errors

* More error fixings

* Format

* fix macos errors

* Another error pass

* Replace bad type signatures

* more fixins
Osspial added a commit that referenced this pull request Jan 5, 2020
* Begin implementing DPI generics

* Fix multithreaded example

* Format

* Fix serde test

* hopefully fix most of the errors

* Fix dpi module errors

* More error fixings

* Format

* fix macos errors

* Another error pass

* Replace bad type signatures

* more fixins
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C - in progress Implementation is proceeding smoothly S - api Design and usability
Development

Successfully merging this pull request may close these issues.

None yet

2 participants