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.

Sign up for GitHub

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

Jump to bottom

Replace uses of `RangeInclusive<f32>` with `emath::Rangef` #3221

Merged

emilk merged 2 commits into master from emilk/range

Aug 10, 2023

Owner

emilk commented Aug 10, 2023 •

edited

Loading

RangeInclusive is unergonomic in not implementing Clone, and having weirdly named .start(() and .end() returning references.

Rangef is an ergonomic replacement for RangeInclusive<f32>

emilk added the emath label


          Replace uses of RangeInclusive<f32> with emath::Rangef

bd9cb63

emilk force-pushed the emilk/range branch from fed4237 to bd9cb63 Compare

August 10, 2023 08:35

emilk added the egui label


          Fix doc-test

emilk merged commit 8cdffc4 into master

emilk deleted the emilk/range branch

August 10, 2023 11:07

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels