From: Behdad Esfahbod <behdad@behdad.org>
Date: Mon, 26 Sep 2016 10:56:40 +0000 (+0100)
Subject: [MATH] Clean up get_math_constant implementation
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=54c0cc38fba24fa4370231408ba121589b190a28;p=platform%2Fupstream%2FlibHarfBuzzSharp.git

[MATH] Clean up get_math_constant implementation
---

diff --git a/src/hb-ot-layout-math-table.hh b/src/hb-ot-layout-math-table.hh
index 7f6c6b0..cbdee48 100644
--- a/src/hb-ot-layout-math-table.hh
+++ b/src/hb-ot-layout-math-table.hh
@@ -77,7 +77,8 @@ struct MathConstants
     return_trace (c->check_struct (this) && sanitize_math_value_records(c));
   }
 
-  inline hb_position_t get_value (hb_font_t *font, hb_ot_math_constant_t constant) const
+  inline hb_position_t get_value (hb_ot_math_constant_t constant,
+				  hb_font_t *font) const
   {
     switch (constant) {
 
@@ -437,10 +438,9 @@ struct MATH
 		  mathGlyphInfo.sanitize (c, this));
   }
 
-  inline bool has_math_constants (void) const { return mathConstants != 0; }
-  inline const MathConstants &get_math_constants (void) const {
-    return this+mathConstants;
-  }
+  inline hb_position_t get_constant (hb_ot_math_constant_t  constant,
+				     hb_font_t             *font) const
+  { return (this+mathConstants).get_value (constant, font); }
 
   inline bool has_math_glyph_info (void) const { return mathGlyphInfo != 0; }
   inline const MathGlyphInfo &get_math_glyph_info (void) const {
diff --git a/src/hb-ot-layout.cc b/src/hb-ot-layout.cc
index 3dcdfb6..359d941 100644
--- a/src/hb-ot-layout.cc
+++ b/src/hb-ot-layout.cc
@@ -1259,15 +1259,14 @@ hb_ot_layout_has_math_data (hb_face_t *face)
  *
  * Return value: the requested constant or 0
  *
- * Since: ????
+ * Since: 1.4
  **/
 hb_position_t
 hb_ot_layout_get_math_constant (hb_font_t *font,
 				hb_ot_math_constant_t constant)
 {
   const OT::MATH &math = _get_math (font->face);
-  return math.has_math_constants() ?
-    math.get_math_constants().get_value(font, constant) : 0;
+  return math.get_constant(constant, font);
 }
 
 /**