src/config.in: size_t and !HAVE_SIZE_T [PATCH]

src/config.in: size_t and !HAVE_SIZE_T [PATCH]

[Richard Dawe]

Hello.

I tried to build Emacs CVS against DJGPP CVS yesterday.

The DJGPP build uses DJGPP's <sys/config.h>, which defines HAVE_*
and other things that configure scripts detect. DJGPP's <sys/config.h>
does not define HAVE_SIZE_T, so the definition of size_t is used
from Emacs's src/config.in. Unfortunately there appears to be a typo
in the definition.

Below is a patch to fix the typo. One thing: why is it enclosed in
#ifdef NOT_C_CODE/#endif? Is src/config.h used in non-C code?

2002-12-30  Richard Dawe  <rich@phekda.freeserve.co.uk>

	* src/config.in (!HAVE_SIZE_T): Fix order of arguments in
	type definition of size_t.

Regards, Rich =]

Index: src/config.in
===================================================================
RCS file: /cvsroot/emacs/emacs/src/config.in,v
retrieving revision 1.178
diff -p -u -3 -r1.178 config.in
--- src/config.in	29 Nov 2002 06:25:05 -0000	1.178
+++ src/config.in	30 Dec 2002 10:28:12 -0000
@@ -961,7 +961,7 @@ char *alloca ();
 # endif /* HAVE_ALLOCA_H */
 #endif /* __GNUC__ */
 #ifndef HAVE_SIZE_T
-typedef size_t unsigned
+typedef unsigned size_t;
 #endif
 #endif /* NOT_C_CODE */

Re: src/config.in: size_t and !HAVE_SIZE_T [PATCH]

[Eli Zaretskii]

> Date: Mon, 30 Dec 2002 11:05:32 +0000
> From: "Richard Dawe" <rich@phekda.freeserve.co.uk>
> 
> Below is a patch to fix the typo. One thing: why is it enclosed in
> #ifdef NOT_C_CODE/#endif? Is src/config.h used in non-C code?

Yes, it's used when Makefile.in is processed by cpp.

Re: src/config.in: size_t and !HAVE_SIZE_T [PATCH]

[Eli Zaretskii]

> Date: Mon, 30 Dec 2002 11:05:32 +0000
> From: "Richard Dawe" <rich@phekda.freeserve.co.uk>
> 
> The DJGPP build uses DJGPP's <sys/config.h>, which defines HAVE_*
> and other things that configure scripts detect. DJGPP's <sys/config.h>
> does not define HAVE_SIZE_T, so the definition of size_t is used
> from Emacs's src/config.in. Unfortunately there appears to be a typo
> in the definition.
> 
> Below is a patch to fix the typo.

Thanks.