-
Notifications
You must be signed in to change notification settings - Fork 157
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
refactor!: Use enum class for NavigationDirection #1206
refactor!: Use enum class for NavigationDirection #1206
Conversation
23ecd4d
to
3d3ff23
Compare
Codecov Report
@@ Coverage Diff @@
## main #1206 +/- ##
==========================================
+ Coverage 47.85% 47.94% +0.09%
==========================================
Files 372 373 +1
Lines 19439 19495 +56
Branches 9148 9152 +4
==========================================
+ Hits 9302 9347 +45
- Misses 3806 3817 +11
Partials 6331 6331
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
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.
Massive support for going to type-safe enums here, just a few comments where I think find-replace might have gone wrong.
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.
There have been some replace errors that can/should be fixed.
I updated the branch and fixed the bad replaces. |
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.
Looks good to me. Can't close the conversations I opened for some reason, though.
Stephen approved the more recent changes.
This change is motivated by arithmetic between non-class enums and other types that are not the same enum is deprecated in C++20. I overload the operators we use these enums in to support the same use cases.
I think this is breaking since it's part of the public propagator interface.
BREAKING CHANGE: The
NavigationDirection
enum changes toenum class
. The values are renamedNavigationDiretcion::forward
->NavigationDirection::Forward
andNavigationDirection::backward
->NavigationDirection::Backward
.