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

Optimisations #1631

Merged
merged 1 commit into from Aug 25, 2018

Conversation

Projects
None yet
2 participants
@GHolli
Copy link
Contributor

GHolli commented Aug 5, 2018

  • Save RAM, move constants to flash (CharTableTM1637, CharTableMAX7219)
  • Remove unnecessary parameter passing (the interface pins do not change during operation for one instance)
Gerald Hollweger
Optimisations
- Save RAM, move constants to flash (CharTableTM1637, CharTableMAX7219)
- Remove unnecessary parameter passing (the interface pins do not change during operation for one instance)
@TD-er

This comment has been minimized.

Copy link
Member

TD-er commented Aug 25, 2018

This code looks OK to me, I will merge it.

@TD-er TD-er merged commit ca5f281 into letscontrolit:mega Aug 25, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
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.