Stratus51/rust_dash7_alp

Put all operands behind feature flags

Closed this issue · 1 comments

Make all operands parsing optional via feature configuration.

While by default, ALP action builders will be removed from the release build if unused, the parsers will all be included.
This could be a hindrance on memory limited systems, where binary size needs to be as small as possible and where the devices will always have to process the same subset of ALP actions.
Each unselected ALP action would trigger an Error result (unknown operand) from the parser.

This is way too heavy of a feature for me to support it.
It most notably would require "per feature set" test sets to verify that any feature flag combination results in a coherent library.