application

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
If this is to log Dialin messages, we should always do that (no switch). If a Dialin command is given/responded to, we definitely want to know about that - so want to see it in the log. Dialin comm...

If this is to log Dialin messages, we should always do that (no switch). If a Dialin command is given/responded to, we definitely want to know about that - so want to see it in the log. Dialin commands are always one-shot type messages, so will not impact log size.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED

RESOLVED

This code is going to be used later in the Log Export for slow transfer & confirmation for the USB port slow transfer issue. I kept it in the application repo to keep it safe. Commented out to not ...

This code is going to be used later in the Log Export for slow transfer & confirmation for the USB port slow transfer issue.
I kept it in the application repo to keep it safe.
Commented out to not affect the coverage.

http://dvm-linux02:8060/cru/#UI-DEN-12716-2CFR-30978 in PostTreatmentReview.qml@40

http://dvm-linux02:8060/cru/#UI-DEN-12716-2CFR-30978
in PostTreatmentReview.qml@40

Where would this value change?

Where would this value change?

At the moment I added that switch I assumed it should only be used for V&V and SysV testing.

At the moment I added that switch I assumed it should only be used for V&V and SysV testing.

This is the requirement. PRS 417: The HD Device manufacturing default timezone shall be (UTC-08:00) Pacific Time.

This is the requirement.
PRS 417: The HD Device manufacturing default timezone shall be (UTC-08:00) Pacific Time.

Why is main.cpp all commented out?

Why is main.cpp all commented out?

All systems will be setup for Pacific time zone?

All systems will be setup for Pacific time zone?

Why must this be removed later?

Why must this be removed later?

UI-DEN-12716_UI BN S70
UI-DEN-12716_UI BN S70
RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

I don't usually remove these kinds of comments they are there for developers as a reminder to keep the indices match. This is like a code comment.

I don't usually remove these kinds of comments they are there for developers as a reminder to keep the indices match.
This is like a code comment.

So then when would we remove this line of code?

So then when would we remove this line of code?

RESOLVED.

RESOLVED.

Disabled the "coco end"s as well.

Disabled the "coco end"s as well.

To make sure it is intentional, and we didn't miss a parameter. also it helps with the copy pasts later.

To make sure it is intentional, and we didn't miss a parameter.
also it helps with the copy pasts later.

IT has been recently removed from the diagnostics screen. I'm not sure when it will be needed to get back again on that screen. Probably keeping it is a lot easier than reimplementing it.

IT has been recently removed from the diagnostics screen.
I'm not sure when it will be needed to get back again on that screen.
Probably keeping it is a lot easier than reimplementing it.

When will this get fixed (removed)?

When will this get fixed (removed)?

Why only comment out? Can't we just remove?

Why only comment out? Can't we just remove?

very good point Sean. Coco ignores it but logs each one which I think it more bothers us than coco. I am going to disable them too.

very good point Sean.
Coco ignores it but logs each one which I think it more bothers us than coco.
I am going to disable them too.

Does it bother CoCo if you disable a begin but leave the end there?

Does it bother CoCo if you disable a begin but leave the end there?

UI-DEN-12258_UI BN S66
UI-DEN-12258_UI BN S66