td-firmware / firmware / App / Services

SystemCommTD.c

Clone Tools

Stats

Commits this week: 2
Total Committers: 5
Last Commit: 12 Jun
Commits this week: 2

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 12 Jun
11:04 pm

Sean Nash

Arpita, we don't want to use doxygen style comments for function prototypes since we cover functions with the function headers. so this is the proper type of comments.

Arpita, we don't want to use doxygen style comments for function prototypes since we cover functions with the function headers. so this is the proper type of comments.

3:50 pm

Vendor - TEL - Arpita Srivastava

pauseBolusResumeState is input

pauseBolusResumeState is input

3:47 pm

Vendor - TEL - Arpita Srivastava

add alarm

add alarm

3:46 pm

Vendor - TEL - Arpita Srivastava

Replace // to ///< for line 56 to 60

Replace // to ///< for line 56 to 60

3:27 pm

Sean Nash

3:26 pm

Sean Nash

3:19 pm

Vendor - TEL - Praneeth Bunne

LDT-4103-1: Addressed CR comments

Options
    • -140
    • +140
    ./SystemCommTD.c
  2. … 2 more files in changeset.
3:06 pm

Sean Nash

Use // comments for function prototypes.

Use // comments for function prototypes.

3:06 pm

Sean Nash

Either no comment or // comment for function prototypes since doxygen captures function details from function headers.

Either no comment or // comment for function prototypes since doxygen captures function details from function headers.

3:05 pm

Sean Nash

These were fine (and preferred) as // because doxygen gets function details from function headers in the .c file.

These were fine (and preferred) as // because doxygen gets function details from function headers in the .c file.

9:32 am

Sean Nash

Should we be looking at tubing set type instead of modality here?

Should we be looking at tubing set type instead of modality here?

9:28 am

Sean Nash

Can we not determine this now? Why TODO?

Can we not determine this now? Why TODO?

9:27 am

Sean Nash

Shouldn't we be checking tubing set type instead of modality here?

Shouldn't we be checking tubing set type instead of modality here?

9:23 am

Sean Nash

verify given type is valid (< NUM_OF_TUBE_SET_TYPES) and trigger s/w fault if invalid.

verify given type is valid (< NUM_OF_TUBE_SET_TYPES) and trigger s/w fault if invalid.

9:22 am

Sean Nash

Initialize to UNKNOWN type. Remove TODO.

Initialize to UNKNOWN type. Remove TODO.

9:21 am

Sean Nash

Consider adding TUBE_SET_TYPE_UNKNOWN = 0 as first enum so you can initialize a type variable to something before we install/identify the type.

Consider adding TUBE_SET_TYPE_UNKNOWN = 0 as first enum so you can initialize a type variable to something before we install/identify the type.

9:18 am

Sean Nash

Use /// when above, use ///< when to the right. Also remove extra space before Tube

Use /// when above, use ///< when to the right.
Also remove extra space before Tube

9:17 am

Sean Nash

Change // to ///<

Change // to ///<

4:02 am

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-4013_Fluid Bolus - FW Implementation - 2/4:
LEAHI-TD-FIRMWARE-LDT-4013_Fluid Bolus - FW Implementation - 2/4:
Thursday 11 Jun
11:20 am

Vendor - TEL - Arpita Srivastava

please add blank space between ( (int)postState ), TRUE );

please add blank space between ( (int)postState ), TRUE );

11:20 am

Vendor - TEL - Arpita Srivastava

please add blank space between ( (int)postState ), TRUE );

please add blank space between ( (int)postState ), TRUE );

11:13 am

Vendor - TEL - Arpita Srivastava

Alarm id should be 196? Description looks same as 194.

Alarm id should be 196?
Description looks same as 194.

Tuesday 09 Jun
4:44 pm

Sean Nash

Change PRES_OCCL to PRESSURE

Change PRES_OCCL to PRESSURE

4:43 pm

Sean Nash

Maybe move OperationModes.c?

Maybe move OperationModes.c?

4:42 pm

Sean Nash

Use generic confirmation functions/messages.

Use generic confirmation functions/messages.

4:41 pm

Sean Nash

Move this function to OperationModes.c.

Move this function to OperationModes.c.

4:40 pm

Sean Nash

From here forward, we can just call sendMessage() with appropriate params.

From here forward, we can just call sendMessage() with appropriate params.

4:37 pm

Sean Nash

Use generic helper function (sendMessage) in MessageSupport.c.

Use generic helper function (sendMessage) in MessageSupport.c.

4:33 pm

Sean Nash

Use generic confirmation functions/messages for off button.

Use generic confirmation functions/messages for off button.

4:26 pm

Sean Nash

Remove change.

Remove change.