application

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-6359: Addressed code review comments

Done

Done

RESOLVED

RESOLVED

So we may need to be careful when we are adding the WiFi driver in here later. RESOLVED.

So we may need to be careful when we are adding the WiFi driver in here later.
RESOLVED.

DEN-4867: Preventing create treatment class from thinking a new treatment profile was saved when the bledevices.conf file is updated.

Perfect, RESOLVED.

Perfect,
RESOLVED.

We need to have a folder next to the application for all the settings since it is not related to the treatment it is only settings configuration files. Since the SD card might be damaged and we sti...

We need to have a folder next to the application for all the settings since it is not related to the treatment it is only settings configuration files.
Since the SD card might be damaged and we still need to store settings.
Please change it to refer to the application location and name the folder as configuration or settings.

DEN-4867: Addressed code review comments

    • -20
    • +60
    /sources/bluetooth/BLEScanner.cpp
Oh I see. RESOLVED

Oh I see.
RESOLVED

deleted.

deleted.

Good catch, Dleted.

Good catch,
Dleted.

You actually had it with the value of 0.1 ms and I increased it to 0.5 because after running the test multiple times I figured it is not enough time. I will review the waits in the pTx-UF COCO.

You actually had it with the value of 0.1 ms and I increased it to 0.5 because after running the test multiple times I figured it is not enough time.
I will review the waits in the pTx-UF COCO.

I kept it for debugging. The usual behavior of the SquishQt is to continue even if an error happens to stop it if an error happens this value needs to be set. I'm still trying to figure out how exa...

I kept it for debugging.
The usual behavior of the SquishQt is to continue even if an error happens to stop it if an error happens this value needs to be set.
I'm still trying to figure out how exactly this works and later may move it in a better file.

I checked out into the code and seems like the VBluetoothDeviceInfo class is not a view class although it has properties. So, why it has V in the naming, and why it has property? Views are only int...

I checked out into the code and seems like the VBluetoothDeviceInfo class is not a view class although it has properties.
So, why it has V in the naming, and why it has property?
Views are only intended to be used in QML to interact with the C++ backend.

the 'on' prefix shall be used for the private slots only. seems like this is not the case here. A better naming would be notifyStatusUpdate().

the 'on' prefix shall be used for the private slots only.
seems like this is not the case here.
A better naming would be notifyStatusUpdate().

Please use macros for the connections, whenever possible.

Please use macros for the connections, whenever possible.

Please use macros for the connections, whenever possible.

Please use macros for the connections, whenever possible.

refer to the above comment about the MainTimer.

refer to the above comment about the MainTimer.

I remember I mentioned somewhere in the code to use the QObject internal timer and also mentioned MainTimer as an example. But I didn't mean to use the MainTimer. Please look at the MainTimer as an...

I remember I mentioned somewhere in the code to use the QObject internal timer and also mentioned MainTimer as an example.
But I didn't mean to use the MainTimer.
Please look at the MainTimer as an example of how to use the QObject internal timer and use it instead of the MainTimer itself.

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

How come a function has been added in the cpp but the header is not automatically added to the code review?

How come a function has been added in the cpp but the header is not automatically added to the code review?

I've moved the comment back so it's inside the if statement

I've moved the comment back so it's inside the if statement

DEN-6359: Addressed code review comments

This is my comment and it is specific to the exact line below the comment. Please revert it back to be able to identify which line in the code it refers to. "This code review includes so many unre...

This is my comment and it is specific to the exact line below the comment.
Please revert it back to be able to identify which line in the code it refers to.

"This code review includes so many unrelated branched to the code review subject"
look at the list of the commits and branches included in this page
http://dvm-linux02:8060/project/UI-DEN-6349?max=100&projectKey=UI-DEN-6349&view=fe