leahi-common

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
please add bpLeftHomeTimerCounter to inputs and outputs.

please add bpLeftHomeTimerCounter to inputs and outputs.

please add bpLeftHomeTimerCounter to inputs and outputs.

please add bpLeftHomeTimerCounter to inputs and outputs.

Why was this function header changed? I think it was correct before.

Why was this function header changed? I think it was correct before.

If these messages are for change to UF volume during treatment, the message ID should have the word TREATMENT in it to distinguish from the original UF volume message that comes in pre-treatment.

If these messages are for change to UF volume during treatment, the message ID should have the word TREATMENT in it to distinguish from the original UF volume message that comes in pre-treatment.

It is weird that we're responding so far from request. It is not clear to me that Dialin has requested anything at this point.

It is weird that we're responding so far from request. It is not clear to me that Dialin has requested anything at this point.

Details looks weird. Where is inputs and outputs like other function headers?

Details looks weird. Where is inputs and outputs like other function headers?

Now it's backward. You have to change the ENABLE to DISABLE too.

Now it's backward. You have to change the ENABLE to DISABLE too.

Add a blank line after banner.

Add a blank line after banner.

Please restore before you merge to staging.

Please restore before you merge to staging.

If I'm reading this right, we're saying that if bad CRC alarm is NOT enabled, initialize the alarm? That seems backward.

If I'm reading this right, we're saying that if bad CRC alarm is NOT enabled, initialize the alarm? That seems backward.

Same comments as ENABLE_ALARM_1 above.

Same comments as ENABLE_ALARM_1 above.

Same comments as ENABLE_ALARM_1 above.

Same comments as ENABLE_ALARM_1 above.

This definitions has a terrible name (I have no idea what it does) and I think it should be reversed (i.e. alarms are disabled when defined so that a release build will have alarms enabled).

This definitions has a terrible name (I have no idea what it does) and I think it should be reversed (i.e. alarms are disabled when defined so that a release build will have alarms enabled).

Call the init for the battery driver here.

Call the init for the battery driver here.

INVALID_BATTERY

INVALID_BATTERY

Remove repeating entries.

Remove repeating entries.

remove extra blank line.

remove extra blank line.

Remove this comment.

Remove this comment.

Remove blank line.

Remove blank line.

initialize all of these in init function.

initialize all of these in init function.

Missing doxygen group

Missing doxygen group

These should be in alphabetical order.

These should be in alphabetical order.

Update all function headers to Leahi format (e.g. input and output have a \b to make them bold).

Update all function headers to Leahi format (e.g. input and output have a \b to make them bold).

Add blank line after banner

Add blank line after banner

Add blank line after banner

Add blank line after banner

Add a blank line after banner. Initialize in init function instead of here.

Add a blank line after banner.

Initialize in init function instead of here.

Move all but I2CBatteryStatus override functions to the battery monitor.

Move all but I2CBatteryStatus override functions to the battery monitor.

Move definitions to top (above functions).

Move definitions to top (above functions).

BatteryDriver

BatteryDriver

Use Leahi function format and helper functions for overrides.

Use Leahi function format and helper functions for overrides.