casanovg / timonel

ATtiny85/45/25 I2C bootloader
MIT License
111 stars 18 forks source link

Use | instead of + when building a flash address. #26

Closed prandeamus closed 3 years ago

prandeamus commented 3 years ago

Use | instead of + when building a 16-bit address from 2 bytes in the message buffer. This is safe, since addition cannot overflow, and consistent with other places in the code where it was already done using | Each change saves 4 bytes (tml-t85-test-comm is 16 bytes shorter) The commit does not rebuild the distribution .hex files. Tested informally with READEEPR and WRITEEPR only

casanovg commented 3 years ago

Sounds good, thanks! I'll test it asap.