Changed signature of all g_static_mutex_* functions to take a pointer
authorSebastian Wilhelmi <wilhelmi@ira.uka.de>
Mon, 15 Feb 1999 08:26:00 +0000 (08:26 +0000)
committerSebastian Wilhelmi <wilhelmi@src.gnome.org>
Mon, 15 Feb 1999 08:26:00 +0000 (08:26 +0000)
1999-02-15  Sebastian Wilhelmi  <wilhelmi@ira.uka.de>

* glib.h, glibconfig.h.win32, configure.in: Changed signature of
all g_static_mutex_* functions to take a pointer rather than the
struct itself. This is not an issue at the moment, because those
funcs are really macros, but is it un'G'ish and might fall on our
feet in the future.

* gthread/testgthread.c (test_mutexes): Use new signature of
g_static_mutex* functions.

14 files changed:
ChangeLog
ChangeLog.pre-2-0
ChangeLog.pre-2-10
ChangeLog.pre-2-12
ChangeLog.pre-2-2
ChangeLog.pre-2-4
ChangeLog.pre-2-6
ChangeLog.pre-2-8
configure.in
glib.h
glib/glib.h
glibconfig.h.win32
gthread/ChangeLog
gthread/testgthread.c

index bb0b04b1a8dd60993a60865cca6b3b521db03ee2..de18db5261de182cf6b88eed15faf13a11b6af0e 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+1999-02-15  Sebastian Wilhelmi  <wilhelmi@ira.uka.de>
+
+       * glib.h, glibconfig.h.win32, configure.in: Changed signature of
+       all g_static_mutex_* functions to take a pointer rather than the
+       struct itself. This is not an issue at the moment, because those
+       funcs are really macros, but is it un'G'ish and might fall on our
+       feet in the future.
+
 Mon Feb 15 07:45:54 1999  Tim Janik  <timj@gtk.org>
 
        * configure.in (--enable-debug): default to yes for development trees.
index bb0b04b1a8dd60993a60865cca6b3b521db03ee2..de18db5261de182cf6b88eed15faf13a11b6af0e 100644 (file)
@@ -1,3 +1,11 @@
+1999-02-15  Sebastian Wilhelmi  <wilhelmi@ira.uka.de>
+
+       * glib.h, glibconfig.h.win32, configure.in: Changed signature of
+       all g_static_mutex_* functions to take a pointer rather than the
+       struct itself. This is not an issue at the moment, because those
+       funcs are really macros, but is it un'G'ish and might fall on our
+       feet in the future.
+
 Mon Feb 15 07:45:54 1999  Tim Janik  <timj@gtk.org>
 
        * configure.in (--enable-debug): default to yes for development trees.
index bb0b04b1a8dd60993a60865cca6b3b521db03ee2..de18db5261de182cf6b88eed15faf13a11b6af0e 100644 (file)
@@ -1,3 +1,11 @@
+1999-02-15  Sebastian Wilhelmi  <wilhelmi@ira.uka.de>
+
+       * glib.h, glibconfig.h.win32, configure.in: Changed signature of
+       all g_static_mutex_* functions to take a pointer rather than the
+       struct itself. This is not an issue at the moment, because those
+       funcs are really macros, but is it un'G'ish and might fall on our
+       feet in the future.
+
 Mon Feb 15 07:45:54 1999  Tim Janik  <timj@gtk.org>
 
        * configure.in (--enable-debug): default to yes for development trees.
index bb0b04b1a8dd60993a60865cca6b3b521db03ee2..de18db5261de182cf6b88eed15faf13a11b6af0e 100644 (file)
@@ -1,3 +1,11 @@
+1999-02-15  Sebastian Wilhelmi  <wilhelmi@ira.uka.de>
+
+       * glib.h, glibconfig.h.win32, configure.in: Changed signature of
+       all g_static_mutex_* functions to take a pointer rather than the
+       struct itself. This is not an issue at the moment, because those
+       funcs are really macros, but is it un'G'ish and might fall on our
+       feet in the future.
+
 Mon Feb 15 07:45:54 1999  Tim Janik  <timj@gtk.org>
 
        * configure.in (--enable-debug): default to yes for development trees.
index bb0b04b1a8dd60993a60865cca6b3b521db03ee2..de18db5261de182cf6b88eed15faf13a11b6af0e 100644 (file)
@@ -1,3 +1,11 @@
+1999-02-15  Sebastian Wilhelmi  <wilhelmi@ira.uka.de>
+
+       * glib.h, glibconfig.h.win32, configure.in: Changed signature of
+       all g_static_mutex_* functions to take a pointer rather than the
+       struct itself. This is not an issue at the moment, because those
+       funcs are really macros, but is it un'G'ish and might fall on our
+       feet in the future.
+
 Mon Feb 15 07:45:54 1999  Tim Janik  <timj@gtk.org>
 
        * configure.in (--enable-debug): default to yes for development trees.
index bb0b04b1a8dd60993a60865cca6b3b521db03ee2..de18db5261de182cf6b88eed15faf13a11b6af0e 100644 (file)
@@ -1,3 +1,11 @@
+1999-02-15  Sebastian Wilhelmi  <wilhelmi@ira.uka.de>
+
+       * glib.h, glibconfig.h.win32, configure.in: Changed signature of
+       all g_static_mutex_* functions to take a pointer rather than the
+       struct itself. This is not an issue at the moment, because those
+       funcs are really macros, but is it un'G'ish and might fall on our
+       feet in the future.
+
 Mon Feb 15 07:45:54 1999  Tim Janik  <timj@gtk.org>
 
        * configure.in (--enable-debug): default to yes for development trees.
index bb0b04b1a8dd60993a60865cca6b3b521db03ee2..de18db5261de182cf6b88eed15faf13a11b6af0e 100644 (file)
@@ -1,3 +1,11 @@
+1999-02-15  Sebastian Wilhelmi  <wilhelmi@ira.uka.de>
+
+       * glib.h, glibconfig.h.win32, configure.in: Changed signature of
+       all g_static_mutex_* functions to take a pointer rather than the
+       struct itself. This is not an issue at the moment, because those
+       funcs are really macros, but is it un'G'ish and might fall on our
+       feet in the future.
+
 Mon Feb 15 07:45:54 1999  Tim Janik  <timj@gtk.org>
 
        * configure.in (--enable-debug): default to yes for development trees.
index bb0b04b1a8dd60993a60865cca6b3b521db03ee2..de18db5261de182cf6b88eed15faf13a11b6af0e 100644 (file)
@@ -1,3 +1,11 @@
+1999-02-15  Sebastian Wilhelmi  <wilhelmi@ira.uka.de>
+
+       * glib.h, glibconfig.h.win32, configure.in: Changed signature of
+       all g_static_mutex_* functions to take a pointer rather than the
+       struct itself. This is not an issue at the moment, because those
+       funcs are really macros, but is it un'G'ish and might fall on our
+       feet in the future.
+
 Mon Feb 15 07:45:54 1999  Tim Janik  <timj@gtk.org>
 
        * configure.in (--enable-debug): default to yes for development trees.
index 428fb77d50f00744b2f95241fdf1f910c6764d0d..53ea07c2acadb861e80aa2a7f782c33a385b48db 100644 (file)
@@ -995,8 +995,8 @@ struct _GStaticMutex
 };
 #define        G_STATIC_MUTEX_INIT     { NULL, { { $g_mutex_contents} } }
 #define        g_static_mutex_get_mutex(mutex) \
-  (g_thread_use_default_impl ? ((GMutex*) &(mutex).aligned_pad_u) : \
-   g_static_mutex_get_mutex_impl (&(mutex).runtime_mutex))
+  (g_thread_use_default_impl ? ((GMutex*) &((mutex)->aligned_pad_u)) : \
+   g_static_mutex_get_mutex_impl (&((mutex)->runtime_mutex)))
 _______EOF
        else
                cat >>$outfile <<_______EOF
@@ -1004,7 +1004,7 @@ $g_enable_threads_def G_THREADS_ENABLED
 #define G_THREADS_IMPL_$g_threads_impl_def
 typedef struct _GMutex* GStaticMutex;
 #define G_STATIC_MUTEX_INIT NULL
-#define g_static_mutex_get_mutex(mutex) (g_static_mutex_get_mutex_impl (&(mutex)))
+#define g_static_mutex_get_mutex(mutex) (g_static_mutex_get_mutex_impl (mutex))
 _______EOF
        fi
 
diff --git a/glib.h b/glib.h
index 1f917b76283ab36ebef4faf2569e04cbf54456e1..78ea3e65ae330c243282e7f623765ebd25706a45 100644 (file)
--- a/glib.h
+++ b/glib.h
@@ -2741,25 +2741,25 @@ extern void glib_dummy_decl (void);
               "file %s: line %d (%s): locking: %s ",             \
               __FILE__,        __LINE__, G_GNUC_PRETTY_FUNCTION, \
                #name);                                            \
-        g_static_mutex_lock (G_LOCK_NAME (name));                 \
+        g_static_mutex_lock (&G_LOCK_NAME (name));                \
      }G_STMT_END
 #    define G_UNLOCK(name)             G_STMT_START{             \
         g_log (G_LOG_DOMAIN, G_LOG_LEVEL_DEBUG,                          \
               "file %s: line %d (%s): unlocking: %s ",           \
               __FILE__,        __LINE__, G_GNUC_PRETTY_FUNCTION, \
                #name);                                            \
-       g_static_mutex_unlock (G_LOCK_NAME (name));                \
+       g_static_mutex_unlock (&G_LOCK_NAME (name));               \
      }G_STMT_END
 #    define G_TRYLOCK(name)            G_STMT_START{             \
         g_log (G_LOG_DOMAIN, G_LOG_LEVEL_DEBUG,                          \
               "file %s: line %d (%s): try locking: %s ",         \
               __FILE__,        __LINE__, G_GNUC_PRETTY_FUNCTION, \
                #name);                                            \
-     }G_STMT_END,      g_static_mutex_trylock (G_LOCK_NAME (name))
+     }G_STMT_END,      g_static_mutex_trylock (&G_LOCK_NAME (name))
 #  else         /* !G_DEBUG_LOCKS */
-#    define G_LOCK(name) g_static_mutex_lock      (G_LOCK_NAME (name)) 
-#    define G_UNLOCK(name) g_static_mutex_unlock   (G_LOCK_NAME (name))
-#    define G_TRYLOCK(name) g_static_mutex_trylock (G_LOCK_NAME (name))
+#    define G_LOCK(name) g_static_mutex_lock      (&G_LOCK_NAME (name)) 
+#    define G_UNLOCK(name) g_static_mutex_unlock   (&G_LOCK_NAME (name))
+#    define G_TRYLOCK(name) g_static_mutex_trylock (&G_LOCK_NAME (name))
 #  endif /* !G_DEBUG_LOCKS */
 #else  /* !G_THREADS_ENABLED */
 #  define G_LOCK_DEFINE_STATIC(name)   extern void glib_dummy_decl (void)
index 1f917b76283ab36ebef4faf2569e04cbf54456e1..78ea3e65ae330c243282e7f623765ebd25706a45 100644 (file)
@@ -2741,25 +2741,25 @@ extern void glib_dummy_decl (void);
               "file %s: line %d (%s): locking: %s ",             \
               __FILE__,        __LINE__, G_GNUC_PRETTY_FUNCTION, \
                #name);                                            \
-        g_static_mutex_lock (G_LOCK_NAME (name));                 \
+        g_static_mutex_lock (&G_LOCK_NAME (name));                \
      }G_STMT_END
 #    define G_UNLOCK(name)             G_STMT_START{             \
         g_log (G_LOG_DOMAIN, G_LOG_LEVEL_DEBUG,                          \
               "file %s: line %d (%s): unlocking: %s ",           \
               __FILE__,        __LINE__, G_GNUC_PRETTY_FUNCTION, \
                #name);                                            \
-       g_static_mutex_unlock (G_LOCK_NAME (name));                \
+       g_static_mutex_unlock (&G_LOCK_NAME (name));               \
      }G_STMT_END
 #    define G_TRYLOCK(name)            G_STMT_START{             \
         g_log (G_LOG_DOMAIN, G_LOG_LEVEL_DEBUG,                          \
               "file %s: line %d (%s): try locking: %s ",         \
               __FILE__,        __LINE__, G_GNUC_PRETTY_FUNCTION, \
                #name);                                            \
-     }G_STMT_END,      g_static_mutex_trylock (G_LOCK_NAME (name))
+     }G_STMT_END,      g_static_mutex_trylock (&G_LOCK_NAME (name))
 #  else         /* !G_DEBUG_LOCKS */
-#    define G_LOCK(name) g_static_mutex_lock      (G_LOCK_NAME (name)) 
-#    define G_UNLOCK(name) g_static_mutex_unlock   (G_LOCK_NAME (name))
-#    define G_TRYLOCK(name) g_static_mutex_trylock (G_LOCK_NAME (name))
+#    define G_LOCK(name) g_static_mutex_lock      (&G_LOCK_NAME (name)) 
+#    define G_UNLOCK(name) g_static_mutex_unlock   (&G_LOCK_NAME (name))
+#    define G_TRYLOCK(name) g_static_mutex_trylock (&G_LOCK_NAME (name))
 #  endif /* !G_DEBUG_LOCKS */
 #else  /* !G_THREADS_ENABLED */
 #  define G_LOCK_DEFINE_STATIC(name)   extern void glib_dummy_decl (void)
index 05ff5f04e9adc481371a587439c8a083c8c12b3f..83ea125a1f7a72fb649392f918d2bd531c6334d8 100644 (file)
@@ -120,8 +120,8 @@ struct _GStaticMutex
 /* This should be NULL followed by the bytes in PTHREAD_MUTEX_INITIALIZER */
 #define        G_STATIC_MUTEX_INIT     { NULL, { { 1, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 } } }
 #define        g_static_mutex_get_mutex(mutex) \
-  (g_thread_use_default_impl ? ((GMutex*) &(mutex).aligned_pad_u) : \
-   g_static_mutex_get_mutex_impl (&(mutex).runtime_mutex))
+  (g_thread_use_default_impl ? ((GMutex*) &((mutex)->aligned_pad_u)) : \
+   g_static_mutex_get_mutex_impl (&((mutex)->runtime_mutex)))
 
 #define G_BYTE_ORDER G_LITTLE_ENDIAN
 
index 4144266814a7d17437ecbc70a2173eb1838aadca..953ed86c5beab774a1158367719ea03e87804ffc 100644 (file)
@@ -1,3 +1,8 @@
+1999-02-15  Sebastian Wilhelmi  <wilhelmi@ira.uka.de>
+
+       * testgthread.c (test_mutexes): Use new signature of
+       g_static_mutex* functions.
+
 1999-02-08  Sebastian Wilhelmi  <wilhelmi@ira.uka.de>
 
        * gthread-posix.c (g_private_get_posix_impl): Use the
index 493a47e9be6a1438d63b38bceaf9886ca6e4a871..72874ccbcb0d6f0398c39b0c1e564abda580a750 100644 (file)
@@ -26,8 +26,8 @@ test_mutexes (void)
   g_mutex_lock (mutex);
   g_mutex_unlock (mutex);
 
-  g_static_mutex_lock (static_mutex);
-  g_static_mutex_unlock (static_mutex);
+  g_static_mutex_lock (&static_mutex);
+  g_static_mutex_unlock (&static_mutex);
 
   g_cond_signal (cond);
   g_cond_broadcast (cond);