Re: Patch: FYI: fix classpath/23863

[Marco Trudel <mtrudel at gmx dot ch>]

Hello Tom

The new function _calloc_r from your patch 
(http://gcc.gnu.org/ml/java-patches/2006-q2/msg00347.html) breaks 
compilation on cygwin. Function:

static void *
_calloc_r (void *ignore, size_t x1, size_t x2)
{
  char *result = (char *) malloc (x1 * x2);
  memset (result, 0, x1 * x2);
  return result;
}

/cygdrive/d/marco/GCC/build/./gcc/xgcc 
-B/cygdrive/d/marco/GCC/build/./gcc/ -B/c
ygdrive/d/marco/GCC/out/i686-pc-cygwin/bin/ 
-B/cygdrive/d/marco/GCC/out/i686-pc-
cygwin/lib/ -isystem /cygdrive/d/marco/GCC/out/i686-pc-cygwin/include 
-isystem /
cygdrive/d/marco/GCC/out/i686-pc-cygwin/sys-include -DHAVE_CONFIG_H -I. 
-I/cygdr
ive/d/marco/GCC/source/libjava/classpath/native/fdlibm -I../../include 
-O2 -g -O
2 -MT mprec.lo -MD -MP -MF .deps/mprec.Tpo -c 
/cygdrive/d/marco/GCC/source/libja
va/classpath/native/fdlibm/mprec.c -o mprec.o
In file included from 
/cygdrive/d/marco/GCC/source/libjava/classpath/native/fdlibm/mprec.c:88:
/cygdrive/d/marco/GCC/source/libjava/classpath/native/fdlibm/mprec.h:297:1: 
warning: "_EXFUN" redefined
In file included from /usr/include/assert.h:9,
                 from 
/cygdrive/d/marco/GCC/source/libjava/classpath/native/fdlibm/mprec.c:84:
/usr/include/_ansi.h:36:1: warning: this is the location of the previous 
definition
/cygdrive/d/marco/GCC/source/libjava/classpath/native/fdlibm/mprec.c:96:1: 
warning: "_REENT_CHECK_MP" redefined
In file included from /usr/include/stdlib.h:16,
                 from 
/cygdrive/d/marco/GCC/source/libjava/classpath/native/fdlibm/mprec.c:85:
/usr/include/sys/reent.h:754:1: warning: this is the location of the 
previous definition
/cygdrive/d/marco/GCC/source/libjava/classpath/native/fdlibm/mprec.c:97:1: 
warning: "_REENT_MP_FREELIST" redefined
/usr/include/sys/reent.h:769:1: warning: this is the location of the 
previous definition
/cygdrive/d/marco/GCC/source/libjava/classpath/native/fdlibm/mprec.c:98:1: 
warning: "_REENT_MP_P5S" redefined
/usr/include/sys/reent.h:768:1: warning: this is the location of the 
previous definition
/cygdrive/d/marco/GCC/source/libjava/classpath/native/fdlibm/mprec.c:101: 
warning: useless type name in empty declaration
/cygdrive/d/marco/GCC/source/libjava/classpath/native/fdlibm/mprec.c:104: 
error: static declaration of 'calloc' follows non-static declaration
/cygdrive/d/marco/GCC/source/libjava/classpath/native/fdlibm/mprec.c:105: 
error: expected identifier or '(' before '{' token
make[5]: *** [mprec.lo] Error 1
make[5]: Leaving directory 
`/cygdrive/d/marco/GCC/build/i686-pc-cygwin/libjava/classpath/native/fdlibm'
make[4]: *** [all-recursive] Error 1
make[4]: Leaving directory 
`/cygdrive/d/marco/GCC/build/i686-pc-cygwin/libjava/classpath/native'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory 
`/cygdrive/d/marco/GCC/build/i686-pc-cygwin/libjava/classpath'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory 
`/cygdrive/d/marco/GCC/build/i686-pc-cygwin/libjava'
make[1]: *** [all-target-libjava] Error 2
make[1]: Leaving directory `/cygdrive/d/marco/GCC/build'
make: *** [all] Error 2

I can fix the static error by remove the static specifier, but I have no 
idea what the second error could mean... Any ideas?


Marco