kanflo / opendps

Give your DPS5005 the upgrade it deserves
MIT License
897 stars 124 forks source link

Rework protocol handling #176

Closed Xenoamor closed 5 years ago

Xenoamor commented 5 years ago

I've removed the macros from the protocol data packing/unpacking. These were getting inlined everywhere and ballooning the ROM usage.

This drops the previous 6244 bytes from the protocol handler to 1496 bytes

I've tested this with the emulator but it would probably be worth another once over before a merge

kanflo commented 5 years ago

Nice! Tested it on a tty and see no drawbacks.