Hi, On Wed, Dec 29, 2010 at 03:04:29PM +0200, [email protected] wrote: > static void > -compScreenUpdate (ScreenPtr pScreen) > +compChildrenUpdate (WindowPtr pWin) > { > - CompScreenPtr cs = GetCompScreen (pScreen); > + ScreenPtr pScreen = pWin->drawable.pScreen; > + CompScreenPtr cs = GetCompScreen (pScreen); > > compCheckTree (pScreen); > - if (cs->damaged) > - { > - compWindowUpdate (pScreen->root); > - cs->damaged = FALSE; > + if (cs->damaged) { > + WindowPtr pChild; > + > + for (pChild = pWin->lastChild; pChild; pChild = pChild->prevSib) > + compWindowUpdate (pChild); > + > + if (pWin == pScreen->root) > + cs->damaged = FALSE;
compWindowUpdate walks all the children itself, so you should be able to replace the for loop with a simple compWindowUpdate(pWin). Other than that, it looks fine to me, so for the series: Reviewed-by: Daniel Stone <[email protected]> Cheers, Daniel
signature.asc
Description: Digital signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
