[PATCH 00/18] xextproto repository cleanup

Keith Packard keithp at keithp.com
Wed Jun 24 23:34:52 PDT 2009


On Thu, 2009-06-25 at 15:26 +1000, Peter Hutterer wrote:

> Anyway. The approach in this patch series is the following:
> for any header file Foo.h, if Foo.h is a (or contains) Xlib headers
> - remove protocol constants into a Fooconst.h
> - move protocol structures into a Foostr.h (or add to it if exists)
> - move the leftover Foo.h to libXext/include and include Fooconst.h from the
>   file.

The 'new' approach has been to create three files for extension <foo>:

<foo>.h - constants used by the wire encoding and Xlib API

X<foo>.h - Xlib API defines (this must include <foo>.h>)

<foo>proto.h - protocol structures for the wire encoding
               (this must include <foo>.h)

Apps do:

 #include <X11/extensions/X<foo>.h>

The Xlib code does:

 #include <X11/extensions/X<foo>.h>
 #include <X11/extensions/<foo>proto.h>

The X server does:

 #include <X11/extensions/<foo>proto.h>

From what I can tell, you're following the general convention, but using
different naming, right?

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.x.org/archives/xorg-devel/attachments/20090624/6167d734/attachment.pgp 


More information about the xorg-devel mailing list