Matt Turner <[email protected]> writes:

>> +#include       <X11/fonts/libxfont.h>
>
> I don't know where this header is supposed to be (maybe it's supposed
> to be libxfont_2.h?), but it's not there and causes a build failure.
> Simply removing the #include lets it compile without warnings.

Yeah, I refactored the includes and now this file just includes
libxfontint.h, which includes libxfont_2.h, which used to be called
libxfont.h until just before I pushed the patch.

> Space before tab on this line

This file was moved from util/, and I didn't clean up the whitespace
after doing that. I've done so now.

Thanks much for your review; this is not the final API for this library,
as the real goal is to eliminate the select masks everywhere, this was
just some API sanitation before I merged the rest of the work.

-- 
-keith

Attachment: signature.asc
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to