On Sun, 1 Aug 2010 14:39:46 -0400, Kristian Høgsberg <[email protected]> wrote: > Before this gets drowned out in janitorial patches... Keith, do you > want a pull request for this and the damageext patch? Did you have > have a look at the damage change?
I read through the change and reviewed the potential impacts on the server. I'm a little concerned about having the semantics of the damage extension accidentally change as the code paths for the post-activity damage region processing. And, I'd love to have that code 'fixed' so that there wasn't additional region processing needed. So, it's not my favorite change, but anything additional seems like a riskier proposition for 1.9. So, I was planning to merge the damage change along with your other fixes for 1.9. It'd be lovely to get other people to take another look at the damage code to see if they can identify potential issues with your change. -- [email protected]
pgp958SArrUAX.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org support Archives: http://lists.freedesktop.org/archives/xorg Info: http://lists.freedesktop.org/mailman/listinfo/xorg Your subscription address: [email protected]
