MsgDefs.h

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
If a retry goes wrong will you trigger an alarm?

If a retry goes wrong will you trigger an alarm?

Add cummulativeRinsebackVolume_mL, fluidType, targetRinsebackVolumePlusAdditional_mL to outputs

Add cummulativeRinsebackVolume_mL, fluidType, targetRinsebackVolumePlusAdditional_mL to outputs

break this line

break this line

Add @Alarms

Add @Alarms

Add @Alarms

Add @Alarms

Any fault alarm here?

Any fault alarm here?

fluidType is an input

fluidType is an input

break this line

break this line

Generally we put all the static functions after all the public functions. Can we reorganize the file as : Public functions Static functions Public Test functions

Generally we put all the static functions after all the public functions.
Can we reorganize the file as :
Public functions
Static functions
Public Test functions

Initialization is missing in the init function for variables from line 72 to 77.

Initialization is missing in the init function for variables from line 72 to 77.

Initialize in init.

Initialize in init.

Initialize in init. Check SyringPump.c for reference.

Initialize in init. Check SyringPump.c for reference.

Have these variables been initialized correctly?

Have these variables been initialized correctly?

please add a TODO or remove commented code.

please add a TODO or remove commented code.

please add a TODO or remove commented code.

please add a TODO or remove commented code.

add resumeTreatmentAlarmResponseRequest to output

add resumeTreatmentAlarmResponseRequest to output

Update input - output list.

Update input - output list.

Add presUFRateMlMin to outputs

Add presUFRateMlMin to outputs

recirculateToBloodPrimeRequest is output.

recirculateToBloodPrimeRequest is output.

Add initialization for rinsebackToRecircRequest and recirculateToBloodPrimeRequest

Add initialization for rinsebackToRecircRequest and recirculateToBloodPrimeRequest

So then no need to initialize it here too. Remove " = FALSE".

So then no need to initialize it here too. Remove " = FALSE".

I agree but it is already in the init function.

I agree but it is already in the init function.

Remove extra blank line.

Remove extra blank line.

We generally don't add { } to switch cases unless they require local variable declarations.

We generally don't add { } to switch cases unless they require local variable declarations.

If we're not including a timestamp, can we just remove this line of code?

If we're not including a timestamp, can we just remove this line of code?

Why commented out?

Why commented out?

Swap order of == for these 2 if statements - put constant on left side.

Swap order of == for these 2 if statements - put constant on left side.

Why?

Why?