[PATCH libFS] include X11/Xmd.h in fs_transport

Alan Coopersmith alan.coopersmith at oracle.com
Sun Jul 26 11:57:47 PDT 2015


On 07/26/15 11:16 AM, Julien Cristau wrote:
> Xtrans defines BytesReadable_t as int or long depending on whether
> LONG64 is defined.  So on a 64bit build it would be int when building
> FSConnServ.c, but long for fs_transport.c.  To fix this inconsistency,
> make fs_transport.c include <X11/Xmd.h>.
>
> Debian bug#749120
>
> Reported-by: Michael Tautschnig <mt at debian.org>
> Signed-off-by: Julien Cristau <jcristau at debian.org>
> ---
>   src/fs_transport.c | 1 +
>   1 file changed, 1 insertion(+)
>
> I'm not sure if this should be fixed in xtrans instead...

Yeah - I think we should add the #include <X11/Xmd.h> to Xtrans.h, not
force every consumer to add it before including Xtrans.h.

-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list