hdfirmware

Clone Tools

Stats

Commits this week: 13
Total Committers: 17
Last Commit: 10 Jun
Commits this week: 13

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
Wednesday 12 Oct 2022
1:31 pm

Michael Garthwaite

DEN-13903: code review fixes for merging.

Options
    • -0
    • +1
    /firmware/App/Modes/Prime.c
1:30 pm

Dara Navaei

Done.

Done.

1:30 pm

Dara Navaei

Done.

Done.

1:29 pm

Dara Navaei

Done.

Done.

1:25 pm

Dara Navaei

This event is used for NV data management events. HD_EVENT_NO_EVENT is not currently in use.

This event is used for NV data management events. HD_EVENT_NO_EVENT is not currently in use.

1:21 pm

Dara Navaei

Done.

Done.

1:20 pm

Dara Navaei

I am waiting for HDD to be updated to reserved so I can rename them.

I am waiting for HDD to be updated to reserved so I can rename them.

1:19 pm

Dara Navaei

Done.

Done.

1:18 pm

Dara Navaei

Done.

Done.

1:16 pm

Dara Navaei

Done.

Done.

1:16 pm

Dara Navaei

Done.

Done.

1:12 pm

Dara Navaei

Done.

Done.

12:43 pm

Darren Cox <darren_m@cox.net>

DEN-14000 Code review updates.

Options
    • -1
    • +1
    /firmware/App/Modes/OperationModes.c
    • -4
    • +3
    /firmware/App/Modes/OperationModes.h
    • -1
    • +2
    /firmware/App/Services/SystemCommMessages.h
10:10 am

Sean Nash

Can we assign request_type here at declaration instead of below?

Can we assign request_type here at declaration instead of below?

10:05 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:05 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:05 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:05 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:03 am

Sean Nash

Add // MSG_ID_... comment above prototype.

Add // MSG_ID_... comment above prototype.

10:00 am

Sean Nash

Add a /// doxygen comment above this new declaration.

Add a /// doxygen comment above this new declaration.

9:59 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

9:59 am

Sean Nash

Should say "private definitions".

Should say "private definitions".

9:58 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

9:34 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

9:33 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

9:32 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

9:25 am

Michael Garthwaite

DEN-13903: DIAL-204: code review comments.

Options
    • -3
    • +4
    /firmware/App/Controllers/BloodFlow.c
    • -7
    • +7
    /firmware/App/Controllers/DialInFlow.c
    • -6
    • +6
    /firmware/App/Controllers/DialOutFlow.c
    • -1
    • +1
    /firmware/App/Modes/Dialysis.c
    • -1
    • +0
    /firmware/App/Services/SystemComm.c
    • -1
    • +0
    /firmware/App/Services/SystemCommMessages.c
9:16 am

Sean Nash

This probably warrants a comment.

This probably warrants a comment.

9:12 am

Sean Nash

Is this a TODO? Can we just do it?

Is this a TODO? Can we just do it?

9:11 am

Sean Nash

Why are we checking non-zero here? Seems like we will get stuck here if years really is set to zero.

Why are we checking non-zero here? Seems like we will get stuck here if years really is set to zero.