Code review comment for lp:~chasedouglas/grail/touch-states

Revision history for this message
Daniel d'Andrada (dandrader) wrote :

AtomicRecognizer::CollectNewTouches() was meant to fill in only the new_touches_ set, hence its name. Now it's also changing all_touches_ and free_touches_ sets. That leaves its name and documentation inconsistent with what it actually does. For clarity's sake I think that method should either be renamed or the other sets updated elsewhere.

« Back to merge proposal