Age | Commit message (Collapse) | Author |
|
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
See also commit c032006d91ee3c. Compare functions passed
to sort functions need to compare for less-than and not
less-or-equal.
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
In MainWindow::on_actionCloudstorageopen_triggered(),
getNotificationWidget()->hideNotification() was called, thus hiding
any error message produced by the cloud code. Notably, the information
"Cannot sync with cloud server, working with offline copy" was
not shown.
Therefore, remove this call. Note that on cloud save messages were
not hidden.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
|
|
CID 208303
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 60227
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 208312
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 208311
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 208309
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 208307
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 208292
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 45171
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 208320
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 45112
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 208332
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 45172
And one non initalized member hAxis could be removed as it is
not used. Surprisingly, Coverity did not see this.
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This fixes a bug where if the user entered very high ascent (or less commonly
descent) rates such that the time to ascend (or descend) from one level to the
next was less than 10s, that leg would be skipped in the dive plan notes.
Reported-by: Alexander Maier <maieralex@me.com>
Signed-off-by: Rick Walsh <rickmwalsh@gmail.com>
|
|
CID 45184
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 208317
CID 208325
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 208318
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 208296. IndexOf can return -1 when not found, which will
not happen in this context, so just to silence Coverity.
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID: 208335
Mainly to silence Coverity, as I do not believe this caused
any problem.
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
clear_vpmb_state() was declared with incorrect signature, and all
functios in this change are extern, so declare them as such.
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
CID 208315
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
With commit 5962f00679fae555, a well known problem was introduced.
Incorrect width setting for the spanning trip lines. And as there
is even a specific functon for that, just call this.
The reason the mentioned commit introduces this, is that
invalidate() causes layoutChanged signals, and invalidateFilter()
does not.
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
This is mainly code maintenance. Instead of emitting explicit
dataChanged signals, we can make sure that setStringList()
is called after all model data manipulation is ready. Accoording
to the Qt docs: "The model will notify any attached views
that its underlying data has changed".
In itself, this does not solve the tripped assert mentioned in
commit 5962f00679fae5, but this calling at the end just feels
better.
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
This "bug" is found using Qt 5.10 compiled in developer mode. Access
the filers, click a little around here, close the filters. Almost every
time the following assert is triggered:
ASSERT failure in QPersistentModelIndex::~QPersistentModelIndex:
"persistent model indexes corrupted", file itemmodels/qabstractitemmodel.cpp, line 643
This is relatively deep down in Qt, and it is triggered by clearing the
filters. Trying to force a crash when using the same scenario in Qt 5.10
compiled for production (so no active asserts) did not result in a crash.
So, upto this time, it is unclear if the Qt assert points out a real problem,
or it is some false alarm (for whatever reason).
Further investigation shows that the assert can be solved by changing the
invalidate() to an invalidateFilter(). Indeed, the last variant is a little
more lightweigt, and does seem to do the same job from a functional point
of view (in this case).
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
DiveLocationFilterProxyModel::lessThan() used less-or-equal instead
of less-than comparison. This can lead to subtle bugs if two elements
compare as equal.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
|
|
FilterModelBase is a direct subclass of QAbstractItemModel. Therefore,
dynamic_cast<>ing the former to the latter is unnecessary. Probably
an artifact of previous code.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
|
|
Coverity CID 208330
Coverity CID 208301
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This is basically to make Coverity happy.
Coverity CID 208300
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Realistically this is a false positive as we should never use a second
BTDiscovery instance - but there's nothing wrong with being extra certain.
Coverity CID 208319
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Coverity CID 45078
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Coverity CID 45121
Coverity CID 45163
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
The previous attempt to fix this in commit 652e382e68 ("Cleanup: avoid a
few memory leaks") was clearly bogus. Oops.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Signed-off-by: Miika Turkia <miika.turkia@gmail.com>
|
|
Coverity CID 208323
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Coverity CID 208289
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
sizeof() is clearly the wrong way to get to the size of that array...
Coverity CID 208294
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
sizeof() is clearly the wrong way to get to the size of that array...
Coverity CID 208290
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
sizeof() is clearly the wrong way to get to the size of that array...
Coverity CID 208297
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Coverity CID 208298
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Coverity CID 208308
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Coverity CID 208314
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
No point in doing the strdup of the password if we then bail.
Coverity CID 208316
Coverity CID 209293
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Don't access cylinder[MAX_CYLINDERS]
Coverity CID 208324
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Coverity CID 208327
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Coverity CID 208333
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|