Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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.

Remove blank line.

Remove blank line.

Should we add _TARGET to these names for clarity?

Should we add _TARGET to these names for clarity?

Is the count really a float?

Is the count really a float?

Do we need an override for the RPM feedback?

Do we need an override for the RPM feedback?

This is a maximum per treatment?

This is a maximum per treatment?

I'm not seeing fixes in this file.

I'm not seeing fixes in this file.

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?

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).