leahi-fwcommon

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Again, I don't like these oriented this way. Please change to #ifndef DISABLE_ALARM_4

Again, I don't like these oriented this way. Please change to #ifndef DISABLE_ALARM_4

If BC switching period is Zero, I would expect the Qd from TD to also be zero, right? But what if it's not? What if Qd from TD is 600 and switching period is zero? You are going to run the pump pre...

If BC switching period is Zero, I would expect the Qd from TD to also be zero, right? But what if it's not? What if Qd from TD is 600 and switching period is zero? You are going to run the pump pretty fast with no BC switching in that case.

We shouldn't require something to be defined to do the right thing. We should require something to be defined to do the wrong thing (i.e. this should say #ifndef _BAD_MSG_CRC_ALARM_DISABLED_) And t...

We shouldn't require something to be defined to do the right thing. We should require something to be defined to do the wrong thing (i.e. this should say #ifndef _BAD_MSG_CRC_ALARM_DISABLED_)
And then, we don't need a TODO. These #defines are only possible in DEBUG builds. A RELEASE build will never allow thing build flags to disable things.

Remove start state.

Remove start state.

Updated

Updated

updated

updated

Yes, removed

Yes, removed

Removed

Removed

yes, removed

yes, removed

Changed

Changed

Removed

Removed

Removed

Removed

Removed

Removed

Added

Added

Added

Added

Updated

Updated

Updated

Updated

Added

Added

Updated

Updated

Updated

Updated

no need to respond.

no need to respond.

don't need response to a response.

don't need response to a response.

don't need this if statement either and don't need what's inside the if.

don't need this if statement either and don't need what's inside the if.

Don't need this if statement or flag.

Don't need this if statement or flag.

use separate time than stability (even if same right now).

use separate time than stability (even if same right now).

Move to transition function and remove this if statement.

Move to transition function and remove this if statement.

do 2 data with the p1 and p2 pressures.

do 2 data with the p1 and p2 pressures.