Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Can we get to this if condition before we've even set bloodPumpOcclusionAfterCartridgeInstall ? And, if so, is that a problem?

Can we get to this if condition before we've even set bloodPumpOcclusionAfterCartridgeInstall ? And, if so, is that a problem?

Extra space before comment.

Extra space before comment.

Blank lines before and after if.

Blank lines before and after if.

Comment says PreTreat, If checks for Treatment. Also space needed after if.

Comment says PreTreat, If checks for Treatment. Also space needed after if.

Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Fixed. Thanks!

Can you clarify what case you're thinking of?

Can you clarify what case you're thinking of?

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Bill, I was just asking why there was a blank line added before final } of last function. The doxygen end group stuff should stay. I will generally add my comments directly below the issue, not above.

Bill, I was just asking why there was a blank line added before final } of last function. The doxygen end group stuff should stay.
I will generally add my comments directly below the issue, not above.

/*@)/ appears to be the last line in all files? Should it put back in. Must be a doxygen command.

/*@)/ appears to be the last line in all files? Should it put back in. Must be a doxygen command.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Removed blank lilne.

Removed blank lilne.

Yes. And don't mark replies as requiring resolution.

Yes. And don't mark replies as requiring resolution.

Not sure why that's there. Will remove it

Not sure why that's there. Will remove it

You mean the blank line?

You mean the blank line?

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.