Correction for Issue #589

usleep() should not be retried when rc!=0, and only issue HHC00075E when errno!=EINTR. This approach may be improved upon alltogether by replacing the usleep() call in herc_usleep with nanosleep().
SDL-Hercules-390 · Aug 8, 2023 · bf1a5d7 · bf1a5d7
1 parent c962c5d
commit bf1a5d7
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/hscutl.c b/hscutl.c
@@ -609,13 +609,9 @@ DLL_EXPORT int herc_usleep( useconds_t usecs, const char* file, int line )
 {
     int  rc, save_errno;
 
-    while (1
+    if (1
         && (rc = usleep( usecs )) != 0
-        && (save_errno = errno) == EINTR
-    )
-        continue;
-
-    if (rc != 0)
+        && (save_errno = errno) != EINTR)
     {
         char fnc[128], msg[128];