let's turn this into a proper request ;-)

counter suggest was

void MouseClickEffect::slotMouseChanged(...)
{
if (!m_enabled) return;
...

objection against this was:
mouse polling is fired pretty often, so why bind the slot just to exit it on invocation (most of the time)?