On Tue, Jul 03, 2007 at 11:39:02AM +0200, Johan Bilien wrote: > On Tue, Jul 03, 2007, Bryce Harrington wrote: > > I've posted .debs of xorg-server with this patch, here: > > > > http://people.ubuntu.com/~bryce/Testing/xorg-server/ > > > > (I plan to start using PPA for these kinds of test packages in the > > future, but will beg forgiveness for just posting the raw .debs for > > now.) > > > > Btw, I noticed a discrepancy between keithp's original patch and the one > > proposed for Feisty. The former had a chunk in its patch that modified > > the code to add some comparisons to 'CompRedirectNone' (which doesn't > > appear to exist). Since I couldn't get that chunk to compile, I went > > with the second proposed-for-Fesity version, but I'm curious about this > > chunk, and if it's at all important? > > Yes I had the same problem, I'm not sure where CompRedirectNone comes > from, probably Keithp's tree. Anyway I think it's meant to be 0, so > removing the > > - if (pWin->redirectDraw) > + if (pWin->redirectDraw != CompRedirectNone) > > should be harmless. It works fine without this part.
Yep, from what I understood from the code, it looked safe to me too, although I'm curious what CompRedirectNone is supposed to be. My only other idea was that maybe he meant RedirectDrawNone there? Bryce -- Ubuntu-mobile mailing list Ubuntu-mobile@lists.ubuntu.com Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/ubuntu-mobile
