This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
Bootstrap failure (Re: [Patch] Java: Prepare for %q, %< and %> in diagnostic messages)
- From: Andrew Haley <aph at redhat dot com>
- To: Ulrich Weigand <weigand at i1 dot informatik dot uni-erlangen dot de>
- Cc: java-patches at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org, rmathew at gmail dot com
- Date: Wed, 6 Oct 2004 13:45:25 +0100
- Subject: Bootstrap failure (Re: [Patch] Java: Prepare for %q, %< and %> in diagnostic messages)
- References: <200410052010.WAA13716@faui1m.informatik.uni-erlangen.de>
Ulrich Weigand writes:
> Hello,
>
> this causes bootstrap failure on s390(x) and presumably all other
> platforms where va_list is an array type.
>
> Note that if va_list is an array type, it decays to a pointer
> when used a formal parameter like here:
> >@@ -3141,7 +3143,7 @@ issue_warning_error_from_context (
> >+ const char *msgid, va_list ap)
>
> and thus &ap does *not* have type va_list *, so this assignment
> >+ text.args_ptr = ≈
> fails due to type mismatch.
>
> The patch below fixes the problem by passing in a va_list *
> to issue_warning_error_from_context in the first place.
>
> Bootstrapped/regtested on s390-ibm-linux and s390x-ibm-linux.
> OK for mainline?
>
> Bye,
> Ulrich
>
>
> ChangeLog:
>
> * parse.y (issue_warning_error_from_context): Use va_list *
> instead of va_list parameter.
> (parse_error_context): Update call.
> (parse_warning_context): Likewise.
OK.
Andrew.