Common.h

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

DEN-12931 addressed the code review comments

Removed the code.

Removed the code.

Done.

Done.

Done.

Done.

I thought exiting the normal state is the last request we want to check after making sure nothing else is requested.

I thought exiting the normal state is the last request we want to check after making sure nothing else is requested.

Done.

Done.

Blood leak is still in development. This will be addressed in DEN-13460.

Blood leak is still in development. This will be addressed in DEN-13460.

Done.

Done.

We only do that if we do the == comparison. This is to make sure the compiler errors out if only one = is put instead of ==.

We only do that if we do the == comparison. This is to make sure the compiler errors out if only one = is put instead of ==.

Done.

Done.

Done.

Done.

This can be reached in VC since it is not in a for loop.

This can be reached in VC since it is not in a for loop.

This can be reached in VC because it is not in a for loop.

This can be reached in VC because it is not in a for loop.

Why removed?

Why removed?

Why removed? I thought we needed this for VC?

Why removed? I thought we needed this for VC?

F has been added.

F has been added.

No. Code removed.

No. Code removed.

If no actions handled in this mode, add a comment saying so.

If no actions handled in this mode, add a comment saying so.

Yes, at bottom of exec function above.

Yes, at bottom of exec function above.

Remove blank line.

Remove blank line.

Can we remove this now?

Can we remove this now?

DEN-13460 updated the code

  1. … 1 more file in changeset.
DEN-13460 updated the NV data

  1. … 2 more files in changeset.
Check NEARLY_ZERO #define. Value should be 0.00000001F. I believe previous review commented indicated that all floating point #defines should have "F" at the end.

Check NEARLY_ZERO #define. Value should be 0.00000001F. I believe previous review commented indicated that all floating point #defines should have "F" at the end.

SW_CONFIG_ENABLE_VALUE should be first in the conditional.

SW_CONFIG_ENABLE_VALUE should be first in the conditional.

Need #define for minimum vRef

Need #define for minimum vRef

Is the data being published elsewhere?

Is the data being published elsewhere?

Is this supposed to have been removed.

Is this supposed to have been removed.

indent of cases looks too small.

indent of cases looks too small.