I like this series, though I'm not quite comfortable enough with the
code to offer a reviewed-by. It seems like a good step towards merging
xfree86 DDX functionality into DIX and eventually killing the
distinction between DDX and DIX though, so I'm all for it.

One note on this patch:

On Sun, Aug 01, 2010 at 09:28:23PM +0200, Tomas Carnecky wrote:
> -INCLUDES = -I$(top_srcdir)/hw/xfree86/dixmods/extmod @XORG_INCS@
> +INCLUDES = @XORG_INCS@

Doesn't this change belong in the earlier patch that deletes the extmod
directory?

Jamey

Attachment: signature.asc
Description: Digital signature

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to