-
Notifications
You must be signed in to change notification settings - Fork 606
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 TrapezoidProfile API #5457
Refactor TrapezoidProfile API #5457
Conversation
Is there a way to make this less immediately breaking from a user perspective, eg with deprecated functions? Or is a hard break required? |
I think we can have separate constructor and calculate() overloads, though using the new ctor overload with the old calculate() will have undefined behavior. |
How about a boolean variable, |
That'll work. |
Do we still need that "breaking" tag if we only deprecated functions? |
77e3e16
to
bf34bd9
Compare
ctor args were moved to
calculate
, all examples were updated, command and subsystem updated as well, and anything else that made it not compile was updated. Fixes #5453.