fix WinEnv assertions

Summary: Closes https://github.com/facebook/rocksdb/pull/2702

Differential Revision: D5585389

Pulled By: ajkr

fbshipit-source-id: cb54041eb481d0d759c440f82a8a2c5b34534173
main
Andrew Kryczka 7 years ago committed by Facebook Github Bot
parent d97a72d63f
commit 47ed3bfc3b
  1. 10
      port/win/env_win.cc

@ -829,7 +829,7 @@ WinEnvThreads::~WinEnvThreads() {
void WinEnvThreads::Schedule(void(*function)(void*), void* arg, Env::Priority pri, void WinEnvThreads::Schedule(void(*function)(void*), void* arg, Env::Priority pri,
void* tag, void(*unschedFunction)(void* arg)) { void* tag, void(*unschedFunction)(void* arg)) {
assert(pri >= Env::Priority::LOW && pri <= Env::Priority::HIGH); assert(pri >= Env::Priority::BOTTOM && pri <= Env::Priority::HIGH);
thread_pools_[pri].Schedule(function, arg, tag, unschedFunction); thread_pools_[pri].Schedule(function, arg, tag, unschedFunction);
} }
@ -878,7 +878,7 @@ void WinEnvThreads::WaitForJoin() {
} }
unsigned int WinEnvThreads::GetThreadPoolQueueLen(Env::Priority pri) const { unsigned int WinEnvThreads::GetThreadPoolQueueLen(Env::Priority pri) const {
assert(pri >= Env::Priority::LOW && pri <= Env::Priority::HIGH); assert(pri >= Env::Priority::BOTTOM && pri <= Env::Priority::HIGH);
return thread_pools_[pri].GetQueueLen(); return thread_pools_[pri].GetQueueLen();
} }
@ -894,17 +894,17 @@ void WinEnvThreads::SleepForMicroseconds(int micros) {
} }
void WinEnvThreads::SetBackgroundThreads(int num, Env::Priority pri) { void WinEnvThreads::SetBackgroundThreads(int num, Env::Priority pri) {
assert(pri >= Env::Priority::LOW && pri <= Env::Priority::HIGH); assert(pri >= Env::Priority::BOTTOM && pri <= Env::Priority::HIGH);
thread_pools_[pri].SetBackgroundThreads(num); thread_pools_[pri].SetBackgroundThreads(num);
} }
int WinEnvThreads::GetBackgroundThreads(Env::Priority pri) { int WinEnvThreads::GetBackgroundThreads(Env::Priority pri) {
assert(pri >= Env::Priority::LOW && pri <= Env::Priority::HIGH); assert(pri >= Env::Priority::BOTTOM && pri <= Env::Priority::HIGH);
return thread_pools_[pri].GetBackgroundThreads(); return thread_pools_[pri].GetBackgroundThreads();
} }
void WinEnvThreads::IncBackgroundThreadsIfNeeded(int num, Env::Priority pri) { void WinEnvThreads::IncBackgroundThreadsIfNeeded(int num, Env::Priority pri) {
assert(pri >= Env::Priority::LOW && pri <= Env::Priority::HIGH); assert(pri >= Env::Priority::BOTTOM && pri <= Env::Priority::HIGH);
thread_pools_[pri].IncBackgroundThreadsIfNeeded(num); thread_pools_[pri].IncBackgroundThreadsIfNeeded(num);
} }

Loading…
Cancel
Save