Age | Commit message (Collapse) | Author |
|
The registration for several Pref* point at wrong C++ object,
correct the paste error.
Signed-off-by: jan Iversen <jan@casacondor.com>
|
|
Since writing the file with QtCreator in the previous commit
pointed all these out...
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
It seemed to make sense to combine all three types in one commit.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Again, several different ways to achieve the same thing.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
There were three different ways (two of which generated warnings) in a single
source file.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This deals with the issue of initializing structs in C++.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Frankly, I find the old API easier...
Also, replace toList() with values()
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
We still support Qt5.9 for now, so we need the version check as birthTime() wasn't
available in 5.9.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
dark() and darker() appear to have the same semantics.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
It's not even clear if we need the setCaseSensitivity() call as it appears
that a case insensitive completer is the default.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
It was initialized, but never referenced.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Replacement was only introduced in Qt 5.14
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
In a total abundance of caution, make sure we don't exit the constructor
leaving fields uninitialized.
Fixes CID 351437
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
While technically the initial value of this variable makes no difference as
it is set when the first dive is displayed, technically Coverity is correct.
Fixes CID 353273
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Fixes CID #353274
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Here the replacement has been around for a very long time.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This seems silly. I don't like that warning and would rather disable the warning.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Newer versions of Qt deprecate using QTime as a timer.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Qt5.11 introduced the suggested replacement QFontMetrics::horizontalAdvance().
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
The last SIGNAL/SLOT cannot be converted due to a limitation in the
new connect() syntax, it does not "understand" default parameters.
Signed-off-by: jan Iversen <jan@casacondor.com>
|
|
Replace SIGNAL/SLOT in favor of new connect()
Signed-off-by: jan Iversen <jan@casacondor.com>
|
|
Replace SIGNAL() with QOverload<double>::of() to allow
new connect syntax.
Signed-off-by: jan Iversen <jan@casacondor.com>
|
|
Convert the last convert() statements, that can be
converted just by changing the syntax.
Signed-off-by: jan Iversen <jan@casacondor.com>
|
|
Replace syntax for connect of QShortcut.
Signed-off-by: jan Iversen <jan@casacondor.com>
|
|
Change connect() to new syntax for QDataTime.
Signed-off-by: jan Iversen <jan@casacondor.com>
|
|
Replace SIGNAL/SLOT with QOverload.
Signed-off-by: jan Iversen <jan@casacondor.com>
|
|
Change connect for QAction::triggered(bool) to new syntax.
Signed-off-by: Jan Iversen <jan@casacondor.com>
|
|
Change connect for QSpinBox::valueChange(int) to new syntax.
Signed-off-by: Jan Iversen <jan@casacondor.com>
|
|
Change connect for QAbstractButton::clickedto new syntax.
Change connect for QDialogButtonBox::accepted/rejected to new syntax.
Signed-off-by: Jan Iversen <jan@casacondor.com>
|
|
No need to connect ui.display_variations to the
same signal/slot twice.
Signed-off-by: Jan Iversen <jan@casacondor.com>
|
|
Change connect for QAbstractButton::toogled to new syntax.
Signed-off-by: Jan Iversen <jan@casacondor.com>
|
|
correct SLOT was sec_bottomsac(double), which
is not reported as an error.
correct to set_bottomsac(double)
Signed-off-by: Jan Iversen <jan@casacondor.com>
|
|
correct SLOT was set_min_swich_duration()(int), which
is not reported as an error.
Signed-off-by: Jan Iversen <jan@casacondor.com>
|
|
correct name mistake in connect()
Signed-off-by: Jan Iversen <jan@casacondor.com>
|
|
Adjust connect() to use plannerShared, avoiding parameter convert
problem.
Signed-off-by: Jan Iversen <jan@casacondor.com>
|
|
QSignalMapper gives a warning that it is depreciated, and the
doc. states that using a lambda function is more efficient.
Replace use of QSignalMapper.
Signed-off-by: Jan Iversen <jan@casacondor.com>
|
|
QSignalMapper had a parameter convert problem, when mapping to
set_deco_mode in plannerShared.
Use lambda function in connect to avoid parameter convert problem.
Signed-off-by: Jan Iversen <jan@casacondor.com>
|
|
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
There were two cases that were handled incorrectly:
- if the user hasn't entered a salinity, obviously there shouldn't be a warning
- if this is a manually entered dive, there is no salinity downloaded from a
dive computer, so equally, no warning
Suggested-by: willemferguson <willemferguson@zoology.up.ac.za>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
We need to show this whenever the value in the dive (which could have been
entered by the user some other time) doesn't correspond to the value in the DC.
This, btw, will point out to the user if different DCs have different values.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
We were royally confused when we didn't know the salinity value (e.g., if the
dive computer didn't provide that information). We somehow treated this as the
same as wanting to use the salinity information in the dive computer. Which
makes no sense.
While cleaning this up, this also adds the textual representations of the water
types as a string list that corresponds to the enum values that we use - this
way it's easier to stay consistent.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
I'm rather upset that I never noticed this in a review. That's just not how you
print numbers.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Added code for string translation.
Added code to improve UI on Windows.
Added some comments to make the code more understandable.
Enable salinity combobox for manually entered dives
Signed-off-by: willemferguson <willemferguson@zoology.up.ac.za>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Both XML and git storage are added.
Signed-off-by: willemferguson <willemferguson@zoology.up.ac.za>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|