From fedca055e98901c88320c72813a4d15a4c91681e Mon Sep 17 00:00:00 2001
From: Chun-wei Fan <fanchunwei@src.gnome.org>
Date: Fri, 21 Oct 2011 17:44:27 +0800
Subject: [PATCH] Revert "Revert "Deprecate clutter_threads_init()""

This reverts commit 5419c11c99f721f2f9bf520c0444c053a88b3725.

This is another part to attempt to investigate on error 0xC0000264
on Windows
---
 clutter/clutter-main.c                    | 31 +++++------------------
 clutter/clutter-main.h                    |  4 +++
 tests/interactive/test-texture-async.c    |  4 +--
 tests/interactive/test-texture-material.c |  2 --
 tests/interactive/test-threads.c          |  2 --
 5 files changed, 13 insertions(+), 30 deletions(-)

diff --git a/clutter/clutter-main.c b/clutter/clutter-main.c
index 1e1efb340..7d4d8159a 100644
--- a/clutter/clutter-main.c
+++ b/clutter/clutter-main.c
@@ -34,7 +34,7 @@
  *   <para>Clutter is <emphasis>thread-aware</emphasis>: all operations
  *   performed by Clutter are assumed to be under the big Clutter lock,
  *   which is created when the threading is initialized through
- *   clutter_threads_init().</para>
+ *   clutter_init().</para>
  *   <example id="example-Thread-Init">
  *     <title>Thread Initialization</title>
  *     <para>The code below shows how to correctly initialize Clutter
@@ -44,12 +44,6 @@
  * int
  * main (int argc, char *argv[])
  * {
- *   /&ast; initialize GLib's threading support &ast;/
- *   g_thread_init (NULL);
- *
- *   /&ast; initialize Clutter's threading support &ast;/
- *   clutter_threads_init ();
- *
  *   /&ast; initialize Clutter &ast;/
  *   clutter_init (&amp;argc, &amp;argv);
  *
@@ -916,18 +910,6 @@ clutter_main (void)
     CLUTTER_TIMER_STOP (uprof_get_mainloop_context (), mainloop_timer);
 }
 
-static void
-clutter_threads_impl_lock (void)
-{
-  g_mutex_lock (&clutter_threads_mutex);
-}
-
-static void
-clutter_threads_impl_unlock (void)
-{
-  g_mutex_unlock (&clutter_threads_mutex);
-}
-
 /**
  * clutter_threads_init:
  *
@@ -942,15 +924,13 @@ clutter_threads_impl_unlock (void)
  * It is safe to call this function multiple times.
  *
  * Since: 0.4
+ *
+ * Deprecated: 1.10: This function does not do anything. Threading support
+ *   is initialized when Clutter is initialized.
  */
 void
 clutter_threads_init (void)
 {
-  if (clutter_threads_lock == NULL)
-    clutter_threads_lock = clutter_threads_impl_lock;
-
-  if (clutter_threads_unlock == NULL)
-    clutter_threads_unlock = clutter_threads_impl_unlock;
 }
 
 /**
@@ -2604,6 +2584,9 @@ clutter_base_init (void)
 
       /* initialise GLib type system */
       g_type_init ();
+
+      /* initialise the Big Clutter Lock™ if necessary */
+      clutter_threads_init_default ();
     }
 }
 
diff --git a/clutter/clutter-main.h b/clutter/clutter-main.h
index 3d9da1799..a50405ad1 100644
--- a/clutter/clutter-main.h
+++ b/clutter/clutter-main.h
@@ -110,7 +110,11 @@ gulong           clutter_get_timestamp              (void);
 gboolean         clutter_get_accessibility_enabled  (void);
 
 /* Threading functions */
+#if !defined(CLUTTER_DISABLE_DEPRECATED) || defined(CLUTTER_COMPILATION)
+CLUTTER_DEPRECATED
 void             clutter_threads_init                  (void);
+#endif
+
 void             clutter_threads_enter                 (void);
 void             clutter_threads_leave                 (void);
 void             clutter_threads_set_lock_functions    (GCallback enter_fn,
diff --git a/tests/interactive/test-texture-async.c b/tests/interactive/test-texture-async.c
index 7e09f3642..20d802b24 100644
--- a/tests/interactive/test-texture-async.c
+++ b/tests/interactive/test-texture-async.c
@@ -120,8 +120,6 @@ test_texture_async_main (int argc, char *argv[])
 {
   gchar *path;
 
-  clutter_threads_init ();
-
   if (clutter_init (&argc, &argv) != CLUTTER_INIT_SUCCESS)
     return 1;
 
@@ -140,7 +138,9 @@ test_texture_async_main (int argc, char *argv[])
                                     task, path,
                                     cleanup_task);
 
+  clutter_threads_enter ();
   clutter_main ();
+  clutter_threads_leave ();
 
   g_free (path);
 
diff --git a/tests/interactive/test-texture-material.c b/tests/interactive/test-texture-material.c
index 867711f79..4886ce03a 100644
--- a/tests/interactive/test-texture-material.c
+++ b/tests/interactive/test-texture-material.c
@@ -10,8 +10,6 @@ test_texture_material_main (int argc, char *argv[])
   ClutterLayoutManager *manager;
   int i;
 
-  g_thread_init (NULL);
-  clutter_threads_init ();
   if (clutter_init (&argc, &argv) != CLUTTER_INIT_SUCCESS)
     return 1;
 
diff --git a/tests/interactive/test-threads.c b/tests/interactive/test-threads.c
index 5bfab8464..01ca8a766 100644
--- a/tests/interactive/test-threads.c
+++ b/tests/interactive/test-threads.c
@@ -209,8 +209,6 @@ test_threads_main (int argc, char *argv[])
     { 400, 150 }
   };
 
-  g_thread_init (NULL);
-  clutter_threads_init ();
   if (clutter_init (&argc, &argv) != CLUTTER_INIT_SUCCESS)
     return 1;