[color] More CPAL rename

This commit is contained in:
Behdad Esfahbod 2018-10-22 14:46:21 -07:00
parent 3600d20603
commit 0befb06c46

View File

@ -49,13 +49,13 @@ struct CPALV1Tail
inline bool inline bool
sanitize (hb_sanitize_context_t *c, const void *base, sanitize (hb_sanitize_context_t *c, const void *base,
unsigned int palette_count, unsigned int paletteEntries) const unsigned int palette_count, unsigned int color_count) const
{ {
TRACE_SANITIZE (this); TRACE_SANITIZE (this);
return_trace (c->check_struct (this) && return_trace (c->check_struct (this) &&
(base+paletteFlagsZ).sanitize (c, palette_count) && (base+paletteFlagsZ).sanitize (c, palette_count) &&
(base+paletteLabelZ).sanitize (c, palette_count) && (base+paletteLabelZ).sanitize (c, palette_count) &&
(base+paletteEntryLabelZ).sanitize (c, paletteEntries)); (base+paletteEntryLabelZ).sanitize (c, color_count));
} }
private: private:
@ -124,7 +124,7 @@ struct CPAL
/* Check for indices sanity so no need for doing it runtime */ /* Check for indices sanity so no need for doing it runtime */
for (unsigned int i = 0; i < numPalettes; ++i) for (unsigned int i = 0; i < numPalettes; ++i)
if (unlikely (colorRecordIndicesZ[i] + numPaletteEntries > numColorRecords)) if (unlikely (colorRecordIndicesZ[i] + numColors > numColorRecords))
return_trace (false); return_trace (false);
/* If version is zero, we are done here; otherwise we need to check tail also */ /* If version is zero, we are done here; otherwise we need to check tail also */
@ -132,7 +132,7 @@ struct CPAL
return_trace (true); return_trace (true);
const CPALV1Tail &v1 = StructAfter<CPALV1Tail> (*this); const CPALV1Tail &v1 = StructAfter<CPALV1Tail> (*this);
return_trace (likely (v1.sanitize (c, this, numPalettes, numPaletteEntries))); return_trace (likely (v1.sanitize (c, this, numPalettes, numColors)));
} }
inline unsigned int get_size (void) const inline unsigned int get_size (void) const
@ -164,19 +164,19 @@ struct CPAL
return HB_NAME_ID_INVALID; return HB_NAME_ID_INVALID;
const CPALV1Tail& cpal1 = StructAfter<CPALV1Tail> (*this); const CPALV1Tail& cpal1 = StructAfter<CPALV1Tail> (*this);
return cpal1.get_palette_entry_name_id (this, palette_entry, numPaletteEntries); return cpal1.get_palette_entry_name_id (this, palette_entry, numColors);
} }
inline unsigned int get_palette_count () const inline unsigned int get_palette_count () const
{ return numPalettes; } { return numPalettes; }
inline unsigned int get_palette_entries_count () const inline unsigned int get_palette_entries_count () const
{ return numPaletteEntries; } { return numColors; }
bool bool
get_color_record_argb (unsigned int color_index, unsigned int palette_index, hb_color_t* color) const get_color_record_argb (unsigned int color_index, unsigned int palette_index, hb_color_t* color) const
{ {
if (unlikely (color_index >= numPaletteEntries || palette_index >= numPalettes)) if (unlikely (color_index >= numColors || palette_index >= numPalettes))
return false; return false;
/* No need for more range check as it is already done on #sanitize */ /* No need for more range check as it is already done on #sanitize */
@ -189,7 +189,7 @@ struct CPAL
protected: protected:
HBUINT16 version; /* Table version number */ HBUINT16 version; /* Table version number */
/* Version 0 */ /* Version 0 */
HBUINT16 numPaletteEntries; /* Number of palette entries in each palette. */ HBUINT16 numColors; /* Number of colors in each palette. */
HBUINT16 numPalettes; /* Number of palettes in the table. */ HBUINT16 numPalettes; /* Number of palettes in the table. */
HBUINT16 numColorRecords; /* Total number of color records, combined for HBUINT16 numColorRecords; /* Total number of color records, combined for
* all palettes. */ * all palettes. */