Mir

Code review comment for lp:~mir-team/mir/touchspot-renderable

Revision history for this message
Andreas Pokorny (andreas-pokorny) wrote :

sryy,but i meant something trivial like:

1441 + {
1442 + std::lock_guard<std::mutex> lg(guard);
1443 +
1444 + if (!enabled)
1445 + {
1446 + remove_touchspot_renderables();
1454 + return;
1455 + }
1456 +
1457 + grow_touchspot_renderables(touches.size());
1461 +
1462 + update_touchspot_renderables(touches);
1487 + }
1488 +
1489 + // TODO (hackish): We may have just moved renderables which with the current
1490 + // architecture of surface observers will not trigger a propagation to the
1491 + // compositor damage callback we need this "emit_scene_changed".
1492 + scene->emit_scene_changed();

« Back to merge proposal