# SDL: testthread: parse arguments using SDLTest\_CommonState + add arguments

**URL:** <https://discourse.libsdl.org/t/sdl-testthread-parse-arguments-using-sdltest-commonstate-add-arguments/43174>\
**Category:** SDL Commits\
**Created:** [March 17, 2023, 4:57pm UTC](https://discourse.libsdl.org/t/sdl-testthread-parse-arguments-using-sdltest-commonstate-add-arguments/43174 "2023-03-17T16:57:13Z")\
**Posts on this page:** 1\
**Page:** 1

<div class="post-metadata">

**Author:** ![SDLPushPostBot](https://discourse.libsdl.org/user_avatar/discourse.libsdl.org/sdlpushpostbot/32/4370_2.png) [@SDLPushPostBot](https://discourse.libsdl.org/u/SDLPushPostBot)\
**Post date:** [March 17, 2023, 4:57pm UTC](https://discourse.libsdl.org/t/sdl-testthread-parse-arguments-using-sdltest-commonstate-add-arguments/43174/1 "2023-03-17T16:57:13Z")

</div>

> <https://github.com/libsdl-org/SDL/commit/8bea41f7379f4bb43d862a828560853b6b785eee>

```diff
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);

```