[gst-devel] Fwd: [Bug 63621] Changed - Add GAtomicInt for atomic integer operations

[David Schleef]

On Thu, Feb 26, 2004 at 04:07:35PM +0100, Benjamin Otte wrote:
> Can we please require glib 2.4 in gstreamer, retire our atomic stuff and use 
> this in 0.8?

We could just as easily have a test for glib-2.4, and if it exists,
wrap the g_atomic code with gst_atomic.  Otherwise, fall back to our
current implementation.  I don't see any need to rush into using
new code when the existing code works.

I'm a bit concerned with glib's lack of test code for the atomic
implementations.  I think I volunteered to help with that a long
time ago.  Also, the fallback code could easily cause a lot of
lock contention in GStreamer, since it uses a global lock, compared
to GStreamer's fine-grained locking.  The fast implementations all
require gcc (as they do with GStreamer's).

By the way, isn't this _way_ past glib's API freeze date?



dave...