diff options
| author | Daniel Kraft <[email protected]> | 2018-05-20 09:15:39 +0200 |
|---|---|---|
| committer | Daniel Kraft <[email protected]> | 2018-05-20 09:15:39 +0200 |
| commit | 60ebc7da4c2f918c821008088e88aba8ab40d867 (patch) | |
| tree | 64375d39d60874eb2a16b6c5760f6659eaac2b68 /src | |
| parent | Merge #13228: Add script to detect circular dependencies between source modules (diff) | |
| download | discoin-60ebc7da4c2f918c821008088e88aba8ab40d867.tar.xz discoin-60ebc7da4c2f918c821008088e88aba8ab40d867.zip | |
trivial: Mark overrides as such.
This trivial change adds the "override" keyword to some methods of
subclasses meant to override interface methods. This ensures that any
future change to the interface' method signatures which are not correctly
mirrored in the subclass will break at compile time with a clear error message,
rather than fail at runtime (which is harder to debug).
Diffstat (limited to 'src')
| -rw-r--r-- | src/bench/bench.h | 12 | ||||
| -rw-r--r-- | src/test/validation_block_tests.cpp | 6 |
2 files changed, 9 insertions, 9 deletions
diff --git a/src/bench/bench.h b/src/bench/bench.h index e15cb8186..5d131c2aa 100644 --- a/src/bench/bench.h +++ b/src/bench/bench.h @@ -111,9 +111,9 @@ public: class ConsolePrinter : public Printer { public: - void header(); - void result(const State& state); - void footer(); + void header() override; + void result(const State& state) override; + void footer() override; }; // creates box plot with plotly.js @@ -121,9 +121,9 @@ class PlotlyPrinter : public Printer { public: PlotlyPrinter(std::string plotly_url, int64_t width, int64_t height); - void header(); - void result(const State& state); - void footer(); + void header() override; + void result(const State& state) override; + void footer() override; private: std::string m_plotly_url; diff --git a/src/test/validation_block_tests.cpp b/src/test/validation_block_tests.cpp index f83557016..37c4f7913 100644 --- a/src/test/validation_block_tests.cpp +++ b/src/test/validation_block_tests.cpp @@ -25,12 +25,12 @@ struct TestSubscriber : public CValidationInterface { TestSubscriber(uint256 tip) : m_expected_tip(tip) {} - void UpdatedBlockTip(const CBlockIndex* pindexNew, const CBlockIndex* pindexFork, bool fInitialDownload) + void UpdatedBlockTip(const CBlockIndex* pindexNew, const CBlockIndex* pindexFork, bool fInitialDownload) override { BOOST_CHECK_EQUAL(m_expected_tip, pindexNew->GetBlockHash()); } - void BlockConnected(const std::shared_ptr<const CBlock>& block, const CBlockIndex* pindex, const std::vector<CTransactionRef>& txnConflicted) + void BlockConnected(const std::shared_ptr<const CBlock>& block, const CBlockIndex* pindex, const std::vector<CTransactionRef>& txnConflicted) override { BOOST_CHECK_EQUAL(m_expected_tip, block->hashPrevBlock); BOOST_CHECK_EQUAL(m_expected_tip, pindex->pprev->GetBlockHash()); @@ -38,7 +38,7 @@ struct TestSubscriber : public CValidationInterface { m_expected_tip = block->GetHash(); } - void BlockDisconnected(const std::shared_ptr<const CBlock>& block) + void BlockDisconnected(const std::shared_ptr<const CBlock>& block) override { BOOST_CHECK_EQUAL(m_expected_tip, block->GetHash()); |