Dave Airlie <[email protected]> writes: > From: Dave Airlie <[email protected]> > > If we don't free this here, it gets freed later in the resource > cleanups, however it then looks up up pmap->pScreen, which we > freed already in this function. So free the default colormap > when we should.
Is there anything else hanging off of a GPU screen that also needs to get freed here? I don't think so, but this would be a good time to make sure. I'd love to have the commit comment in the code as well so that we don't see the free later and think 'why do this one specially?' Reviewed-by: Keith Packard <[email protected]> -- [email protected]
pgp3cBsp4wKdn.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
