fix(battery): Remove pointless lock

This commit is contained in:
Michael Carlberg 2016-05-31 15:51:48 +02:00
parent 260519ec55
commit 688ca94658

View File

@ -77,8 +77,6 @@ void BatteryModule::subthread_runner()
const int poll_seconds = config::get<float>(name(), "poll_interval", 3.0f) / dur.count(); const int poll_seconds = config::get<float>(name(), "poll_interval", 3.0f) / dur.count();
while (this->enabled()) { while (this->enabled()) {
std::unique_lock<concurrency::SpinLock> lck(this->broadcast_lock);
// TODO(jaagr): Keep track of when the values were last read to determine // TODO(jaagr): Keep track of when the values were last read to determine
// if we need to trigger the event manually or not. // if we need to trigger the event manually or not.
if (poll_seconds > 0 && (++i % poll_seconds) == 0) { if (poll_seconds > 0 && (++i % poll_seconds) == 0) {
@ -88,10 +86,8 @@ void BatteryModule::subthread_runner()
i = 0; i = 0;
} }
if (this->state == STATE_CHARGING) { if (this->state == STATE_CHARGING)
lck.unlock();
this->broadcast(); this->broadcast();
}
std::this_thread::sleep_for(dur); std::this_thread::sleep_for(dur);
} }