MsgDefs.h

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
It is there in line 131 It would be nice to have it clubbed it together like #ifdef _TEENSY_CONDUCTIVITY_DRIVER_ place all code for teensy here { MSG_ID_DD_SET_CONDUCTIVITY_MODEL_REQUEST, &testSe...

It is there in line 131

It would be nice to have it clubbed it together like

#ifdef _TEENSY_CONDUCTIVITY_DRIVER_
place all code for teensy here

Unknown macro: { MSG_ID_DD_SET_CONDUCTIVITY_MODEL_REQUEST, &testSetTeenyConductivityModel }

,
#else
// move the all code under #ifndef _TEENSY_CONDUCTIVITY_DRIVER_ here

#endif

fixed.

fixed.

reordered and wrapped in a macro

reordered and wrapped in a macro

see other comment.

see other comment.

reordered MSG IDs such that the teensy specific ones are wrapped in a macro and are far enough in sequence to not cause trouble when we are ready to remove them.

reordered MSG IDs such that the teensy specific ones are wrapped in a macro and are far enough in sequence to not cause trouble when we are ready to remove them.

fixed.

fixed.

removed

removed

fixed.

fixed.

it stop gap to see if it was valve specific. Its been reverted.

it stop gap to see if it was valve specific. Its been reverted.

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 1 more file in changeset.
LDT-2004-3 Merging branch staging into LDT-2004-drybicarb---dd

  1. … 1 more file in changeset.
Add TODO to comment.

Add TODO to comment.

Put () around the 2 conditions

Put () around the 2 conditions

Does it make more sense to have them together?

Does it make more sense to have them together?

Is there no "first" enum for this group?

Is there no "first" enum for this group?

Align comment.

Align comment.

arrange this code and new sensor code in a group- line 131 - 142

arrange this code and new sensor code in a group- line 131 - 142

I think we need a #ifdef TEENSY macro ?

I think we need a #ifdef TEENSY macro ?

Need two spaces before the test function

Need two spaces before the test function

This is already initialized (properly) near top of function. It is not proper (zero) here.

This is already initialized (properly) near top of function. It is not proper (zero) here.

Is there going to be an #else for new sensor?

Is there going to be an #else for new sensor?

Are we supporting multiple models? Is this for Teensy or new sensor or both?

Are we supporting multiple models? Is this for Teensy or new sensor or both?

Will new conductivity sensor driver have this override too? It looks Teensy specific.

Will new conductivity sensor driver have this override too? It looks Teensy specific.

Fixed it

Fixed it

Ya , I forgot the coding guide line . Fixed it

Ya , I forgot the coding guide line .
Fixed it

fixed it

fixed it

Fixed it

Fixed it