diff --git a/include/rocksdb/env.h b/include/rocksdb/env.h index bcfd24c1fc35445153b7bfe4394aa2a82842902a..faa1b0d6e317cdd998708077415fdf652501e252 100644 --- a/include/rocksdb/env.h +++ b/include/rocksdb/env.h @@ -900,7 +900,7 @@ class EnvWrapper : public Env { public: // Initialize an EnvWrapper that delegates all calls to *t explicit EnvWrapper(Env* t) : target_(t) { } - virtual ~EnvWrapper(); + ~EnvWrapper() override; // Return the target to which this Env forwards all calls Env* target() const { return target_; } @@ -935,8 +935,8 @@ class EnvWrapper : public Env { const EnvOptions& options) override { return target_->NewRandomRWFile(fname, result, options); } - virtual Status NewDirectory(const std::string& name, - unique_ptr* result) override { + Status NewDirectory(const std::string& name, + unique_ptr* result) override { return target_->NewDirectory(name, result); } Status FileExists(const std::string& f) override { @@ -998,15 +998,14 @@ class EnvWrapper : public Env { return target_->StartThread(f, a); } void WaitForJoin() override { return target_->WaitForJoin(); } - virtual unsigned int GetThreadPoolQueueLen( - Priority pri = LOW) const override { + unsigned int GetThreadPoolQueueLen(Priority pri = LOW) const override { return target_->GetThreadPoolQueueLen(pri); } - virtual Status GetTestDirectory(std::string* path) override { + Status GetTestDirectory(std::string* path) override { return target_->GetTestDirectory(path); } - virtual Status NewLogger(const std::string& fname, - shared_ptr* result) override { + Status NewLogger(const std::string& fname, + shared_ptr* result) override { return target_->NewLogger(fname, result); } uint64_t NowMicros() override { return target_->NowMicros(); } @@ -1098,10 +1097,10 @@ class WritableFileWrapper : public WritableFile { return target_->InvalidateCache(offset, length); } - virtual void SetPreallocationBlockSize(size_t size) override { + void SetPreallocationBlockSize(size_t size) override { target_->SetPreallocationBlockSize(size); } - virtual void PrepareWrite(size_t offset, size_t len) override { + void PrepareWrite(size_t offset, size_t len) override { target_->PrepareWrite(offset, len); } diff --git a/include/rocksdb/flush_block_policy.h b/include/rocksdb/flush_block_policy.h index 7655901716e90933656f4ab319384434e80f3672..19a58dc01934d88f5e04c717a514fdee79f1c84f 100644 --- a/include/rocksdb/flush_block_policy.h +++ b/include/rocksdb/flush_block_policy.h @@ -48,11 +48,9 @@ class FlushBlockBySizePolicyFactory : public FlushBlockPolicyFactory { public: FlushBlockBySizePolicyFactory() {} - virtual const char* Name() const override { - return "FlushBlockBySizePolicyFactory"; - } + const char* Name() const override { return "FlushBlockBySizePolicyFactory"; } - virtual FlushBlockPolicy* NewFlushBlockPolicy( + FlushBlockPolicy* NewFlushBlockPolicy( const BlockBasedTableOptions& table_options, const BlockBuilder& data_block_builder) const override; diff --git a/include/rocksdb/merge_operator.h b/include/rocksdb/merge_operator.h index 6ec34fe3e45354944657933e104f2dbc23ad520b..0ec737ab88929f58b20a12a767cd5c4675ad5fe0 100644 --- a/include/rocksdb/merge_operator.h +++ b/include/rocksdb/merge_operator.h @@ -188,7 +188,7 @@ class MergeOperator { // The simpler, associative merge operator. class AssociativeMergeOperator : public MergeOperator { public: - virtual ~AssociativeMergeOperator() {} + ~AssociativeMergeOperator() override {} // Gives the client a way to express the read -> modify -> write semantics // key: (IN) The key that's associated with this merge operation. @@ -212,14 +212,12 @@ class AssociativeMergeOperator : public MergeOperator { private: // Default implementations of the MergeOperator functions - virtual bool FullMergeV2(const MergeOperationInput& merge_in, - MergeOperationOutput* merge_out) const override; + bool FullMergeV2(const MergeOperationInput& merge_in, + MergeOperationOutput* merge_out) const override; - virtual bool PartialMerge(const Slice& key, - const Slice& left_operand, - const Slice& right_operand, - std::string* new_value, - Logger* logger) const override; + bool PartialMerge(const Slice& key, const Slice& left_operand, + const Slice& right_operand, std::string* new_value, + Logger* logger) const override; }; } // namespace rocksdb diff --git a/include/rocksdb/utilities/sim_cache.h b/include/rocksdb/utilities/sim_cache.h index 64cb643ce40fc89a2554974420fc6e00a34dc59f..a8581bf2b10271345899c1e0777411bd5cdc8e36 100644 --- a/include/rocksdb/utilities/sim_cache.h +++ b/include/rocksdb/utilities/sim_cache.h @@ -39,9 +39,9 @@ class SimCache : public Cache { public: SimCache() {} - virtual ~SimCache() {} + ~SimCache() override {} - virtual const char* Name() const override { return "SimCache"; } + const char* Name() const override { return "SimCache"; } // returns the maximum configured capacity of the simcache for simulation virtual size_t GetSimCapacity() const = 0; diff --git a/include/rocksdb/utilities/transaction_db.h b/include/rocksdb/utilities/transaction_db.h index ec97438928fe8fe5c207c0034c3c0b454f5c9996..e14f9e06db10c2255cadad417371309d6bc3a53f 100644 --- a/include/rocksdb/utilities/transaction_db.h +++ b/include/rocksdb/utilities/transaction_db.h @@ -148,7 +148,7 @@ class TransactionDB : public StackableDB { StackableDB* db, const TransactionDBOptions& txn_db_options, const std::vector& compaction_enabled_cf_indices, const std::vector& handles, TransactionDB** dbptr); - virtual ~TransactionDB() {} + ~TransactionDB() override {} // Starts a new Transaction. // diff --git a/include/rocksdb/utilities/write_batch_with_index.h b/include/rocksdb/utilities/write_batch_with_index.h index ba6d670575e860d5fb54e8ab85c7ae77a710743a..f429610c4144172f044663ad2b41e8b3fc5a9b9d 100644 --- a/include/rocksdb/utilities/write_batch_with_index.h +++ b/include/rocksdb/utilities/write_batch_with_index.h @@ -97,7 +97,7 @@ class WriteBatchWithIndex : public WriteBatchBase { size_t reserved_bytes = 0, bool overwrite_key = false, size_t max_bytes = 0); - virtual ~WriteBatchWithIndex(); + ~WriteBatchWithIndex() override; using WriteBatchBase::Put; Status Put(ColumnFamilyHandle* column_family, const Slice& key, diff --git a/include/rocksdb/write_batch.h b/include/rocksdb/write_batch.h index d98c56261c7a8f17d5f0ba2098b7d688451eb872..432a7c19bbb143b928f978cd5ad04233f63a4f4a 100644 --- a/include/rocksdb/write_batch.h +++ b/include/rocksdb/write_batch.h @@ -61,7 +61,7 @@ struct SavePoint { class WriteBatch : public WriteBatchBase { public: explicit WriteBatch(size_t reserved_bytes = 0, size_t max_bytes = 0); - ~WriteBatch(); + ~WriteBatch() override; using WriteBatchBase::Put; // Store the mapping "key->value" in the database. diff --git a/java/src/test/java/org/rocksdb/WriteBatchThreadedTest.java b/java/src/test/java/org/rocksdb/WriteBatchThreadedTest.java index e31d83b2843869e57ccab90042e297c9897076a1..7dbe99f4fc1137b30800cd2745688852badfbc35 100644 --- a/java/src/test/java/org/rocksdb/WriteBatchThreadedTest.java +++ b/java/src/test/java/org/rocksdb/WriteBatchThreadedTest.java @@ -62,8 +62,7 @@ public class WriteBatchThreadedTest { try (final WriteBatch wb = new WriteBatch(); final WriteOptions w_opt = new WriteOptions()) { for (int i = offset; i < offset + 100; i++) { - wb.put(ByteBuffer.allocate(4).putInt(i).array(), - "parallel rocks test".getBytes()); + wb.put(ByteBuffer.allocate(4).putInt(i).array(), "parallel rocks test".getBytes()); } db.write(w_opt, wb); }