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

Apply static to Planner class #3925

Merged
merged 2 commits into from
Jun 3, 2016

Conversation

thinkyhead
Copy link
Member

Make most data and methods in the Planner class static for size and performance.

This change actually does increase the binary size by about 12 bytes,
but how does it affect performance?
@thinkyhead thinkyhead added this to the 1.1.0 milestone Jun 1, 2016
@thinkyhead thinkyhead merged commit 1941855 into MarlinFirmware:RCBugFix Jun 3, 2016
@thinkyhead thinkyhead deleted the rc_statics_planner branch June 3, 2016 00:14
@thinkyhead thinkyhead mentioned this pull request Jul 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant