Re: [HarfBuzz] harfbuzz-ng: Branch 'master' - 6 commits

[Jonathan Kew]

This commit:

New commits:
commit 22afd66a30d01b6771405e76777306f600807bea
Author: Behdad Esfahbod<beh...@behdad.org>
Date:   Thu May 17 21:23:49 2012 -0400

     Add hb_atomic_int_set() again

is slightly broken for the HB_NO_MT configuration...

@@ -73,6 +75,7 @@ typedef volatile int hb_atomic_int_t;
  #define HB_ATOMIC_INT_NIL 1
  typedef volatile int hb_atomic_int_t;
  #define hb_atomic_int_add(AI, V)      ((AI) += (V), (AI) - (V))
+#define hb_atomic_int_set(AI)          ((void) ((AI) = (V)))
  #define hb_atomic_int_get(AI)         (AI)

  #endif

The hb_atomic_int_set macro needs two args, not just one.

JK
_______________________________________________
HarfBuzz mailing list
HarfBuzz@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/harfbuzz