Remove extra semi colon from internal_repo_rocksdb/repo/db/table_cach…

…e_sync_and_async.h Summary: `-Wextra-semi` or `-Wextra-semi-stmt` If the code compiles, this is safe to land. Reviewed By: palmje Differential Revision: D54362208 fbshipit-source-id: a47acd4c794c899fccb65285b116b50d9566ea12
rockset · Mar 4, 2024 · d7b8756 · d7b8756
1 parent ced333e
commit d7b8756
Show file tree

Hide file tree

Showing 5 changed files with 3 additions and 5 deletions.
diff --git a/db/table_cache_sync_and_async.h b/db/table_cache_sync_and_async.h
@@ -103,7 +103,6 @@ DEFINE_SYNC_AND_ASYNC(Status, TableCache::MultiGet)
          ++miter) {
       std::string& row_cache_entry = row_cache_entries[row_idx++];
       const Slice& user_key = miter->ukey_with_ts;
-      ;
       GetContext* get_context = miter->get_context;
 
       get_context->SetReplayLog(nullptr);

diff --git a/table/block_based/block_based_table_builder.cc b/table/block_based/block_based_table_builder.cc
@@ -707,7 +707,7 @@ struct BlockBasedTableBuilder::ParallelCompressionRep {
     template <typename T>
     void Fill(T&& rep) {
       slot_.push(std::forward<T>(rep));
-    };
+    }
     void Take(BlockRep*& rep) { slot_.pop(rep); }
 
    private:

diff --git a/table/block_based/block_based_table_iterator.cc b/table/block_based/block_based_table_iterator.cc
@@ -846,7 +846,7 @@ void BlockBasedTableIterator::BlockCacheLookupForReadAheadSize(
     }
     index_iter_->Next();
     is_index_at_curr_block_ = false;
-  };
+  }
 
   if (found_first_miss_block) {
     // Iterate cache hit block handles from the end till a Miss is there, to

diff --git a/table/persistent_cache_helper.cc b/table/persistent_cache_helper.cc
@@ -40,7 +40,6 @@ void PersistentCacheHelper::InsertUncompressed(
   cache_options.persistent_cache
       ->Insert(key.AsSlice(), contents.data.data(), contents.data.size())
       .PermitUncheckedError();
-  ;
 }
 
 Status PersistentCacheHelper::LookupSerialized(

diff --git a/table/table_builder.h b/table/table_builder.h
@@ -232,7 +232,7 @@ class TableBuilder {
   // enforced state (ready to encode to string).
   virtual void SetSeqnoTimeTableProperties(
       const SeqnoToTimeMapping& /*relevant_mapping*/,
-      uint64_t /*oldest_ancestor_time*/){};
+      uint64_t /*oldest_ancestor_time*/){}
 };
 
 }  // namespace ROCKSDB_NAMESPACE