Monitors

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

Resolved

Updated

Updated

Updated

Updated

LDT-3344 added macro for FPGADD register fields

  1. … 1 more file in changeset.
seems to be old , need to merge the staging code

seems to be old , need to merge the staging code

align the comment

align the comment

complicated check should be replaced with boolean flag

complicated check should be replaced with boolean flag

Variable name is not understood -temDataCollectionTimeInterval ?

Variable name is not understood -temDataCollectionTimeInterval ?

LDT-1473 latest from staging

  1. … 7 more files in changeset.
Yes. updated.

Yes. updated.

Align comments.

Align comments.

I would describe function as "reconfigures FPGA driver to use appropriate FPGA register maps".

I would describe function as "reconfigures FPGA driver to use appropriate FPGA register maps".

Add space between } and type name.

Add space between } and type name.

Indenting looks off for most fields in this struct.

Indenting looks off for most fields in this struct.

Why doesn't name and comment match?

Why doesn't name and comment match?

Fixed it

Fixed it

Ya , I forgot the coding guide line . Fixed it

Ya , I forgot the coding guide line .
Fixed it

fixed it

fixed it

Fixed it

Fixed it

fixed it

fixed it

Use explicit BOOL conditions (e.g. TRUE == isFreshDialysatePressureInRange).

Use explicit BOOL conditions (e.g. TRUE == isFreshDialysatePressureInRange).

So shouldn't we stop trimmer heater here then?

So shouldn't we stop trimmer heater here then?

I prefer to be explicit on BOOL conditions like this (e.g. TRUE == isFreshDialysatePressureInRange).

I prefer to be explicit on BOOL conditions like this (e.g. TRUE == isFreshDialysatePressureInRange).

Remove extra blank line.

Remove extra blank line.

LDT-3344 added function to initialize FPGA registers with every setConfig call

  1. … 2 more files in changeset.
I don't think we need to check test config for variable initialization.

I don't think we need to check test config for variable initialization.

fixed it

fixed it

Fixed it

Fixed it

fixed it

fixed it