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

Refactor RC adjustments to remove some code duplication #3410

Merged
merged 1 commit into from Jun 24, 2018

Conversation

Projects
None yet
3 participants
@fiam
Copy link
Member

commented Jun 21, 2018

Saves 544 bytes of flash in F3
Also, removed some FIXME comments since they are no longer relevant

@fiam fiam added this to the 2.0 milestone Jun 21, 2018

@fiam fiam force-pushed the agh_rc_adjustments_refactor branch from f6d30b5 to 5989937 Jun 21, 2018

@fiam fiam changed the title Refactor RC adjustments to use a common function for uint8_t settings Refactor RC adjustments to remove some code duplication Jun 21, 2018

Refactor RC adjustments to remove some code duplication
Saves 560 bytes of flash in F3
Also, removed some FIXME comments since they are no longer relevant

@fiam fiam force-pushed the agh_rc_adjustments_refactor branch from 5989937 to 7e4c508 Jun 21, 2018

@shellixyz

This comment has been minimized.

Copy link
Collaborator

commented Jun 21, 2018

Very nice I thought about doing something like that but didn't do it.

@junwoo091400

This comment has been minimized.

Copy link
Contributor

commented Jun 21, 2018

Nice!

@fiam fiam merged commit 926c6f8 into development Jun 24, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@fiam fiam deleted the agh_rc_adjustments_refactor branch Jun 24, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.