Messaging.c

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Remove extra blank line.

Remove extra blank line.

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

moved the sets into a separate function from init

moved the sets into a separate function from init

Is this temporary? If so, put a TODO to remove later.

Is this temporary? If so, put a TODO to remove later.

Remove extra spaces between { and msg name.

Remove extra spaces between { and msg name.

Is this always needed or is it only needed in one or more specific modes (i.e. gen dialysate mode)? If only needed in specific modes, I think this should be called from those modes (similar to BC e...

Is this always needed or is it only needed in one or more specific modes (i.e. gen dialysate mode)? If only needed in specific modes, I think this should be called from those modes (similar to BC exec).

Do these belong in the spent chamber fill group?

Do these belong in the spent chamber fill group?

I think you are seeing previous commit. It is not there in the present code.

I think you are seeing previous commit. It is not there in the present code.

Done. Thank you.

Done. Thank you.

Changed. Thank you.

Changed. Thank you.

is it merge issue ?

is it merge issue ?

If it is not used for serial message, its OK, but if you are opting for packing , club it together , in line number 36 , or bring that structure defined on top to here

If it is not used for serial message, its OK, but if you are opting for packing , club it together , in line number 36 , or bring that structure defined on top to here

Please bring public functions on top. check it for all function in this file , except test functions

Please bring public functions on top. check it for all function in this file , except test functions

Vendor - TEL - Sameer Poyil this structure is not used to message handling.

Vendor - TEL - Sameer Poyil this structure is not used to message handling.

Done

Done

comments should start with capital letter

comments should start with capital letter

Dara Navaei, this is not a CAN message payload, but it does look like a sensor serial message payload. I'm assuming this code works, so sensor does not expect a packed structure?

Dara Navaei, this is not a CAN message payload, but it does look like a sensor serial message payload. I'm assuming this code works, so sensor does not expect a packed structure?

Resolved

Resolved

Shouldn't EVERY state set ALL valves?

Shouldn't EVERY state set ALL valves?

Change comment from 3 to 5 minutes to match code.

Change comment from 3 to 5 minutes to match code.

I don't see fix

I don't see fix