From e517bfa2c2159e509cbcf8196435b5b86621b1d1 Mon Sep 17 00:00:00 2001 From: Siying Dong Date: Tue, 27 Jun 2017 10:49:34 -0700 Subject: [PATCH] CLANG Tidy Summary: Closes https://github.com/facebook/rocksdb/pull/2502 Differential Revision: D5326498 Pulled By: siying fbshipit-source-id: 2f0ac6dc6ca5ddb23cecf67a278c086e52646714 --- include/rocksdb/env.h | 19 +++++++++---------- include/rocksdb/flush_block_policy.h | 6 ++---- include/rocksdb/merge_operator.h | 14 ++++++-------- include/rocksdb/utilities/sim_cache.h | 4 ++-- include/rocksdb/utilities/transaction_db.h | 2 +- .../utilities/write_batch_with_index.h | 2 +- include/rocksdb/write_batch.h | 2 +- .../org/rocksdb/WriteBatchThreadedTest.java | 3 +-- 8 files changed, 23 insertions(+), 29 deletions(-) diff --git a/include/rocksdb/env.h b/include/rocksdb/env.h index bcfd24c1fc..faa1b0d6e3 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 7655901716..19a58dc019 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 6ec34fe3e4..0ec737ab88 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 64cb643ce4..a8581bf2b1 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 ec97438928..e14f9e06db 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 ba6d670575..f429610c41 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 d98c56261c..432a7c19bb 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 e31d83b284..7dbe99f4fc 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); }