mirror of
https://github.com/subsurface/subsurface.git
synced 2024-11-28 05:00:20 +00:00
8733828380
The memory managements for DeviceDetails was very sketchy. First of all, sharing a pointer to a structure between threads seems like a recipe for disaster. Secondly, the structure was a QObject and when first generated included in the (silly) Qt object tree, but when generated in the threads it was not. Clearly, this leaks. Instead, use value semantics and use local copies of the structure. I didn't go full length and use std::move to move the data, because this doesn't work through signals (which are the wrong abstraction here, but OK) and secondly I didn't have time to analyze whether the caller still needs the data after passing it down to the worker thread. To be able to pass an object through signals, the class has to be registered in the Qt MetaType system. Super ugly, but fine for now. Ultimately, this whole thing should probably be replaced by futures, co-routines, or whatever. Moreover, this removes the prefix from number of "m_*" function parameters. By convention, "m_" marks member variables, which function parameters are not. Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at> make DeviceDetails a metatype So that we can pass it as value through the signal/slot system. (squash with original commit) Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
72 lines
1.4 KiB
C++
72 lines
1.4 KiB
C++
// SPDX-License-Identifier: GPL-2.0
|
|
#include "devicedetails.h"
|
|
|
|
gas::gas(unsigned char oxygen, unsigned char helium, unsigned char type, unsigned char depth) :
|
|
oxygen(oxygen), helium(helium), type(type), depth(depth)
|
|
{
|
|
}
|
|
|
|
setpoint::setpoint(unsigned char sp, unsigned char depth) :
|
|
sp(sp), depth(depth)
|
|
{
|
|
}
|
|
|
|
DeviceDetails::DeviceDetails() :
|
|
syncTime(false),
|
|
setPointFallback(0),
|
|
ccrMode(0),
|
|
calibrationGas(0),
|
|
diveMode(0),
|
|
decoType(0),
|
|
ppO2Max(0),
|
|
ppO2Min(0),
|
|
futureTTS(0),
|
|
gfLow(0),
|
|
gfHigh(0),
|
|
aGFLow(0),
|
|
aGFHigh(0),
|
|
aGFSelectable(0),
|
|
vpmConservatism(0),
|
|
saturation(0),
|
|
desaturation(0),
|
|
lastDeco(0),
|
|
brightness(0),
|
|
units(0),
|
|
samplingRate(0),
|
|
salinity(0),
|
|
diveModeColor(0),
|
|
language(0),
|
|
dateFormat(0),
|
|
compassGain(0),
|
|
pressureSensorOffset(0),
|
|
flipScreen(0),
|
|
safetyStop(0),
|
|
maxDepth(0),
|
|
totalTime(0),
|
|
numberOfDives(0),
|
|
altitude(0),
|
|
personalSafety(0),
|
|
timeFormat(0),
|
|
lightEnabled(false),
|
|
light(0),
|
|
alarmTimeEnabled(false),
|
|
alarmTime(0),
|
|
alarmDepthEnabled(false),
|
|
alarmDepth(0),
|
|
leftButtonSensitivity(0),
|
|
rightButtonSensitivity(0),
|
|
buttonSensitivity(0),
|
|
bottomGasConsumption(0),
|
|
decoGasConsumption(0),
|
|
travelGasConsumption(0),
|
|
modWarning(false),
|
|
dynamicAscendRate(false),
|
|
graphicalSpeedIndicator(false),
|
|
alwaysShowppO2(false),
|
|
tempSensorOffset(0),
|
|
safetyStopLength(0),
|
|
safetyStopStartDepth(0),
|
|
safetyStopEndDepth(0),
|
|
safetyStopResetDepth(0)
|
|
{
|
|
}
|