adafruit / Wippersnapper_Protobuf

Protocol buffer messages for Adafruit.io WipperSnapper Beta
MIT License
0 stars 2 forks source link

API Cleanup, Round #1 #56

Closed brentru closed 3 years ago

brentru commented 3 years ago

Breaking PR

Removed the following APIs which are not currently used:

Removed SignalReq/Response message which was not used, we're currently using I2CRequest/I2CResponse instead in staging I2C work.

Add FT_IGNORE (https://github.com/adafruit/Wippersnapper_Protobuf/issues/44) tags to deprecated signal fields which are not used in C code so the fields are not allocated in the signal message's MAX_LENGTH calculation.

Results: Savings of 53KB

brentru commented 3 years ago

Net FSM merged in, going to merge this as well before a release.