Re: va_arg *, and why not.

[Gabriel Dos Reis <gdr at codesourcery dot com>]

Mark Mitchell <mark@codesourcery.com> writes:

| >>>>> "Gabriel" == Gabriel Dos Reis <gdr@codesourcery.com> writes:
| 
|     Gabriel> Hmm, I'm confused there.  I was under the impression that
|     Gabriel> it was taking the address of a va_list function argument
|     Gabriel> that was not portable.  Furthermore, in his proposed
|     Gabriel> resolution using va_copy was one option.
| 
| But with the caveat that there be a libiberty/system.h equivalent to
| handle systems that do not define va_copy.

Aha, got it.

So what about changing the prototype of report_diagnostic() to

  void report_diagnostic (const char *, va_list *, const char *, int, int)

?

That would solve the need for va_copy.

-- Gaby
CodeSourcery, LLC                             http://www.codesourcery.com