Fisheye on application/sources/canbus

application (bugfix/DEN-16597-UI-BN-S114): Merge branch 'master' into DEN-1396_LogHandler

Behrouz NematiPour — Wed, 15 Jan 2020 18:30:58 -0800

Behrouz NematiPour committed c6a0989 on branch master, 2020_EOY_DEMO and 275 more in application

Merge branch 'master' into DEN-1396_LogHandler

· /main.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/applicationcontroller.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/applicationcontroller.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/caninterface.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/caninterface.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/frameinterface.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/frameinterface.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messageglobals.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messageinterpreter.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messageinterpreter.h

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guicontroller.cpp

(-1, +1)

| History

| Source

| Diff

… 22 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): Revision 56d00a8, line 98...

Behrouz NematiPour — Tue, 14 Jan 2020 13:56:02 -0800

Behrouz NematiPour committed 38e28c4 on branch master, 2020_EOY_DEMO and 275 more in application

Revision 56d00a8, line 98

- Added space between functions.

· /sources/canbus/caninterface.h

(-0, +2)

| History

| Source

| Diff

application (bugfix/DEN-16597-UI-BN-S114): Updated the Copyright with the current year and replaced tabs with 4 spaces

Dara Navaei — Thu, 09 Jan 2020 11:17:31 -0800

Dara Navaei committed 7cb4ad8 on branch master, 2020_EOY_DEMO and 275 more in application

Updated the Copyright with the current year and replaced tabs with 4 spaces

· /sources/canbus/frameinterface.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/frameinterface.h	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/messagebuilder.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/messagebuilder.h	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/messagedispatcher.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/messagedispatcher.h	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/messageglobals.h	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.h	(-1, +1)	\| History	\| Source	\| Diff
· /sources/utility/crc.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/utility/format.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/utility/format.h	(-1, +1)	\| History	\| Source	\| Diff
· /sources/utility/types.h	(-1, +1)	\| History	\| Source	\| Diff
· /unittests/tst_canbus.cpp	(-1, +1)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): Merge branch 'master' into DEN-1090_MessageHandler

Behrouz NematiPour — Thu, 09 Jan 2020 11:08:09 -0800

Behrouz NematiPour committed c933552 on branch master, 2020_EOY_DEMO and 276 more in application

Merge branch 'master' into DEN-1090_MessageHandler

· /main.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/applicationcontroller.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/applicationcontroller.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/caninterface.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/caninterface.h

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guicontroller.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guicontroller.h

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guiglobals.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guiglobals.h

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guiview.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guiview.h

(-1, +1)

| History

| Source

| Diff

· /sources/gui/qml/components/ModalDialog.qml

(-1, +1)

| History

| Source

| Diff

· /sources/gui/qml/components/TouchRect.qml

(-1, +1)

| History

| Source

| Diff

· /sources/gui/qml/main.qml

(-1, +1)

| History

| Source

| Diff

· /sources/gui/qml/pages/SettingsHome.qml

(-1, +1)

| History

| Source

| Diff

… 5 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): DEN-1617 : Implementation - Alarm Logging and Acknowledgment...

Behrouz NematiPour — Tue, 07 Jan 2020 16:54:36 -0800

Behrouz NematiPour committed 862dc05 on branch master, 2020_EOY_DEMO and 275 more in application

DEN-1617 : Implementation - Alarm Logging and Acknowledgment

- Ooooooopppps !

- Forgot a not in some classes.

· /sources/applicationcontroller.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/frameinterface.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/messagedispatcher.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/gui/guicontroller.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/storage/usbwatcher.cpp	(-2, +3)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1617 : Implementation - Alarm Logging and Acknowledgment...

Behrouz NematiPour — Tue, 07 Jan 2020 13:31:16 -0800

Behrouz NematiPour committed 56d00a8 on branch master, 2020_EOY_DEMO and 275 more in application

DEN-1617 : Implementation - Alarm Logging and Acknowledgment

- Added threading to other classes (Application / Gui controller, ...) and restructured the threading to meed the Qt limitation.

- Documented the code

- Ran cppCheck again

- Cleaned the includes and using namespace (They have left after so many changes and some were useless.)

- Added _init flag for init methods so not to be able to call them twice.

- Added another init method with the thread parameter to be able to pass the thread of the class and also be able to use the class without any thread for unit tests.

- Created a Threads namespace to gather all the threads in one place.

- Update the unit tests as well. [to make it work with the new structure/threading.]

· /cppcheck.err

(-6, +4)

| History

| Source

| Diff

· /cppcheck.log

(-18, +18)

| History

| Source

| Diff

· /denali.pro

(-3, +2)

| History

| Source

| Diff

· /main.cpp

(-15, +19)

| History

| Source

| Diff

· /sources/applicationcontroller.cpp

(-3, +88)

| History

| Source

| Diff

· /sources/applicationcontroller.h

(-8, +12)

| History

| Source

| Diff

· /sources/canbus/alarminterpreter.cpp

(-19, +0)

| History

| Source

| Diff

· /sources/canbus/alarminterpreter.h

(-41, +0)

| History

| Source

| Diff

· /sources/canbus/caninterface.cpp

(-29, +60)

| History

| Source

| Diff

· /sources/canbus/caninterface.h

(-7, +18)

| History

| Source

| Diff

· /sources/canbus/frameinterface.cpp

(-21, +56)

| History

| Source

| Diff

· /sources/canbus/frameinterface.h

(-9, +16)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.cpp

(-7, +6)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.h

(-18, +17)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.cpp

(-17, +56)

| History

| Source

| Diff

… 21 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): DEN-1617 : Implementation - Alarm Logging and Acknowledgment...

Behrouz NematiPour — Sun, 05 Jan 2020 22:17:50 -0800

Behrouz NematiPour committed e160521 on branch master, 2020_EOY_DEMO and 275 more in application

DEN-1617 : Implementation - Alarm Logging and Acknowledgment

- Added Threading for :

. Can Frame

. Can Message

. USB Watcher

- Changed the File Handler implementation

- Changed the Singleton from pointer to static reference

// TODO :

- We still need to work on threading on other classes

- We need to have a singleton parent class

- Some code has been added to debug can interface (We still have swap frames)

- Thread Objects need to be gathered all together in one place to be handled easily.

· /denali.pro

(-0, +1)

| History

| Source

| Diff

· /main.cpp

(-13, +13)

| History

| Source

| Diff

· /sources/applicationcontroller.cpp

(-17, +12)

| History

| Source

| Diff

· /sources/applicationcontroller.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/caninterface.cpp

(-38, +84)

| History

| Source

| Diff

· /sources/canbus/caninterface.h

(-1, +6)

| History

| Source

| Diff

· /sources/canbus/frameinterface.cpp

(-8, +30)

| History

| Source

| Diff

· /sources/canbus/frameinterface.h

(-2, +3)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.cpp

(-10, +38)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.h

(-2, +3)

| History

| Source

| Diff

· /sources/canbus/messageglobals.cpp

(-0, +20)

| History

| Source

· /sources/canbus/messageglobals.h

(-0, +5)

| History

| Source

| Diff

· /sources/gui/guicontroller.cpp

(-10, +6)

| History

| Source

| Diff

· /sources/gui/guicontroller.h

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guiview.cpp

(-12, +12)

| History

| Source

| Diff

… 7 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): DEN-1617 : Implementation - Alarm Logging and Acknowledgment...

Behrouz NematiPour — Thu, 02 Jan 2020 22:37:51 -0800

Behrouz NematiPour committed feb3423 on branch master, 2020_EOY_DEMO and 275 more in application

DEN-1617 : Implementation - Alarm Logging and Acknowledgment

- Alarm Logging has been added

- Some other changes has been done to the code and messaging.

- Acknowledgment is not done [ Had an agreement with Sean for design but needs more time and we need to do it all together.]

· /denali.pro

(-1, +8)

| History

| Source

| Diff

· /main.cpp

(-17, +11)

| History

| Source

| Diff

· /scripts/run.sh

(-4, +9)

| History

| Source

| Diff

· /sources/abstract/singleton.h

(-0, +95)

| History

| Source

· /sources/applicationcontroller.cpp

(-4, +5)

| History

| Source

| Diff

· /sources/applicationcontroller.h

(-1, +6)

| History

| Source

| Diff

· /sources/canbus/alarminterpreter.cpp

(-0, +19)

| History

| Source

· /sources/canbus/alarminterpreter.h

(-0, +41)

| History

| Source

· /sources/canbus/caninterface.cpp

(-0, +4)

| History

| Source

| Diff

· /sources/canbus/frameinterface.cpp

(-0, +1)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.cpp

(-0, +1)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.cpp

(-0, +1)

| History

| Source

| Diff

· /sources/canbus/messageglobals.h

(-4, +5)

| History

| Source

| Diff

· /sources/canbus/messageinterpreter.cpp

(-116, +242)

| History

| Source

| Diff

· /sources/canbus/messageinterpreter.h

(-10, +30)

| History

| Source

| Diff

… 22 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): Updated the Copyright with the current year and replaced tabs with 4 spaces

Dara Navaei — Wed, 01 Jan 2020 00:00:03 -0800

Dara Navaei committed 62efc6d on branch master, 2020_EOY_DEMO and 276 more in application

Updated the Copyright with the current year and replaced tabs with 4 spaces

· /main.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/applicationcontroller.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/applicationcontroller.h

(-1, +1)

| History

| Source

| Diff

· /sources/applicationpost.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/applicationpost.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/caninterface.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/caninterface.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messagehandler.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messagehandler.h

(-1, +1)

| History

| Source

| Diff

· /sources/configuration/display.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/configuration/display.h

(-1, +1)

| History

| Source

| Diff

· /sources/configuration/sound.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/configuration/sound.h

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guiactions.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guiactions.h

(-1, +1)

| History

| Source

| Diff

… 38 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Thu, 26 Dec 2019 20:40:57 -0800

Behrouz NematiPour committed 188d761 on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Incomplete merge of the added feature for First parameter of the BloodFlow has to be unsigned int and changed to S32.

- Updated the unit tests.

· /denali.pro	(-1, +2)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.cpp	(-1, +11)	\| History	\| Source	\| Diff
· /sources/gui/guiglobals.h	(-2, +11)	\| History	\| Source	\| Diff
· /sources/gui/qml/pages/TreatmentStart.qml	(-0, +8)	\| History	\| Source	\| Diff
· /sources/utility/types.cpp	(-0, +30)	\| History	\| Source	\| Diff
· /sources/utility/types.h	(-1, +16)	\| History	\| Source	\| Diff
· /unittests/tst_canbus.cpp	(-12, +21)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): Merge branch 'DEN-1090_MessageHandler' into DEN-1396_LogHandler...

Behrouz NematiPour — Thu, 26 Dec 2019 16:17:36 -0800

Behrouz NematiPour committed 8c5f54b on branch master, 2020_EOY_DEMO and 275 more in application

Merge branch 'DEN-1090_MessageHandler' into DEN-1396_LogHandler

- Added the changes to the BloodFlow first parameter form unsigned to signed to be updated.

· /sources/canbus/messageinterpreter.cpp	(-3, +3)	\| History	\| Source	\| Diff
· /sources/gui/qml/pages/TreatmentStart.qml	(-2, +2)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Thu, 26 Dec 2019 16:02:35 -0800

Behrouz NematiPour committed 12f922a on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- First parameter of the BloodFlow has to be unsigned int and changed to S32.

· /sources/canbus/messageinterpreter.cpp	(-3, +7)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.h	(-4, +4)	\| History	\| Source	\| Diff
· /sources/gui/qml/pages/TreatmentStart.qml	(-7, +8)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1396 : UI Log Handler...

Behrouz NematiPour — Thu, 26 Dec 2019 15:52:12 -0800

Behrouz NematiPour committed 5963f00 on branch master, 2020_EOY_DEMO and 275 more in application

DEN-1396 : UI Log Handler

MISC :

- Updated the Singleton defines to delete the instance.

- Added/moved the floatCompare in the types.cpp and created the file to do so.

LOGGING :

- Implemented logging

- Added LOG_EVENT / LOG_DATUM / LOG_ERROR in code instead of qDebug() as sample of the log event .

- Added export to USB feature.

- Added "Export Button"

· /denali.pro

(-1, +2)

| History

| Source

| Diff

· /denali.qrc

(-0, +1)

| History

| Source

| Diff

· /main.cpp

(-0, +21)

| History

| Source

| Diff

· /sources/applicationcontroller.cpp

(-0, +9)

| History

| Source

| Diff

· /sources/applicationcontroller.h

(-0, +2)

| History

| Source

| Diff

· /sources/canbus/caninterface.cpp

(-2, +6)

| History

| Source

| Diff

· /sources/canbus/frameinterface.cpp

(-3, +4)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.cpp

(-6, +7)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.cpp

(-2, +3)

| History

| Source

| Diff

· /sources/canbus/messageglobals.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messageinterpreter.cpp

(-4, +15)

| History

| Source

| Diff

· /sources/gui/guicontroller.cpp

(-0, +5)

| History

| Source

| Diff

· /sources/gui/guicontroller.h

(-0, +3)

| History

| Source

| Diff

· /sources/gui/guiglobals.cpp

(-1, +2)

| History

| Source

| Diff

· /sources/gui/guiview.cpp

(-0, +9)

| History

| Source

| Diff

… 11 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): DEN-1434 : UI In-Treatment Screen (Monitoring)...

Behrouz NematiPour — Fri, 20 Dec 2019 17:58:40 -0800

Behrouz NematiPour committed 2fdeb1c on branch master, 2020_EOY_DEMO and 275 more in application

DEN-1434 : UI In-Treatment Screen (Monitoring)

- Added "PWM Duty Cycle" parameter to BloodFlow Monitoring command

· /main.cpp	(-3, +7)	\| History	\| Source	\| Diff
· /sources/canbus/messageglobals.h	(-4, +4)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.cpp	(-2, +17)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.h	(-4, +4)	\| History	\| Source	\| Diff
· /sources/gui/guiglobals.h	(-3, +11)	\| History	\| Source	\| Diff
· /sources/gui/qml/pages/TreatmentStart.qml	(-6, +15)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Thu, 19 Dec 2019 15:00:47 -0800

Behrouz NematiPour committed 9552cc6 on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Fixed a typo

· /sources/canbus/frameinterface.cpp

(-1, +1)

| History

| Source

| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Thu, 19 Dec 2019 14:19:42 -0800

Behrouz NematiPour committed 48379ed on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Fixed Comment Spacing [Code Review]

· /sources/canbus/messagebuilder.cpp

(-5, +5)

| History

| Source

| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Thu, 19 Dec 2019 14:08:24 -0800

Behrouz NematiPour committed 9392f5b on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Updated the constants by enums [Code Review]

· /sources/canbus/frameinterface.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/messagebuilder.cpp	(-6, +6)	\| History	\| Source	\| Diff
· /sources/canbus/messageglobals.h	(-0, +5)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.cpp	(-1, +1)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Thu, 19 Dec 2019 03:37:24 -0800

Behrouz NematiPour committed 5610552 on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Trying a solution to have separate unit tests for each feature.

- Changed the unittests class to tst_canbus since this is the unit test for implemented canbus messages and interaction.

- removed .pro.user from gitignore since denali.pro.user contains configurations for local unit tests setup and need to be cloned for the new ones.

- Fixed issues which has been found during the unit tests.

· /.gitignore	(-1, +0)	\| History	\| Source	\| Diff
· /denali.pro	(-7, +9)	\| History	\| Source	\| Diff
· /denali.pro.user	(-0, +737)	\| History	\| Source
· /main.cpp	(-2, +2)	\| History	\| Source	\| Diff
· /sources/canbus/caninterface.cpp	(-0, +1)	\| History	\| Source	\| Diff
· /sources/canbus/caninterface.h	(-2, +2)	\| History	\| Source	\| Diff
· /sources/canbus/frameinterface.h	(-2, +2)	\| History	\| Source	\| Diff
· /sources/canbus/messagebuilder.cpp	(-7, +7)	\| History	\| Source	\| Diff
· /sources/canbus/messagedispatcher.h	(-0, +6)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.cpp	(-0, +1)	\| History	\| Source	\| Diff
· /sources/utility/format.cpp	(-0, +7)	\| History	\| Source	\| Diff
· /sources/utility/format.h	(-3, +3)	\| History	\| Source	\| Diff
· /unittests/tst_canbus.cpp	(-0, +277)	\| History	\| Source
· /unittests/tst_canbus.h	(-0, +62)	\| History	\| Source
· /unittests/unittests.cpp	(-136, +0)	\| History	\| Source	\| Diff
· /unittests/unittests.h	(-51, +0)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Wed, 18 Dec 2019 21:24:31 -0800

Behrouz NematiPour committed d3f9838 on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- renamed MessageHandler to FrameInterface to understand easier in SDD.

· /denali.pro	(-2, +2)	\| History	\| Source	\| Diff
· /main.cpp	(-5, +5)	\| History	\| Source	\| Diff
· /sources/canbus/caninterface.cpp	(-2, +2)	\| History	\| Source	\| Diff
· /sources/canbus/frameinterface.cpp	(-0, +168)	\| History	\| Source
· /sources/canbus/frameinterface.h	(-0, +102)	\| History	\| Source
· /sources/canbus/messagedispatcher.cpp	(-3, +3)	\| History	\| Source	\| Diff
· /sources/canbus/messagehandler.cpp	(-168, +0)	\| History	\| Source	\| Diff
· /sources/canbus/messagehandler.h	(-101, +0)	\| History	\| Source	\| Diff
· /unittests/unittests.cpp	(-20, +20)	\| History	\| Source	\| Diff
· /unittests/unittests.h	(-8, +8)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Wed, 18 Dec 2019 13:36:13 -0800

Behrouz NematiPour committed 4b96196 on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- The keepAlive implementation was against the design.

· /sources/applicationcontroller.cpp	(-0, +29)	\| History	\| Source	\| Diff
· /sources/applicationcontroller.h	(-0, +3)	\| History	\| Source	\| Diff
· /sources/canbus/messagedispatcher.cpp	(-29, +0)	\| History	\| Source	\| Diff
· /sources/canbus/messagedispatcher.h	(-3, +0)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Wed, 18 Dec 2019 01:02:29 -0800

Behrouz NematiPour committed b5c381d on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Added code documentation for the message dispatcher.

· /sources/canbus/messagedispatcher.cpp	(-2, +37)	\| History	\| Source	\| Diff
· /sources/canbus/messagedispatcher.h	(-47, +61)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Tue, 17 Dec 2019 20:30:01 -0800

Behrouz NematiPour committed c538964 on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Documented the code

- Added gcc flag to use all the warning as errors and don't let the application compile.

- Modified the build frame structure to return bool true as of successful build message and moved the frame list into the function parameter list.

This way by using __attribute_warn_unused_result__ we can force the code who uses this method to check for the return value of successful build.

- Same has been done for some other methods.

· /denali.pro	(-1, +1)	\| History	\| Source	\| Diff
· /main.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/caninterface.cpp	(-11, +11)	\| History	\| Source	\| Diff
· /sources/canbus/caninterface.h	(-6, +6)	\| History	\| Source	\| Diff
· /sources/canbus/messagebuilder.cpp	(-26, +187)	\| History	\| Source	\| Diff
· /sources/canbus/messagebuilder.h	(-7, +7)	\| History	\| Source	\| Diff
· /sources/canbus/messagedispatcher.cpp	(-2, +6)	\| History	\| Source	\| Diff
· /sources/canbus/messagedispatcher.h	(-1, +1)	\| History	\| Source	\| Diff
· /sources/canbus/messageglobals.h	(-10, +11)	\| History	\| Source	\| Diff
· /sources/canbus/messagehandler.cpp	(-27, +27)	\| History	\| Source	\| Diff
· /sources/canbus/messagehandler.h	(-4, +4)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.cpp	(-56, +56)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.h	(-6, +7)	\| History	\| Source	\| Diff
· /sources/gui/guiglobals.cpp	(-1, +1)	\| History	\| Source	\| Diff
· /sources/utility/format.cpp	(-2, +2)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Mon, 16 Dec 2019 21:19:58 -0800

Behrouz NematiPour committed fee7fab on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Fixed application close which wasn't closing Can Device connection.

- Changed to date to 2019 in copyright section.

- In MessageInterpreter, named getBloodFlowData method parameters to the meaningful data names.

- Moved fromVariant method to Format utility class.

- Added Code documentation and description.

· /main.cpp

(-2, +6)

| History

| Source

| Diff

· /sources/applicationcontroller.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/applicationcontroller.h

(-1, +1)

| History

| Source

| Diff

· /sources/applicationpost.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/applicationpost.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/caninterface.cpp

(-39, +46)

| History

| Source

| Diff

· /sources/canbus/caninterface.h

(-7, +26)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messageglobals.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messagehandler.cpp

(-9, +37)

| History

| Source

| Diff

· /sources/canbus/messagehandler.h

(-12, +38)

| History

| Source

| Diff

· /sources/canbus/messageinterpreter.cpp

(-45, +118)

| History

| Source

| Diff

… 47 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Mon, 16 Dec 2019 14:22:59 -0800

Behrouz NematiPour committed 4c1551d on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Moved toHexString to a place more general so can be used everywhere.

- added command line switches to enable the can messages debugging easier.

- Added an Unhandled console out message.

- Added two other alarm message to the registered list but will console out the Unhandled message.

· /denali.pro	(-3, +5)	\| History	\| Source	\| Diff
· /main.cpp	(-2, +14)	\| History	\| Source	\| Diff
· /sources/canbus/messagebuilder.cpp	(-2, +3)	\| History	\| Source	\| Diff
· /sources/canbus/messagehandler.cpp	(-1, +3)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.cpp	(-10, +26)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.h	(-0, +1)	\| History	\| Source	\| Diff
· /sources/gui/guiglobals.cpp	(-9, +0)	\| History	\| Source	\| Diff
· /sources/gui/guiglobals.h	(-7, +8)	\| History	\| Source	\| Diff
· /sources/utility/format.cpp	(-0, +27)	\| History	\| Source
· /sources/utility/format.h	(-0, +28)	\| History	\| Source

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Mon, 16 Dec 2019 03:26:37 -0800

Behrouz NematiPour committed 460df09 on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Added CRC to the messages [added to the sent message and checked for the received message]

- Added ability to to enable/disable crc and keepAlive from project during build

- Added a feature to write the can messages in color formatted to the console in Can interface and Message Handler for easier debugging. Which can also be disabled by code.

- Added Animation to the TouchRect Touch to better understand if it has been touch.

- Added a demo/sample data text to display the corrent Blood Flow values on the StartTreatment Screen.

· /denali.pro

(-3, +20)

| History

| Source

| Diff

· /main.cpp

(-0, +2)

| History

| Source

| Diff

· /sources/canbus/caninterface.cpp

(-15, +13)

| History

| Source

| Diff

· /sources/canbus/caninterface.h

(-5, +5)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.cpp

(-26, +85)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.h

(-12, +21)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.cpp

(-19, +3)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.h

(-0, +1)

| History

| Source

| Diff

· /sources/canbus/messageglobals.h

(-15, +17)

| History

| Source

| Diff

· /sources/canbus/messageinterpreter.cpp

(-20, +130)

| History

| Source

| Diff

· /sources/canbus/messageinterpreter.h

(-1, +8)

| History

| Source

| Diff

· /sources/gui/guiglobals.cpp

(-0, +9)

| History

| Source

| Diff

· /sources/gui/guiglobals.h

(-1, +4)

| History

| Source

| Diff

· /sources/gui/qml/components/TouchRect.qml

(-0, +13)

| History

| Source

| Diff

· /sources/gui/qml/pages/SettingsHome.qml

(-0, +1)

| History

| Source

| Diff

… 4 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Fri, 13 Dec 2019 01:30:23 -0800

Behrouz NematiPour committed 1732e83 on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Modified enum to be enum class for clarification of the code.

- Added Message Interpreter

- Updated Message Builder to check CRC as a placeholder.

- Fixed Message Builder for messages with exactly 255 data length.

- Updated Message Builder methods scope for clarification of the code.

- Updated Message (Model) isComplete method.

- Updated Message Dispatcher to accept only command which has been defined for channel.

- Updated Message Dispatcher to remove the message from model when it has been transmitted.

· /denali.pro

(-18, +18)

| History

| Source

| Diff

· /main.cpp

(-0, +4)

| History

| Source

| Diff

· /sources/applicationcontroller.cpp

(-3, +3)

| History

| Source

| Diff

· /sources/canbus/caninterface.cpp

(-20, +38)

| History

| Source

| Diff

· /sources/canbus/caninterface.h

(-11, +14)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.cpp

(-41, +52)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.h

(-35, +35)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.cpp

(-1, +115)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.h

(-7, +36)

| History

| Source

| Diff

· /sources/canbus/messageglobals.h

(-10, +16)

| History

| Source

| Diff

· /sources/canbus/messagehandler.cpp

(-82, +48)

| History

| Source

| Diff

· /sources/canbus/messagehandler.h

(-14, +14)

| History

| Source

| Diff

· /sources/canbus/messageinterface.cpp

(-21, +0)

| History

| Source

| Diff

· /sources/canbus/messageinterface.h

(-37, +0)

| History

| Source

| Diff

· /sources/canbus/messageinterpreter.cpp

(-0, +113)

| History

| Source

… 4 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Mon, 09 Dec 2019 22:50:18 -0800

Behrouz NematiPour committed 272bb81 on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Changed the datatype interface between Message Interface and from Byte Array to Variant List for different type of parameter support. Also it gives the flexibility to have a Message Interpreter as messaging interface level.

- Added basic classes for Message interface.

- Moved Gui Actions in Gui Globals for simplicity.

· /denali.pro

(-2, +2)

| History

| Source

| Diff

· /main.cpp

(-1, +0)

| History

| Source

| Diff

· /sources/applicationcontroller.cpp

(-6, +6)

| History

| Source

| Diff

· /sources/applicationcontroller.h

(-5, +5)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messageglobals.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/messagehandler.cpp

(-13, +23)

| History

| Source

| Diff

· /sources/canbus/messagehandler.h

(-8, +4)

| History

| Source

| Diff

· /sources/canbus/messageinterface.cpp

(-0, +21)

| History

| Source

· /sources/canbus/messageinterface.h

(-0, +37)

| History

| Source

· /sources/gui/guiactions.cpp

(-24, +0)

| History

| Source

| Diff

· /sources/gui/guiactions.h

(-53, +0)

| History

| Source

| Diff

· /sources/gui/guicontroller.cpp

(-5, +5)

| History

| Source

| Diff

· /sources/gui/guicontroller.h

(-6, +6)

| History

| Source

| Diff

· /sources/gui/guiglobals.cpp

(-2, +14)

| History

| Source

| Diff

… 4 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Mon, 09 Dec 2019 17:30:59 -0800

Behrouz NematiPour committed b2ebb6f on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Added Message Dispatcher class basic empty class

- Renamed Message Partitioner to Message Builder

- Removed Message namespace from MessageGlobals

· /denali.pro	(-2, +4)	\| History	\| Source	\| Diff
· /sources/canbus/messagebuilder.cpp	(-0, +175)	\| History	\| Source
· /sources/canbus/messagebuilder.h	(-0, +79)	\| History	\| Source
· /sources/canbus/messagedispatcher.cpp	(-0, +21)	\| History	\| Source
· /sources/canbus/messagedispatcher.h	(-0, +84)	\| History	\| Source
· /sources/canbus/messageglobals.h	(-5, +37)	\| History	\| Source	\| Diff
· /sources/canbus/messagehandler.cpp	(-12, +12)	\| History	\| Source	\| Diff
· /sources/canbus/messagehandler.h	(-4, +4)	\| History	\| Source	\| Diff
· /sources/canbus/messagepartitioner.cpp	(-175, +0)	\| History	\| Source	\| Diff
· /sources/canbus/messagepartitioner.h	(-92, +0)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Fri, 06 Dec 2019 19:49:34 -0800

Behrouz NematiPour committed 68fe14b on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Add Message Partitioner to handle building and stripping can frames.

· /denali.pro	(-0, +3)	\| History	\| Source	\| Diff
· /sources/canbus/messageglobals.h	(-0, +68)	\| History	\| Source
· /sources/canbus/messagehandler.cpp	(-163, +15)	\| History	\| Source	\| Diff
· /sources/canbus/messagehandler.h	(-99, +4)	\| History	\| Source	\| Diff
· /sources/canbus/messagepartitioner.cpp	(-0, +175)	\| History	\| Source
· /sources/canbus/messagepartitioner.h	(-0, +92)	\| History	\| Source
· /sources/gui/guiactions.h	(-0, +1)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Fri, 06 Dec 2019 13:31:11 -0800

Behrouz NematiPour committed 4c53c7c on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Adding multi message transmit capability.

- [only transmit for now]

· /sources/applicationcontroller.cpp	(-11, +5)	\| History	\| Source	\| Diff
· /sources/applicationcontroller.h	(-4, +4)	\| History	\| Source	\| Diff
· /sources/canbus/caninterface.cpp	(-1, +2)	\| History	\| Source	\| Diff
· /sources/canbus/messagehandler.cpp	(-23, +44)	\| History	\| Source	\| Diff
· /sources/canbus/messagehandler.h	(-10, +10)	\| History	\| Source	\| Diff
· /sources/gui/guicontroller.cpp	(-2, +2)	\| History	\| Source	\| Diff
· /sources/gui/guicontroller.h	(-5, +5)	\| History	\| Source	\| Diff
· /sources/gui/guiview.cpp	(-4, +26)	\| History	\| Source	\| Diff
· /sources/gui/guiview.h	(-4, +7)	\| History	\| Source	\| Diff