theuni/xbmc — comment on commit 2daa4f49bc92bb44f954f9444d7f62d0967b26d0
File:  line 0
By theuni:

yea, i went back and forth on this one.

The other thing i considered was:
for fullscreen trackers (always and on_change), there's no need to ever keep any more than 1 region, since it will always be displayed in its entirety.
so for those trackers, it would work to only add to the regions list if size == 0.

that would remove the need for the param to CleanMarkedRegions, and would ensure that we don't do any extra rendering. And yes, when using VIEWPORT_ON_CHANGE, rendering 1 1080p frame rather than 3+ for single-frame changes (one menu-item to another for ex) is quite significant imo.

thoughts?