Remove the OnTimer events once executor is ready

[janwilmans]

First thing that comes to mind:

void LogSources::Listen()
{
    bool dirty = false;
    const long graceTimeMs = 40;
    for (;;)
    {
        WaitForNextEvent();

        if (!dirty)
        {
            //GuiExecutor::CallAfter(graceTimeMs, [dirty] { 
            //  m_updateTrigger();
            //  GuiExecutor::CallAfter(10, [dirty] { m_updateTrigger(); dirty = false; }
            //});
            dirty = true;
        }
        // todo: protect dirty against races

        if (m_end)
            break;
    }
}

[janwilmans]

In the code above, its crucial to guarantee:

• the code will not block > 1us (the HPC resolution) otherwise the timestamps of the messages will skew
• m_updateTrigger is always called after a message arrives (it is not acceptable that a message is displayed only after the next message arrives)

[janwilmans]

Closed by 206bfa6fe890adb9e8c59dc1043053c4f0ff9a37