td-firmware / firmware

.cproject

Clone Tools

Stats

Commits this week: 1
Total Committers: 3
Last Commit: 04 Feb
Commits this week: 1

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
Today
13 mins ago

Vendor - TEL - Arpita Srivastava

add typedef

add typedef

13 mins ago

Vendor - TEL - Arpita Srivastava

add typedef

add typedef

14 mins ago

Vendor - TEL - Arpita Srivastava

We can delete line 45 and 46. And update line 44 to } TUBE_SET_INSTALL_STATE_T;

We can delete line 45 and 46. And update line 44 to } TUBE_SET_INSTALL_STATE_T;

15 mins ago

Vendor - TEL - Arpita Srivastava

We can delete line 46 and 47 and update line 45 to } TUBE_SET_AUTO_EJECT_STATE_T;

We can delete line 46 and 47 and update line 45 to } TUBE_SET_AUTO_EJECT_STATE_T;

20 mins ago

Vendor - TEL - Arpita Srivastava

add a space between message and )

add a space between message and )

24 mins ago

Vendor - TEL - Arpita Srivastava

Change // to /// for doxygen to catch it.

Change // to /// for doxygen to catch it.

40 mins ago

Vendor - TEL - Arpita Srivastava

Can we remove the parameter and switch directly over currentPostTreatmentState?

Can we remove the parameter and switch directly over currentPostTreatmentState?

50 mins ago

Vendor - TEL - Arpita Srivastava

The public function name signalGoToInstallState is not sufficiently descriptive. Would renaming it to requestTubeSetInstall() be appropriate?

The public function name signalGoToInstallState is not sufficiently descriptive. Would renaming it to requestTubeSetInstall() be appropriate?

1 hr ago

Sean Nash

I prefer you create a local BOOL result so you can have a simpler return statement.

I prefer you create a local BOOL result so you can have a simpler return statement.

1 hr ago

Sean Nash

Should go to your new complete state here.

Should go to your new complete state here.

1 hr ago

Vendor - TEL - Arpita Srivastava

The number looks incorrect. This was an issue in the staging branch, which is fixed now. Can you please pull latest staging and update the values of your enums?

The number looks incorrect. This was an issue in the staging branch, which is fixed now. Can you please pull latest staging and update the values of your enums?

Monday 04 May
4:30 pm

Sean Nash

check alarm status instead of switch state.

check alarm status instead of switch state.

Sunday 03 May
11:44 pm

Sean Nash

11:43 pm

Sean Nash

Is the folder name necessary?

Is the folder name necessary?

11:42 pm

Sean Nash

Remove unnecessary spaces before =

Remove unnecessary spaces before =

11:41 pm

Sean Nash

Align =

Align =

11:41 pm

Sean Nash

Align =.

Align =.

11:41 pm

Sean Nash

Align =.

Align =.

11:40 pm

Sean Nash

We don't have to align =, but if you're going to have all of these spaces before the =, you should align.

We don't have to align =, but if you're going to have all of these spaces before the =, you should align.

11:38 pm

Sean Nash

Why are we doing this? Having a return here is against our coding standards (it's like a goto).

Why are we doing this?
Having a return here is against our coding standards (it's like a goto).

11:37 pm

Sean Nash

Describe what kind of services are being provided.

Describe what kind of services are being provided.

11:36 pm

Sean Nash

Consider separating the wait for door close to another state.

Consider separating the wait for door close to another state.

11:32 pm

Sean Nash

Why are we doing this?

Why are we doing this?

11:31 pm

Sean Nash

Describe what kind of services being provided.

Describe what kind of services being provided.

11:29 pm

Sean Nash

Should not change states in a signal function. Should set a request flag that gets looked at in dry self-tests state handler function which would return install state if the flag is set.

Should not change states in a signal function. Should set a request flag that gets looked at in dry self-tests state handler function which would return install state if the flag is set.

Sunday 26 Apr
1:32 pm

Sean Nash

Remove blank line.

Remove blank line.

1:31 pm

Sean Nash

Move this transition function call to the transitionToPostTreatmentMode function below.

Move this transition function call to the transitionToPostTreatmentMode function below.

1:29 pm

Sean Nash

Invalid payload length shouldn't be a reject reason, it should cause a s/w fault (because it suggests the UI and TD software are not compatible versions).

Invalid payload length shouldn't be a reject reason, it should cause a s/w fault (because it suggests the UI and TD software are not compatible versions).

Wednesday 22 Apr
2:05 pm

Sean Nash

Not necessary

Not necessary

2:05 pm

Sean Nash

This command should be given once on way out of prior state instead of every 50ms while in this state. We also need to zero the timer counter at same time so the timeout check below works.

This command should be given once on way out of prior state instead of every 50ms while in this state.
We also need to zero the timer counter at same time so the timeout check below works.