Missing bExact -> iExact conversions
authorTom Sepez <tsepez@chromium.org>
Tue, 28 Jul 2015 23:27:37 +0000 (16:27 -0700)
committerTom Sepez <tsepez@chromium.org>
Tue, 28 Jul 2015 23:27:37 +0000 (16:27 -0700)
This is the cause of the mac embeddertest failure when fx_bool is
replaced by bool (an incorrect overload).  The types still
match on master, prior to the conversion, so it's not noticed.

Find other instances as well.

R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1262683003 .

core/src/fxge/android/fx_android_font.cpp
core/src/fxge/android/fx_android_font.h
core/src/fxge/apple/fx_mac_imp.cpp

index 3e2fc45..fd72e47 100644 (file)
@@ -23,7 +23,7 @@ FX_BOOL CFX_AndroidFontInfo::EnumFontList(CFX_FontMapper* pMapper)
 {
     return FALSE;
 }
-void* CFX_AndroidFontInfo::MapFont(int weight, FX_BOOL bItalic, int charset, int pitch_family, const FX_CHAR* face, FX_BOOL& bExact)
+void* CFX_AndroidFontInfo::MapFont(int weight, FX_BOOL bItalic, int charset, int pitch_family, const FX_CHAR* face, int& iExact)
 {
     if (!m_pFontMgr) {
         return NULL;
index 83c04a0..e0101b3 100644 (file)
@@ -20,7 +20,7 @@ public:
 
     virtual    FX_BOOL         EnumFontList(CFX_FontMapper* pMapper);
 
-    virtual void*              MapFont(int weight, FX_BOOL bItalic, int charset, int pitch_family, const FX_CHAR* face, FX_BOOL& bExact);
+    virtual void*              MapFont(int weight, FX_BOOL bItalic, int charset, int pitch_family, const FX_CHAR* face, int& bExact);
 
     virtual void*              GetFont(const FX_CHAR* face);
     virtual FX_DWORD   GetFontData(void* hFont, FX_DWORD table, uint8_t* buffer, FX_DWORD size);
index 04b04b8..1bcbc9d 100644 (file)
@@ -28,7 +28,7 @@ Base14Substs[] = {
 class CFX_MacFontInfo : public CFX_FolderFontInfo
 {
 public:
-    virtual void*              MapFont(int weight, FX_BOOL bItalic, int charset, int pitch_family, const FX_CHAR* family, FX_BOOL& bExact);
+    virtual void*              MapFont(int weight, FX_BOOL bItalic, int charset, int pitch_family, const FX_CHAR* family, int& iExact);
 };
 #define JAPAN_GOTHIC "Hiragino Kaku Gothic Pro W6"
 #define JAPAN_MINCHO "Hiragino Mincho Pro W6"
@@ -44,14 +44,14 @@ static void GetJapanesePreference(CFX_ByteString& face, int weight, int picth_fa
         face = JAPAN_MINCHO;
     }
 }
-void* CFX_MacFontInfo::MapFont(int weight, FX_BOOL bItalic, int charset, int pitch_family, const FX_CHAR* cstr_face, FX_BOOL& bExact)
+void* CFX_MacFontInfo::MapFont(int weight, FX_BOOL bItalic, int charset, int pitch_family, const FX_CHAR* cstr_face, int& iExact)
 {
     CFX_ByteString face = cstr_face;
     int iBaseFont;
     for (iBaseFont = 0; iBaseFont < 12; iBaseFont ++)
         if (face == CFX_ByteStringC(Base14Substs[iBaseFont].m_pName)) {
             face = Base14Substs[iBaseFont].m_pSubstName;
-            bExact = TRUE;
+            iExact = TRUE;
             break;
         }
     if (iBaseFont < 12) {