Skip to content

Commit

Permalink
Always set mHasPendingClientStates to false after emptying FIFO
Browse files Browse the repository at this point in the history
The condition can be removed because
- only non-empty clientStates are pushed to the FIFO
- calling handleRtClientState(...) with an empty clientState is a no-op
  • Loading branch information
cdi-ableton authored and fgo-ableton committed Feb 20, 2018
1 parent a5069cb commit 85a42b9
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions include/ableton/link/Controller.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -598,13 +598,10 @@ class Controller
void processPendingClientStates()
{
const auto clientState = buildMergedPendingClientState();
if (clientState.timeline || clientState.startStopState)
{
mController.mIo->async([this, clientState]() {
mController.handleRtClientState(clientState);
mHasPendingClientStates = false;
});
}
mController.mIo->async([this, clientState]() {
mController.handleRtClientState(clientState);
mHasPendingClientStates = false;
});
}

Controller& mController;
Expand Down

0 comments on commit 85a42b9

Please sign in to comment.