common

Clone Tools

Stats

Commits this week: 7
Total Committers: 18
Last Commit: 11:19
Commits this week: 7

Lines of code count not available

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
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 21 Apr 2023
2:06 pm

Sean Nash

1:51 pm

Dara Navaei

11:41 am

Sean Nash

Note in brief that function will return FALSE if Dialin not logged in.

Note in brief that function will return FALSE if Dialin not logged in.

11:40 am

Sean Nash

testConfig is an input, not an output.

testConfig is an input, not an output.

11:39 am

Sean Nash

Change "release" to "test".

Change "release" to "test".

11:38 am

Sean Nash

Yes, s/w fault (for invalid config). Ignore command for not logged in.

Yes, s/w fault (for invalid config). Ignore command for not logged in.

11:37 am

Sean Nash

Add check for Dialin logged in.

Add check for Dialin logged in.

11:36 am

Sean Nash

Yes, s/w fault (for invalid config). Ignore command for not logged in.

Yes, s/w fault (for invalid config). Ignore command for not logged in.

11:34 am

Sean Nash

Also, Dialin must be logged in to accept a test config command.

Also, Dialin must be logged in to accept a test config command.

11:32 am

Sean Nash

memset wants a byte. Just say 0 instead of using KEY #define. Also, TEST_CONFIG_T is an enum, so size is probably 1 and so this probably won't initialize whole array.

memset wants a byte. Just say 0 instead of using KEY #define.
Also, TEST_CONFIG_T is an enum, so size is probably 1 and so this probably won't initialize whole array.

11:26 am

Sean Nash

Add blank line between banner and #define.

Add blank line between banner and #define.

11:26 am

Sean Nash

Add blank line between banner and #define.

Add blank line between banner and #define.

11:25 am

Sean Nash

Are DG and HD sharing this enum?

Are DG and HD sharing this enum?

11:17 am

Sean Nash

We want test configs be active only if explicitly enabled. Safest condition would be to say ( TRUE == getTest.... ). Make this change everywhere getTestConfigStatus function is used.

We want test configs be active only if explicitly enabled. Safest condition would be to say ( TRUE == getTest.... ).
Make this change everywhere getTestConfigStatus function is used.

11:15 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

11:14 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

11:06 am

Sean Nash

Fixed.

Fixed.

11:04 am

Sean Nash

Fixed.

Fixed.

10:16 am

jtaylor  (deleted user)

Thursday 20 Apr 2023
11:14 am

wbracken  (deleted user)

Add testStartTime, audioTestStartTime, alarmAudioTestToneRequested and to header

Add testStartTime, audioTestStartTime, alarmAudioTestToneRequested and to header

11:07 am

wbracken  (deleted user)

Extra blank line.

Extra blank line.

11:02 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

11:02 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

9:04 am

Dara Navaei

9:01 am

jtaylor  (deleted user) , Sean Nash and Michael Garthwaite

8:52 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

8:45 am

Dara Navaei

It is not needed. So I removed it.

It is not needed. So I removed it.

8:34 am

Sean Nash

Needed?

Needed?

8:23 am

Sean Nash

Fixed.

Fixed.

8:20 am

Sean Nash

Fixed.

Fixed.