aboutsummaryrefslogtreecommitdiffstats
path: root/core/divesitehelpers.h
diff options
context:
space:
mode:
authorGravatar Berthold Stoeger <bstoeger@mail.tuwien.ac.at>2018-07-31 07:41:19 +0200
committerGravatar Tomaz Canabrava <tcanabrava@kde.org>2018-07-31 11:02:34 +0200
commitdf156a56c08a56eb380711a507ef739d8150a71f (patch)
tree0d0da57857fff1ba844c84fbc5de2e6cd872cb1e /core/divesitehelpers.h
parent7fe76a5dbdef4a833122e0311191dd51896575f4 (diff)
downloadsubsurface-df156a56c08a56eb380711a507ef739d8150a71f.tar.gz
Cleanup: replace virtual by override where appropriate
The keyword "virtual" signalizes that the function is virtual, i.e. the function of the derived class is called, even if the call is on the parent class. It is not necessary to repeat the "virtual" keyword in derived classes. To highlight derived virtual functions, the keyword "override" should be used instead. It results in a hard compile- error, if no function is overridden, thus avoiding subtle bugs. Replace "virtual" by "override" where appropriate. Moreover, replace Q_DECL_OVERRIDE by override, since we require reasonably recent compilers anyway. Likewise, replace /* reimp */ by "override" for consistency and compiler support. Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'core/divesitehelpers.h')
-rw-r--r--core/divesitehelpers.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/core/divesitehelpers.h b/core/divesitehelpers.h
index a6a347b69..9227aa5c2 100644
--- a/core/divesitehelpers.h
+++ b/core/divesitehelpers.h
@@ -10,7 +10,7 @@ Q_OBJECT
public:
static ReverseGeoLookupThread *instance();
void lookup(struct dive_site *ds);
- void run() Q_DECL_OVERRIDE;
+ void run() override;
private:
ReverseGeoLookupThread(QObject *parent = 0);