[glib/wip/mutexes: 11/12] Stop using GStaticMutex in two testcases



commit 595d04847c24b84f2c435ead23a09e42b087fea4
Author: Ryan Lortie <desrt desrt ca>
Date:   Sat Sep 17 19:33:30 2011 -0400

    Stop using GStaticMutex in two testcases

 gthread/tests/gwakeuptest.c |   14 ++++++--------
 tests/thread-test.c         |   14 +++++++-------
 2 files changed, 13 insertions(+), 15 deletions(-)
---
diff --git a/gthread/tests/gwakeuptest.c b/gthread/tests/gwakeuptest.c
index 4b11f17..097441e 100644
--- a/gthread/tests/gwakeuptest.c
+++ b/gthread/tests/gwakeuptest.c
@@ -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);
 }
diff --git a/tests/thread-test.c b/tests/thread-test.c
index 709c387..fa4f02b 100644
--- a/tests/thread-test.c
+++ b/tests/thread-test.c
@@ -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)



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]