andreacavalli/rocksdb
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix UNLIKELY parenthesis

Browse Source 
Summary: Ooops :) status.ok() is acutally highly likely :)

Test Plan: none

Reviewers: rven, yhchiang, anthony

Reviewed By: anthony

Subscribers: dhruba, leveldb

Differential Revision: https://reviews.facebook.net/D38043
This commit is contained in:
Igor Canadi 2015-05-05 08:57:34 -07:00
parent 9bdbaab94f
commit 36a7408896
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
db/db_impl.cc
View File

@ -3200,8 +3200,8 @@ Status DBImpl::Write(const WriteOptions& write_options, WriteBatch* my_batch) {
status = ScheduleFlushes(&context); status = ScheduleFlushes(&context);
} }
if (UNLIKELY(status.ok()) && if (UNLIKELY(status.ok() && (write_controller_.IsStopped() ||
(write_controller_.IsStopped() || write_controller_.GetDelay() > 0)) { write_controller_.GetDelay() > 0))) {
// If writer is stopped, we need to get it going, // If writer is stopped, we need to get it going,
// so schedule flushes/compactions // so schedule flushes/compactions
if (context.schedule_bg_work_) { if (context.schedule_bg_work_) {