t3knomanzer / maxmix-software

Maxmix volume mixer software repository
https://maxmixproject.com
Apache License 2.0
126 stars 31 forks source link

First pass code cleanup converting Display.ino to Display.cpp #171

Closed XScorpion2 closed 4 years ago

XScorpion2 commented 4 years ago

Issues

Description

Converted Display.ino to Display.cpp and performed a pass on code cleanup in Display.cpp. In addition, remove the default adafruit logo bitmap. Compile output summary for firmware size from platformio:

Before: RAM: [====== ] 59.5% (used 1219 bytes from 2048 bytes) Flash: [========= ] 86.5% (used 26562 bytes from 30720 bytes)

After: RAM: [====== ] 59.2% (used 1213 bytes from 2048 bytes) Flash: [======== ] 77.9% (used 23932 bytes from 30720 bytes)

Types of changes

Put an x in the boxes that apply

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...