application / sources / canbus

FrameInterface.h

Clone Tools

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 10 Sep
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
Thursday 30 Jan
8:28 pm

Dara Navaei

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

Options
    • -1
    • +1
    ./FrameInterface.h
  2. … 619 more files in changeset.
Friday 15 Nov 2024
12:54 pm

Dara Navaei

Friday 07 Jun 2024
9:05 am

Dara Navaei

9:04 am

Dara Navaei

UI-DEN-SUNRISE
UI-DEN-SUNRISE
Wednesday 24 Jan 2024
2:28 pm

Dara Navaei

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

Options
    • -1
    • +1
    ./FrameInterface.h
  2. … 612 more files in changeset.
Thursday 19 Oct 2023
8:26 pm

pmontazemi  (deleted user)

Thursday 03 Aug 2023
2:23 pm

Sean Nash

2:05 pm

Dara Navaei

Wednesday 02 Aug 2023
9:33 am

Sean Nash

Monday 08 May 2023
8:49 am

Dara Navaei

UI Staging Unreviewed Commits
UI Staging Unreviewed Commits
Friday 10 Feb 2023
10:12 am

Dara Navaei

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

Options
    • -1
    • +1
    ./FrameInterface.h
  2. … 50 more files in changeset.
9:23 am

Behrouz NematiPour

9:23 am

Michael Garthwaite , vduong  (deleted user) and jishii  (deleted user)

9:17 am

vduong  (deleted user)

Ok, sounds good. RESOLVED

Ok, sounds good.

RESOLVED

8:17 am

Behrouz NematiPour

Good point, I agree with the two intervals, it would be clarifying. For the counter, though, it might not be always the case. I will start doing it in my next task, since I would like to change all...

Good point,
I agree with the two intervals, it would be clarifying.
For the counter, though, it might not be always the case.
I will start doing it in my next task, since I would like to change all, to be consistent across the entire code base.

7:40 am

vduong  (deleted user)

Only a suggestion: It is a good idea to append the unit to the end of the variable name especially for time. It helps improve readability and clarity. For instance, here, we have ms and the next o...

Only a suggestion:
It is a good idea to append the unit to the end of the variable name especially for time. It helps improve readability and clarity.

For instance, here, we have ms and the next one, _pendingInterval is in seconds. and I believe _pendingCounter is in minutes based on the .cpp file's comment.

_interval_ms
_pendingInterval_sec
_pendingCounter_min

7:16 am

vduong  (deleted user)

RESOLVED

RESOLVED

7:12 am

Behrouz NematiPour

Thanks for the feedback. Fixed.

Thanks for the feedback.
Fixed.

7:03 am

vduong  (deleted user)

The variable, destination, itself is a QString, you don't need to do QString("%1").arg(destination), you can pass destination in directly.

The variable, destination, itself is a QString, you don't need to do QString("%1").arg(destination), you can pass destination in directly.

Tuesday 31 Jan 2023
11:22 pm

Behrouz NematiPour

Saturday 21 Jan 2023
7:43 pm

Behrouz NematiPour

DEN-14524 - Fixed CppCheck

- Fixed CppCheck error for Qt Macros

Options
    • -4
    • +3
    ./FrameInterface.h
  2. … 4 more files in changeset.
Tuesday 17 Jan 2023
9:01 am

Dara Navaei

UI-DEN-14517_UI BN S88
UI-DEN-14517_UI BN S88
Wednesday 04 Jan 2023
4:48 pm

Dara Navaei

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

Options
    • -1
    • +1
    ./FrameInterface.h
  2. … 608 more files in changeset.
Thursday 08 Sep 2022
3:32 pm

Behrouz NematiPour

3:30 pm

Tiffany Mejia

Tuesday 06 Sep 2022
11:20 am

Michael Garthwaite

Tuesday 12 Jul 2022
11:52 am

Sean Nash

11:51 am

Sean Nash

RESOLVED.

RESOLVED.

Monday 11 Jul 2022
10:01 am

Behrouz NematiPour

Sure Sean, if that is the decision, will do it soon and create a case for that. I also believe some other CL Switches need to be removed, which were only implemented for our testing at the time (CA...

Sure Sean, if that is the decision, will do it soon and create a case for that.
I also believe some other CL Switches need to be removed, which were only implemented for our testing at the time (CANBus).

Issue DEN-13441 - MR: Make the Dialin log permanent with no switch [CL switches review/removal] has been successfully created

Wednesday 06 Jul 2022
9:35 am

Sean Nash

If this is to log Dialin messages, we should always do that (no switch). If a Dialin command is given/responded to, we definitely want to know about that - so want to see it in the log. Dialin comm...

If this is to log Dialin messages, we should always do that (no switch). If a Dialin command is given/responded to, we definitely want to know about that - so want to see it in the log. Dialin commands are always one-shot type messages, so will not impact log size.