Stop using GStaticMutex in two testcases
authorRyan Lortie <desrt@desrt.ca>
Sat, 17 Sep 2011 23:33:30 +0000 (19:33 -0400)
committerRyan Lortie <desrt@desrt.ca>
Wed, 21 Sep 2011 20:06:53 +0000 (16:06 -0400)
gthread/tests/gwakeuptest.c
tests/thread-test.c

index 4b11f17..097441e 100644 (file)
@@ -78,7 +78,7 @@ struct token
 struct context
 {
   GSList *pending_tokens;
-  GStaticMutex lock;
+  GMutex lock;
   GWakeup *wakeup;
   gboolean quit;
 };
@@ -95,10 +95,8 @@ static volatile gint tokens_alive;
 static void
 context_init (struct context *ctx)
 {
-  GStaticMutex lock = G_STATIC_MUTEX_INIT;
-
   ctx->pending_tokens = NULL;
-  ctx->lock = lock;
+  g_mutex_init (&ctx->lock);
   ctx->wakeup = g_wakeup_new ();
   ctx->quit = FALSE;
 }
@@ -124,11 +122,11 @@ context_pop_token (struct context *ctx)
 {
   struct token *token;
 
-  g_static_mutex_lock (&ctx->lock);
+  g_mutex_lock (&ctx->lock);
   token = ctx->pending_tokens->data;
   ctx->pending_tokens = g_slist_remove_link (ctx->pending_tokens,
                                              ctx->pending_tokens);
-  g_static_mutex_unlock (&ctx->lock);
+  g_mutex_unlock (&ctx->lock);
 
   return token;
 }
@@ -139,9 +137,9 @@ context_push_token (struct context *ctx,
 {
   g_assert (token->owner == ctx);
 
-  g_static_mutex_lock (&ctx->lock);
+  g_mutex_lock (&ctx->lock);
   ctx->pending_tokens = g_slist_prepend (ctx->pending_tokens, token);
-  g_static_mutex_unlock (&ctx->lock);
+  g_mutex_unlock (&ctx->lock);
 
   g_wakeup_signal (ctx->wakeup);
 }
index 709c387..fa4f02b 100644 (file)
@@ -110,25 +110,25 @@ test_g_static_rec_mutex (void)
 
 static GStaticPrivate test_g_static_private_private1 = G_STATIC_PRIVATE_INIT;
 static GStaticPrivate test_g_static_private_private2 = G_STATIC_PRIVATE_INIT;
-static GStaticMutex test_g_static_private_mutex = G_STATIC_MUTEX_INIT;
+static GMutex test_g_static_private_mutex = G_MUTEX_INIT;
 static guint test_g_static_private_counter = 0;
 static guint test_g_static_private_ready = 0;
 
 static gpointer
 test_g_static_private_constructor (void)
 {
-  g_static_mutex_lock (&test_g_static_private_mutex);
+  g_mutex_lock (&test_g_static_private_mutex);
   test_g_static_private_counter++;
-  g_static_mutex_unlock (&test_g_static_private_mutex);  
+  g_mutex_unlock (&test_g_static_private_mutex);  
   return g_new (guint,1);
 }
 
 static void
 test_g_static_private_destructor (gpointer data)
 {
-  g_static_mutex_lock (&test_g_static_private_mutex);
+  g_mutex_lock (&test_g_static_private_mutex);
   test_g_static_private_counter--;
-  g_static_mutex_unlock (&test_g_static_private_mutex);  
+  g_mutex_unlock (&test_g_static_private_mutex);  
   g_free (data);
 }
 
@@ -162,9 +162,9 @@ test_g_static_private_thread (gpointer data)
       g_assert (number == *private1);
       g_assert (number * 2 == *private2);      
     }
-  g_static_mutex_lock (&test_g_static_private_mutex);
+  g_mutex_lock (&test_g_static_private_mutex);
   test_g_static_private_ready++;
-  g_static_mutex_unlock (&test_g_static_private_mutex);  
+  g_mutex_unlock (&test_g_static_private_mutex);  
 
   /* Busy wait is not nice but that's just a test */
   while (test_g_static_private_ready != 0)