glib-gen: Use g_once_init to protect enum registration
authorOlivier Crête <olivier.crete@collabora.co.uk>
Mon, 13 Jul 2009 16:13:10 +0000 (12:13 -0400)
committerOlivier Crête <olivier.crete@collabora.co.uk>
Mon, 13 Jul 2009 16:13:10 +0000 (12:13 -0400)
Make use that enums are only registered once by using g_once_init_enter/leave()

glib-gen.mak

index f9027da..e5fcde1 100644 (file)
@@ -30,9 +30,9 @@ enum_headers=$(foreach h,$(glib_enum_headers),\n\#include \"$(h)\")
        glib-mkenums \
        --fhead "#include \"$*-enumtypes.h\"\n$(enum_headers)" \
        --fprod "\n/* enumerations from \"@filename@\" */" \
-       --vhead "GType\n@enum_name@_get_type (void)\n{\n  static GType etype = 0;\n  if (etype == 0) {\n    static const G@Type@Value values[] = {"     \
+       --vhead "GType\n@enum_name@_get_type (void)\n{\n  static volatile gsize g_define_type_id__volatile = 0;\n  if (g_once_init_enter (&g_define_type_id__volatile)) {\n    static const G@Type@Value values[] = {"     \
        --vprod "      { @VALUENAME@, \"@VALUENAME@\", \"@valuenick@\" }," \
-       --vtail "      { 0, NULL, NULL }\n    };\n    etype = g_@type@_register_static (\"@EnumName@\", values);\n  }\n  return etype;\n}\n" \
+       --vtail "      { 0, NULL, NULL }\n    };\n    GType g_define_type_id = g_@type@_register_static (\"@EnumName@\", values);\n    g_once_init_leave (&g_define_type_id__volatile, g_define_type_id);\n  }\n  return g_define_type_id__volatile;\n}\n" \
        $^ > $@
 
 # a hack rule to make sure .Plo files exist because they get include'd