[PATCH 06/22] miext/shadow: Remove shadowInit
Keith Packard
keithp at keithp.com
Mon Jul 28 12:35:44 PDT 2014
Adam Jackson <ajax at redhat.com> writes:
> This code is nonsensical. You end up creating a screen-sized pixmap
> that's totally detached from everything else, which you then listen for
> damage on, which means you'll never hear any damage, which means your
> shadow update hooks will never get called. Any driver using this would
> be sorely disappointed.
I have this vague memory that a caller of this would actually pull the
pixmap out of the shadow code and use that as the screen pixmap. I'm not
sure that's correct, but I can't see how else this could be useful
either.
In any case, that would be crazy.
Reviewed-by: Keith Packard <keithp at keithp.com>
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140728/216f8df1/attachment.sig>
More information about the xorg-devel
mailing list