Fix more build issues

Hopefully most bots come back with this...

src/check-libstdc++.sh

@@ -21,12 +21,13 @@ else
 fi
 
 tested=false
-for soname in harfbuzz harfbuzz-icu harfbuzz-subset; do
+# harfbuzz-icu links to libstdc++ because icu does.
+for soname in harfbuzz harfbuzz-subset harfbuzz-gobject; do
 	for suffix in so dylib; do
 		so=$libs/lib$soname.$suffix
 		if ! test -f "$so"; then continue; fi
 
-		echo "Checking that we are not linking to libstdc++ or libc++"
+		echo "Checking that we are not linking to libstdc++ or libc++ in $so"
 		if $LDD $so | grep 'libstdc[+][+]\|libc[+][+]'; then
 			echo "Ouch, linked to libstdc++ or libc++"
 			stat=1

src/check-symbols.sh

@@ -17,7 +17,7 @@ fi
 
 echo "Checking that we are not exposing internal symbols"
 tested=false
-for soname in harfbuzz harfbuzz-icu harfbuzz-subset; do
+for soname in harfbuzz harfbuzz-subset harfbuzz-icu harfbuzz-gobject; do
 	for suffix in so dylib; do
 		so=$libs/lib$soname.$suffix
 		if ! test -f "$so"; then continue; fi

src/hb-buffer.h

@@ -92,7 +92,6 @@ typedef struct hb_glyph_info_t {
 typedef enum { /*< flags >*/
   HB_GLYPH_FLAG_UNSAFE_TO_BREAK		= 0x00000001,
 
-  /*< private >*/
   HB_GLYPH_FLAG_DEFINED			= 0x00000001 /* OR of all defined flags */
 } hb_glyph_flags_t;
 

src/hb-gobject-structs.h

@@ -111,6 +111,12 @@ HB_EXTERN GType hb_gobject_segment_properties_get_type (void);
 HB_EXTERN GType hb_gobject_user_data_key_get_type (void);
 #define HB_GOBJECT_TYPE_USER_DATA_KEY (hb_gobject_user_data_key_get_type ())
 
+HB_EXTERN GType hb_gobject_ot_math_glyph_variant_get_type (void);
+#define HB_GOBJECT_TYPE_USER_DATA_KEY (hb_gobject_ot_math_glyph_variant_get_type ())
+
+HB_EXTERN GType hb_gobject_ot_math_glyph_part_get_type (void);
+#define HB_GOBJECT_TYPE_USER_DATA_KEY (hb_gobject_ot_math_glyph_part_get_type ())
+
 
 HB_END_DECLS
 

src/hb-subset-plan.cc

@@ -29,9 +29,13 @@
 #include "hb-subset-plan.hh"
 #include "hb-ot-cmap-table.hh"
 
-HB_INTERNAL int
+static int
-_hb_codepoint_t_cmp (const void *l, const void *r) {
+_hb_codepoint_t_cmp (const void *pa, const void *pb)
-  return *((hb_codepoint_t *) l) - *((hb_codepoint_t *) r);
+{
+  hb_codepoint_t a = * (hb_codepoint_t *) pa;
+  hb_codepoint_t b = * (hb_codepoint_t *) pb;
+
+  return a < b ? -1 : a > b ? +1 : 0;
 }
 
 hb_bool_t

src/hb-subset.cc

@@ -251,7 +251,7 @@ hb_subset_face_add_table (hb_face_t *face, hb_tag_t tag, hb_blob_t *blob)
   return true;
 }
 
-HB_INTERNAL bool
+static bool
 _add_head_and_set_loca_version (hb_face_t *source, bool use_short_loca, hb_face_t *dest)
 {
   hb_blob_t *head_blob = OT::Sanitizer<OT::head>().sanitize (hb_face_reference_table (source, HB_OT_TAG_head));
@@ -278,7 +278,7 @@ _add_head_and_set_loca_version (hb_face_t *source, bool use_short_loca, hb_face_
   return has_head;
 }
 
-HB_INTERNAL bool
+static bool
 _subset_glyf (hb_subset_plan_t *plan, hb_face_t *source, hb_face_t *dest)
 {
   hb_blob_t *glyf_prime = nullptr;
@@ -300,7 +300,7 @@ _subset_glyf (hb_subset_plan_t *plan, hb_face_t *source, hb_face_t *dest)
   return success;
 }
 
-HB_INTERNAL bool
+static bool
 _subset_table (hb_subset_plan_t *plan,
                hb_face_t        *source,
                hb_tag_t          tag,