RE: boehm-gc support for s390 architecture

["Boehm, Hans" <hans_boehm at hp dot com>]

I've attached a diff between the 3.1 version of cordprnt.c and my current
development version.  This was also based on the Debian code, but it seems
slightly safer to me.  But I haven't done a lot of portability testing.  I'm
open to better ways of doing this.

Note that AFAIK, nothing in boehm-gc/cord is actually used by gcj.

Hans

> -----Original Message-----
> From: Gerhard Tonn [mailto:GerhardTonn@swol.de]
> Sent: Saturday, April 13, 2002 8:02 AM
> To: java-patches@gcc.gnu.org
> Subject: boehm-gc support for s390 architecture
> 
> 
> Hi,
> the s390 boehm-gc support as part of the 3.1 branch doesn't 
> compile for me, 
> since va_list assignment is not portable in cordprnt.c. 
> Attached is the patch 
> I am using in Debian, but it is probably Linux specific. Does 
> anyone know how 
> to fix it portable?
> 
> Regards,
> Gerhard
> 
> 
> diff -urN gcc-20020401-bak/boehm-gc/cord/cordprnt.c 
> gcc-20020401/boehm-gc/cord/cordprnt.c
> --- gcc-20020401-bak/boehm-gc/cord/cordprnt.c	Tue Oct  2 19:40:38 2001
> +++ gcc-20020401/boehm-gc/cord/cordprnt.c	Wed Apr  3 09:18:32 2002
> @@ -255,12 +255,13 @@
>              	/* Use standard sprintf to perform conversion */
>              	{
>              	    register char * buf;
> -            	    va_list vsprintf_args = args;
> +            	    va_list vsprintf_args;
>              	    	/* The above does not appear to be 
> sanctioned	*/
>              	    	/* by the ANSI C standard.		
> 	*/
>              	    int max_size = 0;
>              	    int res;
>              	    	
> +            	    __va_copy(vsprintf_args, args);
>              	    if (width == VARIABLE) width = va_arg(args, int);
>              	    if (prec == VARIABLE) prec = va_arg(args, int);
>              	    if (width != NONE) max_size = width;
>

Attachment: cord.diff
Description: Binary data