Services

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 13 more files in changeset.
Merge branch 'DEN-14344-sw-s85-mg-air-pump' into develop

  1. … 4 more files in changeset.
RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

DEN-14307 addressed a code review comment

RESOLVED IN CODEWALKTHROUGH.

RESOLVED IN CODEWALKTHROUGH.

DEN-14307 addressed the code review comments

Done

Done

Data is a local variable but added alarmButtonBlockers

Data is a local variable but added alarmButtonBlockers

RESOLVED IN CODEWALKTHROUGH.

RESOLVED IN CODEWALKTHROUGH.

RESOLVED IN CODEWALKTHROUGH.

RESOLVED IN CODEWALKTHROUGH.

data and alarmButtonBlockers?

data and alarmButtonBlockers?

Should alarmButtonBlockers and ALARM_TABLE also be added.

Should alarmButtonBlockers and ALARM_TABLE also be added.

RESOLVED IN CODEWALKTHROUGH.

RESOLVED IN CODEWALKTHROUGH.

RESOLVED IN CODEWALKTHROUGH.

RESOLVED IN CODEWALKTHROUGH.

RESOLVED IN CODEWALKTHROUGH.

RESOLVED IN CODEWALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

DEN-14307: Removed obsolete build switches.

  1. … 4 more files in changeset.
DEN-14307 addressed code review comments

  1. … 1 more file in changeset.
Done

Done

Done

Done

I think we still want to keep the comment. It has not been implemented yet.

I think we still want to keep the comment. It has not been implemented yet.

Done

Done

Done

Done

Done

Done

Fixed the setServiceTime function. Thanks.

Fixed the setServiceTime function. Thanks.

Done.

Done.

Agree, though this comment belongs in NVDataMgmt.c where setServiceTime() is lives. That function should return TRUE if successful.

Agree, though this comment belongs in NVDataMgmt.c where setServiceTime() is lives. That function should return TRUE if successful.

All of these terms are integers and cast is coming too late (after division). To prevent loss of precision on division, we want to cast both sides of the division as floats.

All of these terms are integers and cast is coming too late (after division). To prevent loss of precision on division, we want to cast both sides of the division as floats.