Comment Search

Its like we need to compile the safe library and include it. But if we are using only memcpy and memset, memcmp and static code analyzer check the destination buffer size , I am marking it as resolved

done

added. thanks!

Moved

Oh, ok, I thought 'i' only covers 8 bit not 32

This is for alpha

Fixed.

You don't need the if statement. The override function will check it automatically.

Should use existing override payload structure (the one with the index) located in TestSupport.h.
Add an enum in this unit for the 5 DD treatment parameters so you can interpret the index field of the message payload to find the specific parameter that is being overridden.

Changed the comments

publish filter flush status , add other important data , inlet pressure etc

Fixed.

I don't see this function resetting Tx params. Probably a good idea to do that, so add call to reset.

can we make this a more general method and put in a shared file and update throughout.. updatet to something like

def setText(obj, text):
    obj["text"] = text
    return obj:

Dara Navaei, this is not a CAN message payload, but it does look like a sensor serial message payload. I'm assuming this code works, so sensor does not expect a packed structure?

another mouseClick(waitForObject(confirmButton)) after fw validates parameter is needed

Add tst_general_alarm_requirements_instruction to suite.conf file

statusColor is so general, but is bound to the heparinStatusColor.
Please either change the status color to be heparin only or change the heparinStatus Color name to be more general.
The latter is better.

fixed it

Add blank line before return statement

Not sure how Doxygen will handle this comment between the /// and the const. I think we should remove it.

updated

updated

keep lock button enabled and update logic

revert