dgfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Need to have 2 blank lines.

Need to have 2 blank lines.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Need to add a check if switchId is in valid range.

Need to add a check if switchId is in valid range.

Do we need to add copyrights or bamboo will do it?

Do we need to add copyrights or bamboo will do it?

Common has an enumeration (OPN_CLS_STATE_T) that is similar to this. Suggest using from common.

Common has an enumeration (OPN_CLS_STATE_T) that is similar to this. Suggest using from common.

Should this function move to ModeRecirculate? It does not seem to contain any information related to drain pump.

Should this function move to ModeRecirculate? It does not seem to contain any information related to drain pump.

What happens when calibration data is invalid?

What happens when calibration data is invalid?

The comment is out-date and need to be updated.

The comment is out-date and need to be updated.

Param name is not matched.

Param name is not matched.

This can fit in one line.

This can fit in one line.

Should it be 0 and is the first one in the enumeration?

Should it be 0 and is the first one in the enumeration?

Done.

Done.

This is fixed in the latest commit.

This is fixed in the latest commit.

Done.

Done.

The default pump control mode is none.

The default pump control mode is none.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

This structure was transferred DGDefs.h

This structure was transferred DGDefs.h

Where did this go?

Where did this go?

So these & operators are going to yield a zero or a non-zero number that will be different for each switch (not a boolean). I think your intent is for these switch data to be an enum. So, for examp...

So these & operators are going to yield a zero or a non-zero number that will be different for each switch (not a boolean). I think your intent is for these switch data to be an enum. So, for example, this one should be "... = ( ( fpgaRegister & CONCENTRATE_CAP_SWITCH_BIT_MASK ) != 0 ? OPEN : CLOSED );