Re: [Qemu-devel] cvs cpu-exec.c (bug?)

[renzo@cs.unibo.it (Renzo Davoli)]

I have just returned from a conference and I have just checked out the
latest version from the cvs.

My compiler refused to compile the cpu-exec.c without the following
trivial patch, in line 906:

<int cpu_signal_handler(int host_signum, siginfo *info,
<                       void *puc)
>int cpu_signal_handler(int host_signum, struct siginfo *info,
>                       void *puc)

the keyword "struct" is missing defining the argument named info.
Maybe several other compilers have the implicit typedef on structs and
give either no error or just a warning.

For the sake of portability (and consistency, siginfo is always named as
"struct" in the remaining of the code) I feel it should be better to fix
it.

Thanks. Ciao.
		renzo