hdfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-15274 addressed the code review comments

    • -152
    • +157
    /firmware/App/Modes/ModePreTreat.c
Done

Done

Done

Done

Done

Done

I changed the code.

I changed the code.

Done

Done

Done

Done

I was thinking to send the command anyways. If the active reservoir is what we need, it will be ignored. But I changed the code.

I was thinking to send the command anyways. If the active reservoir is what we need, it will be ignored. But I changed the code.

Done

Done

Done. This indentation was picked by Code Composer Studios though.

Done. This indentation was picked by Code Composer Studios though.

Done

Done

DEN-15104 Updates per code review.

No longer needed.

No longer needed.

DEN-15274 added code to check the checkin message from dialin

DIAL-251: fixed payload preparation for operation status event. Fixed compiler warnings

    • -8
    • +13
    /firmware/App/Modes/OperationModes.c
    • -0
    • +18
    /firmware/App/Modes/OperationModes.h
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Per my other comment, please avoid in-function static variables.

Per my other comment, please avoid in-function static variables.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Not a fan of in function statics. Probably due to Vectorcast having a hard time initializing them before function is called. Please avoid doing this.

Not a fan of in function statics. Probably due to Vectorcast having a hard time initializing them before function is called. Please avoid doing this.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Now it's gone. No longer needed?

Now it's gone. No longer needed?

Bamboo Commit: Updated HDCommon.h with build versions from Bamboo

Merge branch 'DEN-15229-fw-dpi-pump-on-during-inverted-dialyzer' into develop

    • -13
    • +38
    /firmware/App/Modes/ModePreTreat.c