| Message ID | 1365885950-27598-1-git-send-email-sw@weilnetz.de |
|---|---|
| State | Accepted |
| Headers | show |
On Sat, Apr 13, 2013 at 10:45:50PM +0200, Stefan Weil wrote: > GetLastError() returns a DWORD value which is unsigned long, > so the correct format specifier is %lu. > > Signed-off-by: Stefan Weil <sw@weilnetz.de> > --- > > Debian's w32 cross compiler does not complain about the wrong > specifiers, but the w64 cross compiler does. > > I'm sorry that I did not notice this before sending the pull > request. > > > cpus.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Thanks, applied to the trivial patches tree: https://github.com/stefanha/qemu/commits/trivial-patches Stefan
diff --git a/cpus.c b/cpus.c index 97e9ab4..3f754c6 100644 --- a/cpus.c +++ b/cpus.c @@ -865,7 +865,7 @@ static void qemu_cpu_kick_thread(CPUState *cpu) CONTEXT tcgContext; if (SuspendThread(cpu->hThread) == (DWORD)-1) { - fprintf(stderr, "qemu:%s: GetLastError:%d\n", __func__, + fprintf(stderr, "qemu:%s: GetLastError:%lu\n", __func__, GetLastError()); exit(1); } @@ -881,7 +881,7 @@ static void qemu_cpu_kick_thread(CPUState *cpu) cpu_signal(0); if (ResumeThread(cpu->hThread) == (DWORD)-1) { - fprintf(stderr, "qemu:%s: GetLastError:%d\n", __func__, + fprintf(stderr, "qemu:%s: GetLastError:%lu\n", __func__, GetLastError()); exit(1); }
GetLastError() returns a DWORD value which is unsigned long, so the correct format specifier is %lu. Signed-off-by: Stefan Weil <sw@weilnetz.de> --- Debian's w32 cross compiler does not complain about the wrong specifiers, but the w64 cross compiler does. I'm sorry that I did not notice this before sending the pull request. cpus.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)