Skip to content

Commit

Permalink
Remove extra semi colon from cachelib/common/hothash/HotHashDetector.cpp
Browse files Browse the repository at this point in the history
Summary:
`-Wextra-semi` or `-Wextra-semi-stmt`

If the code compiles, this is safe to land.

Reviewed By: jaesoo-fb

Differential Revision: D52965979

fbshipit-source-id: 48ac40eaffc1d34e243c6413bb7f2f3a14f2ae48
  • Loading branch information
r-barnes authored and facebook-github-bot committed Jan 24, 2024
1 parent 9f69381 commit 1c8a21b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cachelib/common/hothash/HotHashDetector.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ namespace cachelib {
uint8_t HotHashDetector::bumpHash(uint64_t hash) {
if (UNLIKELY(++bumpsSinceMaintenance_ >= maintenanceInterval_)) {
doMaintenance();
};
}
auto idx = l1HashFunction(hash);
auto l1count = ++l1Vector_[idx];
// Checking against threshold/2, hot index should pass after one decay
Expand Down Expand Up @@ -114,7 +114,7 @@ void HotHashDetector::doMaintenance() {
for (auto& cell : l2Vector_) {
cell.count = std::min<uint32_t>(hotnessMultiplier_ - 1, cell.count / 2);
cell.hashHits = cell.hashHits / 2;
};
}

// Hashes in L2 may need to move if their preceding cell has decayed. We
// have to run at least size() elements, but also continue running in case
Expand Down

0 comments on commit 1c8a21b

Please sign in to comment.