[PATCH v5 xserver 1/7] dix: Add dixPrivatesCreated helper function

Keith Packard keithp at keithp.com
Wed Sep 7 13:06:22 UTC 2016


Hans de Goede <hdegoede at redhat.com> writes:

> TL;DR: Yes this is not ideal, but the workaround
> is tiny and the improvements this patch-set brings
> are large, so I do not consider this a blocker.

Yeah, I was reading that code and it looks possible to make it work in
the future, but it's not something that's going to happen before
1.19. I'll see if I can't untangle that code enough to at least add some
more comments to it.

So, if you're willing to add a comment in patch 7/7 describing the
kludge and what would be necessary to remove it, the then this is 
fine with me.

1/7: Reviewed-by: Keith Packard <keithp at keithp.com>

-- 
-keith
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <https://lists.x.org/archives/xorg-devel/attachments/20160907/b8d04d46/attachment.sig>


More information about the xorg-devel mailing list