Patchwork [1.1] qemu-timer: Fix wrong error message

login
register
mail settings
Submitter Stefan Weil
Date May 12, 2012, 6:51 a.m.
Message ID <1336805512-18492-1-git-send-email-sw@weilnetz.de>
Download mbox | patch
Permalink /patch/158692/
State Accepted
Headers show

Comments

Stefan Weil - May 12, 2012, 6:51 a.m.
Function timeSetEvent returns 0 when it fails, but it does not set
an error code which can be retrieved by GetLastError.

Therefore calling GetLastError is useless.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
---
 qemu-timer.c |    7 ++-----
 1 files changed, 2 insertions(+), 5 deletions(-)

Patch

diff --git a/qemu-timer.c b/qemu-timer.c
index 4a6fc98..de98977 100644
--- a/qemu-timer.c
+++ b/qemu-timer.c
@@ -635,8 +635,7 @@  static int mm_start_timer(struct qemu_alarm_timer *t)
                             TIME_ONESHOT | TIME_CALLBACK_FUNCTION);
 
     if (!mm_timer) {
-        fprintf(stderr, "Failed to initialize win32 alarm timer: %ld\n",
-                GetLastError());
+        fprintf(stderr, "Failed to initialize win32 alarm timer\n");
         timeEndPeriod(mm_tc.wPeriodMin);
         return -1;
     }
@@ -667,9 +666,7 @@  static void mm_rearm_timer(struct qemu_alarm_timer *t, int64_t delta)
                             TIME_ONESHOT | TIME_CALLBACK_FUNCTION);
 
     if (!mm_timer) {
-        fprintf(stderr, "Failed to re-arm win32 alarm timer %ld\n",
-                GetLastError());
-
+        fprintf(stderr, "Failed to re-arm win32 alarm timer\n");
         timeEndPeriod(mm_tc.wPeriodMin);
         exit(1);
     }