Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Give ejector retract command and zero timer counter here too.

Give ejector retract command and zero timer counter here too.

Not needed if you do this on way out of prior state.

Not needed if you do this on way out of prior state.

Start BP and zero timer counter here.

Start BP and zero timer counter here.

Not necessary if fault property of alarm is TRUE.

Not necessary if fault property of alarm is TRUE.

Not necessary if you do this on way out of prior state.

Not necessary if you do this on way out of prior state.

Remove ejector homed stuff. Not needed.

Remove ejector homed stuff. Not needed.

Give eject cmd and zero timer counter here.

Give eject cmd and zero timer counter here.

give BP home command and zero timer counter here.

give BP home command and zero timer counter here.

Not necessary. Just make alarm have fault property = TRUE.

Not necessary. Just make alarm have fault property = TRUE.

Again, ejector should have been retracted way back in pre-treatment mode. We shouldn't need to do it again.

Again, ejector should have been retracted way back in pre-treatment mode. We shouldn't need to do it again.

Shouldn't need this if statement. If we timeout, we want to stop the pump and alarm.

Shouldn't need this if statement. If we timeout, we want to stop the pump and alarm.

Better to home BP on your way out of previous state so you don't have to use a cmd issued flag. Ejector should already be retracted, so I see no reason to home the ejector here. You should also zer...

Better to home BP on your way out of previous state so you don't have to use a cmd issued flag.
Ejector should already be retracted, so I see no reason to home the ejector here.
You should also zero the autoEjectTimerCounter too.

Not necessary to request fault mode. Just set alarm fault property to TRUE.

Not necessary to request fault mode. Just set alarm fault property to TRUE.

This is way too late. UI s/w expects response immediately (so always send it in message handler function.)

This is way too late. UI s/w expects response immediately (so always send it in message handler function.)

rejectionReason should be set to none (zero) here too.

rejectionReason should be set to none (zero) here too.

You should send response regardless of success - so I think this should be moved to the very end of the function, outside of any if statements so that it always responds. You should set accept flag...

You should send response regardless of success - so I think this should be moved to the very end of the function, outside of any if statements so that it always responds. You should set accept flag and reject reason codes in these if statements so the response that gets sent will have the correct payload.

Declare locals at top of scope (so before memcpy). Also, when assigning to a BOOL, be explicit (i.e. TRUE or FALSE). You can use a ternary operation for this (e.g. BOOL b = <bool expr> ? TRUE : FAL...

Declare locals at top of scope (so before memcpy).
Also, when assigning to a BOOL, be explicit (i.e. TRUE or FALSE).
You can use a ternary operation for this (e.g. BOOL b = <bool expr> ? TRUE : FALSE

Does this message need a BOOL? Will UI ever send this message with a FALSE? Check with Nico on this.

Does this message need a BOOL? Will UI ever send this message with a FALSE? Check with Nico on this.

Add a space before )

Add a space before )

Set the fault property for this alarm to TRUE and then you don't need to request to go to fault mode - the alarm will do it for you. You should never have to request fault mode like this.

Set the fault property for this alarm to TRUE and then you don't need to request to go to fault mode - the alarm will do it for you. You should never have to request fault mode like this.

Add a blank line after declaration(s) for a little separation.

Add a blank line after declaration(s) for a little separation.

Need function header above function.

Need function header above function.

Need unique s/w malf id.

Need unique s/w malf id.

Is this right?

Is this right?

Align =

Align =

Should be in Ejector.c

Should be in Ejector.c

LEAHI-TD-FIRMWARE-LDT-1886_Blood Set Auto-Load and Auto-Eject - TD
LEAHI-TD-FIRMWARE-LDT-1886_Blood Set Auto-Load and Auto-Eject - TD
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 96 more files in changeset.
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 15 more files in changeset.