[PATCH] render.h: Include X.h for definition of XID

David Sharp whereami at gmail.com
Mon Oct 22 18:35:25 PDT 2007


On 10/22/07, Matthias Hopf <mhopf at suse.de> wrote:
> On Oct 09, 07 21:47:26 -0700, David Sharp wrote:
> > No replies... should I open a bug?
>
> Considering how all the other include files behave, any source has to
> include X.h before including other X header files anyway. So the correct
> fix would be in mpx to include X.h before render.h

I had wondered if that was the case. I think this is strange, but I
won't argue with the status quo in this case.

> P.S. it typically helps to state the package your patch should be
> applied to. Trivial in this case, but that might not always be the case.

I'll keep that in mind. Thanks.

> Matthias
>
> --
> Matthias Hopf <mhopf at suse.de>      __        __   __
> Maxfeldstr. 5 / 90409 Nuernberg   (_   | |  (_   |__          mat at mshopf.de
> Phone +49-911-74053-715           __)  |_|  __)  |__  R & D   www.mshopf.de
> _______________________________________________
> xorg mailing list
> xorg at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/xorg
>



More information about the xorg mailing list