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?

-- 
[email protected]

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
xorg-devel mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to