DG-DEN-14706

DG-DEN-14706

Activity on this page:

Line History

Line history chart not available

Line history chart not available

 

Stats

Commits this week: 0
Total Commits: 3,303
Last Commit: 20 Nov
Commits this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 
Wednesday 06 Sep 2023
7:38 am

Michael Garthwaite

Tuesday 05 Sep 2023
11:43 am

Sean Nash , jtaylor  (deleted user) and wbracken  (deleted user)

10:06 am

jtaylor  (deleted user)

Is the "0" an ACK,NACK, PENDING_OVERRIDE or some other flag? Suggest a #define. (or a comment)

Is the "0" an ACK,NACK, PENDING_OVERRIDE or some other flag? Suggest a #define. (or a comment)

10:06 am

jtaylor  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:05 am

Michael Garthwaite

We typically dont #define for 0. However, the logic has been reworked that we no longer need to compare by 0.

We typically dont #define for 0. However, the logic has been reworked that we no longer need to compare by 0.

Friday 01 Sep 2023
11:19 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

11:19 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

11:19 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

11:19 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

11:18 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Thursday 31 Aug 2023
2:37 pm

Sean Nash

This if doesn't appear to be doing anything. Remove.

This if doesn't appear to be doing anything. Remove.

2:36 pm

Sean Nash

This should be inverse of what you had before. I think this is too brief.

This should be inverse of what you had before. I think this is too brief.

2:11 pm

Michael Garthwaite

Pushed wip code. I'm unsure what or if we need to put anything to compare override values.

Pushed wip code. I'm unsure what or if we need to put anything to compare override values.

2:11 pm

Dara Navaei

2:11 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

2:10 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

12:45 pm

Sean Nash

Too many () after &&?

Too many () after &&?

12:44 pm

Sean Nash

Remove extra blank line.

Remove extra blank line.

12:44 pm

Sean Nash

Already TRUE - why set again? I think "if" should be reversed - if not override, set used to FALSE. And then after if, always set result to TRUE (means we found a match - which we did).

Already TRUE - why set again?
I think "if" should be reversed - if not override, set used to FALSE. And then after if, always set result to TRUE (means we found a match - which we did).

12:18 pm

Dara Navaei

DG-DEN-14706_Need Mechanism For Alarm ID Can Message Not Acked And Alarm ID DG Can Message Not Acked
DG-DEN-14706_Need Mechanism For Alarm ID Can Message Not Acked And Alarm ID DG Can Message Not Acked
Monday 21 Aug 2023
11:27 am

Michael Garthwaite

11:24 am

wbracken  (deleted user) and Sean Nash

11:10 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

9:49 am

jtaylor  (deleted user)

Suggest aligning comments in the block above

Suggest aligning comments in the block above

Friday 18 Aug 2023
12:35 pm

Michael Garthwaite

Removed #define. Thanks!

Removed #define. Thanks!

11:04 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:13 am

Sean Nash

I think override should be OR'd with timeout only. The used condition should still have to be true even if override is in place.

I think override should be OR'd with timeout only. The used condition should still have to be true even if override is in place.

9:58 am

Sean Nash

I don't see where this #define is ever used. Remove if not needed.

I don't see where this #define is ever used. Remove if not needed.

Thursday 17 Aug 2023
6:04 pm

Dara Navaei

DG-DEN-14706_Need Mechanism For Alarm ID Can Message Not Acked And Alarm ID DG Can Message Not Acked
DG-DEN-14706_Need Mechanism For Alarm ID Can Message Not Acked And Alarm ID DG Can Message Not Acked