DDCommon.h

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Bamboo Commit: Updated DDCommon.h with build versions from Bamboo.

marking it as resolved since it is temporary driver

marking it as resolved since it is temporary driver

As per Sean's comment mark it as resolved

As per Sean's comment mark it as resolved

marking it as resolved since this is a temporary driver

marking it as resolved since this is a temporary driver

I am marking it as resolved since it is temporary driver

I am marking it as resolved since it is temporary driver

Its like we need to compile the safe library and include it. But if we are using only memcpy and memset, memcmp and static code analyzer check the destination buffer size , I am marking it as resolved

Its like we need to compile the safe library and include it. But if we are using only memcpy and memset, memcmp and static code analyzer check the destination buffer size , I am marking it as resolved

This function has an output.

This function has an output.

Remove extra blank line.

Remove extra blank line.

Bamboo Commit: Updated DDCommon.h with build versions from Bamboo.

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?

Bamboo Commit: Updated DDCommon.h with build versions from Bamboo.

Done

Done

comments should start with capital letter

comments should start with capital letter

Shouldn't EVERY state set ALL valves?

Shouldn't EVERY state set ALL valves?

Suggestive comment => can we make it as a run time configuration option so that we don't need to change the code ?

Suggestive comment => can we make it as a run time configuration option so that we don't need to change the code ?

Let public function on top

Let public function on top

Suggestive comment => Shall we do it as a run time configurable so that we don't need to change the code?

Suggestive comment => Shall we do it as a run time configurable so that we don't need to change the code?

Mixing static and public function , let public function comes on top

Mixing static and public function , let public function comes on top

comment should start with capital letter

comment should start with capital letter

These set of valves defined as per system architecture (to make sure all valves initialized as expected).

These set of valves defined as per system architecture (to make sure all valves initialized as expected).

Done.

Done.

Done.

Done.

Done.

Done.