silence cppcheck by not updating state after the last pixel
initialize all members in the constructor

Though I wonder if setting m_run(true) isn't a bit early, when we
could set it in Run().