td-firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Add ons

    • -31
    • +59
    /firmware/App/Modes/ModeStandby.c
Remove blank line.

Remove blank line.

Still there.

Still there.

Remove if not needed.

Remove if not needed.

Why commented out? If not needed, delete.

Why commented out? If not needed, delete.

Use if/else or ternary to set result to TRUE or FALSE explicitly.

Use if/else or ternary to set result to TRUE or FALSE explicitly.

Added handlers for the states

    • -25
    • +84
    /firmware/App/Modes/ModePostTreat.c
    • -21
    • +123
    /firmware/App/Services/Messaging.c
Remove extra blank line.

Remove extra blank line.

Fix indent for break. Add comment before break "// unhandled command - do nothing.

Fix indent for break. Add comment before break "// unhandled command - do nothing.

Fix indent for break. Add comment before break "// unhandled position - do nothing.

Fix indent for break. Add comment before break "// unhandled position - do nothing.

Do we need this code? Can we delete it?

Do we need this code? Can we delete it?

Remove extra blank line.

Remove extra blank line.

Change these comments (H1 and H19 home arrays) to something like "Command list for H1 homing function."

Change these comments (H1 and H19 home arrays) to something like "Command list for H1 homing function."

Change comment to "Command list for pinch valve position change function."

Change comment to "Command list for pinch valve position change function."

Add a space before &

Add a space before &

LDT-4029:Updated heparin setup state

    • -2
    • +2
    /firmware/App/Modes/StatePreTxHeparinSetup.c
LDT-4029:Updated Heparin setup state

    • -0
    • +1
    /firmware/App/Controllers/SyringePump.c
    • -5
    • +13
    /firmware/App/Modes/StatePreTxHeparinSetup.c
Does default mean invalid state (s/w fault) or does it mean it's a state that we don't need to do anything for? If latter, add a comment here saying that.

Does default mean invalid state (s/w fault) or does it mean it's a state that we don't need to do anything for? If latter, add a comment here saying that.

Use if else or ternary operator to set BOOL to TRUE or FALSE explicitly.

Use if else or ternary operator to set BOOL to TRUE or FALSE explicitly.

Add a TODO comment to adjust this when new syringe pump assembly is ready.

Add a TODO comment to adjust this when new syringe pump assembly is ready.

Is this still necessary? Comment out before merging to staging.

Is this still necessary? Comment out before merging to staging.

Why are registers being repeated? Sync w/ latest HDD.

Why are registers being repeated? Sync w/ latest HDD.

Can we remove this commented out line of code?

Can we remove this commented out line of code?

Why is this commented out?

Why is this commented out?

Why is state transition and publish removed?

Why is state transition and publish removed?

Please address the commented code.

Please address the commented code.

Should we remove the commented code?

Should we remove the commented code?

We should manage the commented-out code.

We should manage the commented-out code.

break this line

break this line

break this line

break this line