TreatmentRecirc.c

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Resolved merge conflicts from DENBUG-318

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

  1. … 19 more files in changeset.
Merged release/p1b/staging

  1. … 17 more files in changeset.
DENBUG-298 updated the code from staging

  1. … 19 more files in changeset.
HD-DENBUG-298_User Unable TO Deliver Saline P1B Staging
HD-DENBUG-298_User Unable TO Deliver Saline P1B Staging
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 19 more files in changeset.
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done

Done

Bad practice to have two calls to same function here. Possible that 2nd call will return different result. Should call function once to assign value to a local var prior to if statement and then re...

Bad practice to have two calls to same function here. Possible that 2nd call will return different result.
Should call function once to assign value to a local var prior to if statement and then reference the local var in the if statement.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Yes. I think we should have these events to information.

Yes. I think we should have these events to information.

Are these permanent?

Are these permanent?

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Add comment explaining what we're doing here.

Add comment explaining what we're doing here.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Moved it to OperationModes

Moved it to OperationModes

For other confirmations, an executive (e.g. alarm mgmt monitor or standby mode exec) is responsible for checking confirmation status and we don't call specific handlers from here. Why can't we foll...

For other confirmations, an executive (e.g. alarm mgmt monitor or standby mode exec) is responsible for checking confirmation status and we don't call specific handlers from here. Why can't we follow that design pattern for this confirmation?

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Merge branch 'DENBUG-298-user-unable-to-deliver-saline-bolus-during-alarm-conditions-where-the-blood-pump-is-recirculating' into develop

  1. … 3 more files in changeset.
DENBUG-302 continued with the changes for the UF rate screen support

  1. … 9 more files in changeset.
Done

Done

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

2 blank lines above/below test banner.

2 blank lines above/below test banner.

Done

Done

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.