firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Magic #.

Magic #.

Seems we are not consistent about using #ifndef RELEASE. We have it here, but not in the if above on line 400.

Seems we are not consistent about using #ifndef RELEASE. We have it here, but not in the if above on line 400.

I think we should use <> for library includes. And they should go on top.

I think we should use <> for library includes. And they should go on top.

Remove blank line. And what is new remove field for?

Remove blank line. And what is new remove field for?

Have you tried a release build to see if we have this right?

Have you tried a release build to see if we have this right?

They do not. Remove volatile from these declarations.

They do not. Remove volatile from these declarations.

Fix indenting here.

Fix indenting here.

HD-DEN-12121_SW Dev Sprint 65
HD-DEN-12121_SW Dev Sprint 65
Sean Nash please reply.

Sean Nash please reply.

Good point, I updated it.

Good point, I updated it.

I uninitialized it.

I uninitialized it.

That is right.

That is right.

Sean Nash please reply.

Sean Nash please reply.

Sean Nash please respond,.

Sean Nash please respond,.

This code has been removed already.

This code has been removed already.

Yes. We might go back to it.

Yes. We might go back to it.

It must be removed.

It must be removed.

Shouldn't this include be in TaskGeneral.c?

Shouldn't this include be in TaskGeneral.c?

Is the commented test code still needed?

Is the commented test code still needed?

Venous pressure does not have an error count. Is it needed?

Venous pressure does not have an error count. Is it needed?

Do we want to keep this commented out defs?

Do we want to keep this commented out defs?

Should 300 be a magic number?

Should 300 be a magic number?

Should 300 be a magic number?

Should 300 be a magic number?

There are no changes in this file and its not a new file.

There are no changes in this file and its not a new file.

Did we ever get clarification on these threshold values?

Did we ever get clarification on these threshold values?

Is this supposed to be assigned to zero? it is unlike the other variables that have been changed

Is this supposed to be assigned to zero? it is unlike the other variables that have been changed

based on the struct, fanStatus.dutycycle does not need to be initialized NUM_OF_FANS_NAMES times.

based on the struct, fanStatus.dutycycle does not need to be initialized NUM_OF_FANS_NAMES times.

Do these variables need to be volatile?

Do these variables need to be volatile?

HD-DEN-11750_DG Dev Dialysate Temperature Control Tune Up
HD-DEN-11750_DG Dev Dialysate Temperature Control Tune Up