SDL: testthread: parse arguments using SDLTest_CommonState + add arguments

From 8bea41f7379f4bb43d862a828560853b6b785eee Mon Sep 17 00:00:00 2001
From: Anonymous Maarten <[EMAIL REDACTED]>
Date: Thu, 16 Mar 2023 23:47:35 +0100
Subject: [PATCH] testthread: parse arguments using SDLTest_CommonState + add
 arguments

---
 test/testlock.c   |  6 +++---
 test/testthread.c | 38 ++++++++++++++++++++++++++++++--------
 2 files changed, 33 insertions(+), 11 deletions(-)

diff --git a/test/testlock.c b/test/testlock.c
index a0189e4b25e4..8c2ca5e89ad3 100644
--- a/test/testlock.c
+++ b/test/testlock.c
@@ -69,7 +69,7 @@ static void closemutex(int sig)
 }
 
 static int SDLCALL
-DoWork(void *data)
+Run(void *data)
 {
     if (SDL_ThreadID() == mainthread) {
         (void)signal(SIGTERM, closemutex);
@@ -195,7 +195,7 @@ int main(int argc, char *argv[])
     for (i = 0; i < nb_threads; ++i) {
         char name[64];
         (void)SDL_snprintf(name, sizeof(name), "Worker%d", i);
-        threads[i] = SDL_CreateThread(DoWork, name, &threads[i]);
+        threads[i] = SDL_CreateThread(Run, name, NULL);
         if (threads[i] == NULL) {
             SDL_LogError(SDL_LOG_CATEGORY_APPLICATION, "Couldn't create thread!\n");
         }
@@ -208,7 +208,7 @@ int main(int argc, char *argv[])
 #endif
 
     (void)signal(SIGINT, terminate);
-    DoWork(NULL);
+    Run(NULL);
 
     return 0; /* Never reached */
 }
diff --git a/test/testthread.c b/test/testthread.c
index 4e9f22fa018d..15297e320aed 100644
--- a/test/testthread.c
+++ b/test/testthread.c
@@ -17,15 +17,18 @@
 
 #include <SDL3/SDL.h>
 #include <SDL3/SDL_main.h>
+#include <SDL3/SDL_test.h>
 
 static SDL_TLSID tls;
 static int alive = 0;
 static int testprio = 0;
+SDLTest_CommonState *state;
 
 /* Call this instead of exit(), so we can clean up SDL: atexit() is evil. */
 static void
 quit(int rc)
 {
+    SDLTest_CommonDestroyState(state);
     SDL_Quit();
     exit(rc);
 }
@@ -82,12 +85,38 @@ killed(int sig)
 
 int main(int argc, char *argv[])
 {
-    int arg = 1;
+    int i;
     SDL_Thread *thread;
 
+    /* Initialize test framework */
+    state = SDLTest_CommonCreateState(argv, 0);
+    if (state == NULL) {
+        return 1;
+    }
+
     /* Enable standard application logging */
     SDL_LogSetPriority(SDL_LOG_CATEGORY_APPLICATION, SDL_LOG_PRIORITY_INFO);
 
+    /* Parse commandline */
+    for (i = 1; i < argc;) {
+        int consumed;
+
+        consumed = SDLTest_CommonArg(state, i);
+        if (!consumed) {
+            if (SDL_strcmp("--prio", argv[i]) == 0) {
+                testprio = 1;
+                consumed = 1;
+            }
+        }
+        if (consumed <= 0) {
+            static const char *options[] = { "[--prio]", NULL };
+            SDLTest_CommonLogUsage(state, argv[0], options);
+            exit(1);
+        }
+
+        i += consumed;
+    }
+
     /* Load the SDL library */
     if (SDL_Init(0) < 0) {
         SDL_LogError(SDL_LOG_CATEGORY_APPLICATION, "Couldn't initialize SDL: %s\n", SDL_GetError());
@@ -100,13 +129,6 @@ int main(int argc, char *argv[])
         return 0;
     }
 
-    while (argv[arg] && *argv[arg] == '-') {
-        if (SDL_strcmp(argv[arg], "--prio") == 0) {
-            testprio = 1;
-        }
-        ++arg;
-    }
-
     tls = SDL_TLSCreate();
     SDL_assert(tls);
     SDL_TLSSet(tls, "main thread", NULL);