hdfirmware / firmware / App / Controllers

AirPump.c

Clone Tools

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 19 Nov
Commits this week: 0

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 07 Feb
4:07 pm

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -1
    • +1
    ./AirPump.c
  2. … 116 more files in changeset.
Wednesday 29 Jan
6:09 pm

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -1
    • +1
    ./AirPump.c
  2. … 116 more files in changeset.
Tuesday 19 Mar 2024
7:07 pm

Dara Navaei

6:30 pm

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -2
    • +2
    ./AirPump.c
  2. … 4 more files in changeset.
6:13 pm

Sean Nash and Vinayakam Mani

5:26 pm

Dara Navaei

5:25 pm

Dara Navaei

DEN-16773 added the changes from staging

Options
    • -44
    • +6
    ./AirPump.c
  2. … 4 more files in changeset.
4:48 pm

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -2
    • +2
    ./AirPump.c
  2. … 4 more files in changeset.
4:26 pm

Dara Navaei

4:23 pm

pvedantam  (deleted user) , Sean Nash and Vinayakam Mani

4:11 pm

Dara Navaei

HD-DEN-16773_Add Chemical Disinfect Enable Institutional Record 2
HD-DEN-16773_Add Chemical Disinfect Enable Institutional Record 2
10:03 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:03 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

9:29 am

Dara Navaei

Done

Done

9:29 am

Dara Navaei

Done

Done

8:56 am

Sean Nash

It's 1 line of code either way. Just on top or below in an else. Vectorcast will see as a branch you need to test either way. If you put it in an else, the processor does less work and I think it's...

It's 1 line of code either way. Just on top or below in an else. Vectorcast will see as a branch you need to test either way.
If you put it in an else, the processor does less work and I think it's a lot easier to read.

8:54 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

8:28 am

Dara Navaei

I thought about having an else below but this way we write less code. I have done similar in the past.

I thought about having an else below but this way we write less code. I have done similar in the past.

8:23 am

Dara Navaei

Done

Done

Monday 18 Mar 2024
9:39 pm

Sean Nash

Put this in an else below.

Put this in an else below.

9:38 pm

Sean Nash

This should probably be in an else below.

This should probably be in an else below.

9:35 pm

Sean Nash

disinfect is misspelled.

disinfect is misspelled.

4:57 pm

Dara Navaei

DEN-16773 removed unused code in the airpump.c

Options
    • -13
    • +0
    ./AirPump.c
3:40 pm

Vinayakam Mani

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

3:32 pm

Dara Navaei

DEN-16773 addressed a code review comment

Options
    • -7
    • +6
    ./AirPump.c
3:26 pm

Dara Navaei

Done

Done

3:12 pm

Vinayakam Mani

Can we remove the AIR_PUMP_OPERATION_TIMEOUT Macro?

Can we remove the AIR_PUMP_OPERATION_TIMEOUT Macro?

2:40 pm

Dara Navaei

2:38 pm

Dara Navaei

DEN-16773 removed alarm 318

Options
    • -24
    • +0
    ./AirPump.c
2:22 pm

Dara Navaei

HD-DEN-16773_Add Chemical Disinfect Enable TO The Institutional Record
HD-DEN-16773_Add Chemical Disinfect Enable TO The Institutional Record