sources

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
added

added

Added

Added

Please put //DEBUG

Please put //DEBUG

Please put //DEBUG

Please put //DEBUG

DEN-15905 - DEN-15548 - UI-BN-S102-Log178 - Application [ Ready ]
DEN-15905 - DEN-15548 - UI-BN-S102-Log178 - Application [ Ready ]
these are the removed/red lines. the added/green lines and not-changed/white lines are alinged.

these are the removed/red lines.
the added/green lines and not-changed/white lines are alinged.

for arterial and venous pressure values we align them to the right. this is correct.

for arterial and venous pressure values we align them to the right.
this is correct.

for arterial and venous pressure values we align them to the right. this is correct.

for arterial and venous pressure values we align them to the right.
this is correct.

UI-DEN-15918 - DEN-15918-UI-VD-S102-Sd-Card-TmpFolder - [Ready]
UI-DEN-15918 - DEN-15918-UI-VD-S102-Sd-Card-TmpFolder - [Ready]
code alignment

code alignment

code alignment

code alignment

code alignment

code alignment

Changed

Changed

Added.

Added.

Added.

Added.

Changed based on how GuiView handles the connection from a created-in-QML object.

Changed based on how GuiView handles the connection from a created-in-QML object.

This I will align similar to how the others look in the AppController. The SLOT/SIGNAL keywords are aligned.

This I will align similar to how the others look in the AppController. The SLOT/SIGNAL keywords are aligned.

RESOLVED

RESOLVED

this is incorrect connection usage. it should be moved to the applicationController class. The rule of API design: *the API is just emitting the signal *and is not responsible for the usage. *an...

this is incorrect connection usage.
it should be moved to the applicationController class.

The rule of API design:

  • the API is just emitting the signal
  • and is not responsible for the usage.
  • any object which needs to use it will connect and use (connects the signal to its own slot and runs the logic in its own thread)


Please move to ApplicationController class.

please check, lists are not empty. can be checked by private member variable: isConfigOk = isConfigOk && ! _<Property Name>.isEmpty();

please check, lists are not empty.
can be checked by private member variable:

isConfigOk = isConfigOk && ! _<Property Name>.isEmpty();
add an empty line before and after

add an empty line before and after

please align SIGNAL(did SLOT( on

please align
SIGNAL(did
SLOT( on

please do not use else. make it like the others on top. Look at lines 79 - 83. And insert it in between lines 89,90 with two empty line on top and bottom of the if block if ( ! codition ) { err ...

please do not use else.
make it like the others on top.
Look at lines 79 - 83.
And insert it in between lines 89,90
with two empty line on top and bottom of the if block

if ( ! codition ) {
  err = ;
  LOG_ ;
  return err;
}


thanks

yes, to easily checkout on all the relevant repositories.

yes, to easily checkout on all the relevant repositories.