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

[Hans de Goede]

Hi,

On 07-09-16 15:06, Keith Packard wrote:
> 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.

Ok, I've added such comments to the version in me personal tree.

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

Thanks.

Regards,

Hans