[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#749120: [PATCH libFS] include X11/Xmd.h in fs_transport



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@debian.org>
Signed-off-by: Julien Cristau <jcristau@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@oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


Reply to: