Comment Search

break this line

Should we move this up before the test functions?

add @messages MSG_ID_TD_UI_GENERIC_CONFIRMATION_REQUEST

Set to local BOOL and then return the BOOL.

output is none

add a blank line above

break this line

align comment

break lines of this function to match our standard of 100 columns.

add accept

While you are updating this file, can please break this line?

Should we remove APP_MONITORS and align with other header files.
Example: _BUTTONS_H_

Resolved.

What is the use case of including this header?

this does not work. if i replace 0 with 3 this still passes.

please update to use the lamba and test the value like the other test cases for all these

Should we move this to second number and add a blank line after it?

Added the test script which is same has in staging

My understanding is that Standby should stop all pumps and heaters, including D108. Although the D108 driver implementation is not available yet, I thought keeping this call would preserve the intended Standby behavior and allow it to work once driver support is added. Could you please let me know if there is any concern with leaving it in place for now?

recirculateToBloodPrimeRequest is output.

Add @message id MSG_ID_TD_UI_GENERIC_CONFIRMATION_REQUEST

Inputs : fpgaSensorReadings
Outputs: none

Delete empty line

add a space before and after macro in [ ]

Thank you for clarifying!

add a space before and size in [ ]