components

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

RESOLVED.

Disregard. Found where it is used.

Disregard. Found where it is used.

What is the purpose of this alias?

What is the purpose of this alias?

Why commented out?

Why commented out?

Why commented out?

Why commented out?

Why is this commented out?

Why is this commented out?

When will we do this?

When will we do this?

File name does not appear to match image.

File name does not appear to match image.

The declaration in the header file has const for all 3 parameters.

The declaration in the header file has const for all 3 parameters.

That is a very good point. No one ever mentioned that. Let me investigate, why I had all these repeated else. Could be for performance not to check next, but there is another way for that too. If ...

That is a very good point.
No one ever mentioned that.

Let me investigate, why I had all these repeated else.
Could be for performance not to check next, but there is another way for that too.
If I can't find a quick answer for that let me create a case for that since it is everywhere.

RESOLVED.

RESOLVED.

There has to be and there is. It is just a long line and those little ones are at the end.

There has to be and there is.
It is just a long line and those little ones are at the end.

It seems you have multiple else statements that do the same, is that right? If yes, why multiple elses?

It seems you have multiple else statements that do the same, is that right? If yes, why multiple elses?

Isn't is safer to have a break at the end of each case?

Isn't is safer to have a break at the end of each case?

UI-BUG-S62
UI-BUG-S62
RESOLVED.

RESOLVED.

RESOLVED

RESOLVED

Fixed. Thanks for your attention to the details.

Fixed.
Thanks for your attention to the details.

Some functions and variables in this file use PatientID instead of PtientID, was this spelling intentional?

Some functions and variables in this file use PatientID instead of PtientID, was this spelling intentional?

Added

Added

Needs function header?

Needs function header?

RESOLVED

RESOLVED

that is fine because...

that is fine because...

it is a test for the code review demo

it is a test for the code review demo