Lines of code count not available
We tend to initialize the local variables as a standard practice.
These don't need to be initialized to zero.
Done
Done.
No. It is permanent, I would like to know the value for the homing.
The else in this trinary looks wrong - doesn't round up to multiple of 8.
This if should have an else where we zero .valveOpsStartTime (in case it briefly got set but then failed this if before timeout).
Declarations should be at top of scope.
Is this temporary?
Updated.
Yes, we need to
Deleted.
Does OperationModes.c need to include TxParams.h? TxParams is not a mode.
I think we should initialize pre-treatment mode here now that we have a pre-treatment mode.
Should we request pre-treatment mode here instead? How do we get to pre-treatment mode?
Yes, I deleted.
Removed
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.