[ot-color] Cosmetic changes (#962)

src/hb-open-type-private.hh

@@ -909,7 +909,6 @@ struct UnsizedArrayOf
     return_trace (true);
   }
 
-  private:
   inline bool sanitize_shallow (hb_sanitize_context_t *c, unsigned int count) const
   {
     TRACE_SANITIZE (this);

src/hb-ot-color-cbdt-table.hh

@@ -302,7 +302,7 @@ struct BitmapSizeTable
 struct GlyphBitmapDataFormat17
 {
   SmallGlyphMetrics	glyphMetrics;
-  ArrayOf<HBUINT8, HBUINT32>	data;
+  LArrayOf<HBUINT8>	data;
   public:
   DEFINE_SIZE_ARRAY(9, data);
 };
@@ -310,14 +310,14 @@ struct GlyphBitmapDataFormat17
 struct GlyphBitmapDataFormat18
 {
   BigGlyphMetrics	glyphMetrics;
-  ArrayOf<HBUINT8, HBUINT32>	data;
+  LArrayOf<HBUINT8>	data;
   public:
   DEFINE_SIZE_ARRAY(12, data);
 };
 
 struct GlyphBitmapDataFormat19
 {
-  ArrayOf<HBUINT8, HBUINT32>	data;
+  LArrayOf<HBUINT8>	data;
   public:
   DEFINE_SIZE_ARRAY(4, data);
 };

src/hb-ot-color-sbix-table.hh

@@ -45,7 +45,8 @@ struct SBIXGlyph
   Tag		graphicType;	/* Indicates the format of the embedded graphic
 				 * data: one of 'jpg ', 'png ' or 'tiff', or the
 				 * special format 'dupe'. */
-  HBUINT8	data[VAR];	/* The actual embedded graphic data. The total
+  UnsizedArrayOf<HBUINT8>
+		data;		/* The actual embedded graphic data. The total
 				 * length is inferred from sequential entries in
 				 * the glyphDataOffsets array and the fixed size
 				 * (8 bytes) of the preceding fields. */
@@ -61,17 +62,15 @@ struct SBIXStrike
   {
     TRACE_SANITIZE (this);
     return_trace (c->check_struct (this) &&
-		  c->check_array (imageOffsetsZ,
+		  imageOffsetsZ.sanitize_shallow (c, c->num_glyphs + 1));
-				  sizeof (HBUINT32),
-				  1 + c->num_glyphs));
   }
 
+  protected:
   HBUINT16	ppem;		/* The PPEM size for which this strike was designed. */
   HBUINT16	resolution;	/* The device pixel density (in PPI) for which this
 				 * strike was designed. (E.g., 96 PPI, 192 PPI.) */
-  protected:
+  UnsizedArrayOf<LOffsetTo<SBIXGlyph> >
-  LOffsetTo<SBIXGlyph>	imageOffsetsZ[VAR]; // VAR=maxp.numGlyphs + 1
+		imageOffsetsZ;	/* Offset from the beginning of the strike data header
-					/* Offset from the beginning of the strike data header
 				 * to bitmap data for an individual glyph ID. */
   public:
   DEFINE_SIZE_STATIC (8);
@@ -121,7 +120,7 @@ struct sbix
 	  if (strike.imageOffsetsZ[glyph + 1] - strike.imageOffsetsZ[glyph] > 0)
 	  {
 	    const SBIXGlyph &sbixGlyph = strike.imageOffsetsZ[glyph]((const void *) &strike);
-            callback ((const uint8_t*) sbixGlyph.data,
+	    callback ((const uint8_t*) &sbixGlyph.data,
 		      strike.imageOffsetsZ[glyph + 1] - strike.imageOffsetsZ[glyph] - 8,
 		      group, glyph);
 	  }
@@ -141,7 +140,7 @@ struct sbix
   HBUINT16	version;	/* Table version number — set to 1 */
   HBUINT16	flags;		/* Bit 0: Set to 1. Bit 1: Draw outlines.
 				 * Bits 2 to 15: reserved (set to 0). */
-  ArrayOf<LOffsetTo<SBIXStrike>, HBUINT32>
+  LArrayOf<LOffsetTo<SBIXStrike> >
 		strikes;	/* Offsets from the beginning of the 'sbix'
 				 * table to data for each individual bitmap strike. */
   public:

src/hb-ot-color-svg-table.hh

@@ -45,7 +45,7 @@ struct SVGDocumentIndexEntry
   {
     TRACE_SANITIZE (this);
     return_trace (c->check_struct (this) &&
-      c->check_range (&svgDoc (base), svgDocLength));
+		  (base+svgDoc).sanitize (c, svgDocLength));
   }
 
   protected:
@@ -53,7 +53,7 @@ struct SVGDocumentIndexEntry
 				 * this index entry. */
   HBUINT16	endGlyphID;	/* The last glyph ID in the range described by
 				 * this index entry. Must be >= startGlyphID. */
-  LOffsetTo<const uint8_t *>
+  LOffsetTo<UnsizedArrayOf<HBUINT8> >
 		svgDoc;		/* Offset from the beginning of the SVG Document Index
 				 * to an SVG document. Must be non-zero. */
   HBUINT32 svgDocLength;	/* Length of the SVG document.
@@ -88,7 +88,7 @@ struct SVG
   {
     TRACE_SANITIZE (this);
     return_trace (c->check_struct (this) &&
-      svgDocIndex (this).sanitize (c));
+		  (this+svgDocIndex).sanitize (c));
   }
 
   struct accelerator_t
@@ -107,10 +107,11 @@ struct SVG
       hb_blob_destroy (svg_blob);
     }
 
-    inline void dump (void (*callback) (const uint8_t* data, unsigned int length,
+    inline void
+    dump (void (*callback) (const uint8_t* data, unsigned int length,
 			    unsigned int start_glyph, unsigned int end_glyph)) const
     {
-      const SVGDocumentIndex &index = svg->svgDocIndex (svg);
+      const SVGDocumentIndex &index = svg+svg->svgDocIndex;
       const ArrayOf<SVGDocumentIndexEntry> &entries = index.entries;
       for (unsigned int i = 0; i < entries.len; ++i)
       {