TDCommon.h

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Bamboo Commit: Updated TDCommon.h with build versions from Bamboo.

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 15 more files in changeset.
Merge branch 'staging' into LDT-1886-blood-set-auto-load-and-auto-eject-implement-ejector-sensor

  1. … 15 more files in changeset.
LDT-1886 addressed the code review comment

We tend to initialize the local variables as a standard practice.

We tend to initialize the local variables as a standard practice.

LDT-1886 updated the valves code to be able to rehome the valves if needed

  1. … 1 more file in changeset.
These don't need to be initialized to zero.

These don't need to be initialized to zero.

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done.

Done.

No. It is permanent, I would like to know the value for the homing.

No. It is permanent, I would like to know the value for the homing.

The else in this trinary looks wrong - doesn't round up to multiple of 8.

The else in this trinary looks wrong - doesn't round up to multiple of 8.

This if should have an else where we zero .valveOpsStartTime (in case it briefly got set but then failed this if before timeout).

This if should have an else where we zero .valveOpsStartTime (in case it briefly got set but then failed this if before timeout).

Declarations should be at top of scope.

Declarations should be at top of scope.

Is this temporary?

Is this temporary?

Done

Done

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

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

Done

Done

LDT-1886 changed the code for disabling the ejector motor

  1. … 1 more file in changeset.
Bamboo Commit: Updated TDCommon.h with build versions from Bamboo.

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 94 more files in changeset.
move static function after public function

move static function after public function

let public function on top

let public function on top

Let all public functions come on top for all files

Let all public functions come on top for all files

Should we comment this out before merging?

Should we comment this out before merging?