fontselect: use stdbool.h wherever appropriate

diff --git a/libass/ass_coretext.c b/libass/ass_coretext.c
index 006547bff787fd2865c1f5e31e4db0f36f353d94..2eacd9244acdc60fe4ceeb6004c697a452daf239 100644 (file)
--- a/libass/ass_coretext.c
+++ b/libass/ass_coretext.c
@@ -53,15 +53,15 @@ static void destroy_font(void *priv)
     SAFE_CFRelease(set);
 }
 
-static int check_glyph(void *priv, uint32_t code)
+static bool check_glyph(void *priv, uint32_t code)
 {
     CFCharacterSetRef set = priv;
 
     if (!set)
-        return 1;
+        return true;
 
     if (code == 0)
-        return 1;
+        return true;
 
     return CFCharacterSetIsLongCharacterMember(set, code);
 }

diff --git a/libass/ass_directwrite.c b/libass/ass_directwrite.c
index 13e20a34a6a781ae494ef9efdfcb340251b2208a..bf40cfbc0569d1652e8d020cd81bd0d157a448d4 100644 (file)
--- a/libass/ass_directwrite.c
+++ b/libass/ass_directwrite.c
@@ -323,21 +323,20 @@ static size_t get_data(void *data, unsigned char *buf, size_t offset,
 }
 
 /*
- * Checks if the passed font has a specific unicode
- * character. Returns 0 for failure and 1 for success
+ * Check if the passed font has a specific unicode character.
  */
-static int check_glyph(void *data, uint32_t code)
+static bool check_glyph(void *data, uint32_t code)
 {
     HRESULT hr = S_OK;
     FontPrivate *priv = (FontPrivate *) data;
     BOOL exists = FALSE;
 
     if (code == 0)
-        return 1;
+        return true;
 
     hr = IDWriteFont_HasCharacter(priv->font, code, &exists);
     if (FAILED(hr))
-        return 0;
+        return false;
 
     return exists;
 }

diff --git a/libass/ass_fontconfig.c b/libass/ass_fontconfig.c
index 8b0824bc5748513fbdb5c52fec7369128f7acb08..b3b7100163fd11e13fbaa6348a85293def408f24 100644 (file)
--- a/libass/ass_fontconfig.c
+++ b/libass/ass_fontconfig.c
@@ -40,23 +40,23 @@ typedef struct fc_private {
     FcCharSet *fallback_chars;
 } ProviderPrivate;
 
-static int check_glyph(void *priv, uint32_t code)
+static bool check_glyph(void *priv, uint32_t code)
 {
     FcPattern *pat = (FcPattern *)priv;
     FcCharSet *charset;
 
     if (!pat)
-        return 1;
+        return true;
 
     if (code == 0)
-        return 1;
+        return true;
 
     FcResult result = FcPatternGetCharSet(pat, FC_CHARSET, 0, &charset);
     if (result != FcResultMatch)
-        return 0;
+        return false;
     if (FcCharSetHasChar(charset, code) == FcTrue)
-        return 1;
-    return 0;
+        return true;
+    return false;
 }
 
 static void destroy(void *priv)

diff --git a/libass/ass_fontselect.c b/libass/ass_fontselect.c
index 2f577f1073eeb263072a16b4728e5286a3f7db80..e179ed92f11f95d7fca0f31b5a52510c6bda48aa 100644 (file)
--- a/libass/ass_fontselect.c
+++ b/libass/ass_fontselect.c
@@ -110,12 +110,12 @@ struct font_data_ft {
     int idx;
 };
 
-static int check_glyph_ft(void *data, uint32_t codepoint)
+static bool check_glyph_ft(void *data, uint32_t codepoint)
 {
     FontDataFT *fd = (FontDataFT *)data;
 
     if (!codepoint)
-        return 1;
+        return true;
 
     return !!FT_Get_Char_Index(fd->face, codepoint);
 }
@@ -243,7 +243,7 @@ static void ass_font_provider_free_fontinfo(ASS_FontInfo *info)
  * \param data private data for the font
  * \return success
  */
-int
+bool
 ass_font_provider_add_font(ASS_FontProvider *provider,
                            ASS_FontProviderMetaData *meta, const char *path,
                            int index, void *data)
@@ -343,11 +343,11 @@ ass_font_provider_add_font(ASS_FontProvider *provider,
     info->provider = provider;
 
     selector->n_font++;
-    return 0;
+    return false;
 
 error:
     ass_font_provider_free_fontinfo(info);
-    return 1;
+    return true;
 }
 
 /**
@@ -487,7 +487,7 @@ static void font_info_dump(ASS_FontInfo *font_infos, size_t len)
 }
 #endif
 
-static int check_glyph(ASS_FontInfo *fi, uint32_t code)
+static bool check_glyph(ASS_FontInfo *fi, uint32_t code)
 {
     ASS_FontProvider *provider = fi->provider;
     assert(provider && provider->funcs.check_glyph);
@@ -727,7 +727,7 @@ char *ass_font_select(ASS_FontSelector *priv, ASS_Library *library,
  * \param info metadata, returned here
  * \return success
  */
-static int
+static bool
 get_font_info(FT_Library lib, FT_Face face, ASS_FontProviderMetaData *info)
 {
     int i;
@@ -742,7 +742,7 @@ get_font_info(FT_Library lib, FT_Face face, ASS_FontProviderMetaData *info)
 
     // we're only interested in outlines
     if (!(face->face_flags & FT_FACE_FLAG_SCALABLE))
-        return 0;
+        return false;
 
     for (i = 0; i < num_names; i++) {
         FT_SfntName name;
@@ -816,7 +816,7 @@ get_font_info(FT_Library lib, FT_Face face, ASS_FontProviderMetaData *info)
         info->n_fullname = num_fullname;
     }
 
-    return 0;
+    return false;
 
 error:
     for (i = 0; i < num_family; i++)
@@ -829,7 +829,7 @@ error:
     free(info->fullnames);
     free(postscript_name);
 
-    return 1;
+    return true;
 }
 
 /**

diff --git a/libass/ass_fontselect.h b/libass/ass_fontselect.h
index 09749ad6ffb6c14aa8dd41fab809e2edf4766ca8..eceb2f08fd4ed587462ffe69ebca742e280a5f3c 100644 (file)
--- a/libass/ass_fontselect.h
+++ b/libass/ass_fontselect.h
@@ -58,9 +58,9 @@ typedef size_t  (*GetDataFunc)(void *font_priv, unsigned char *data,
  *
  * \param font_priv font private data
  * \param codepont Unicode codepoint (UTF-32)
- * \return non-zero value if codepoint is supported by the font
+ * \return true if codepoint is supported by the font
  */
-typedef int     (*CheckGlyphFunc)(void *font_priv, uint32_t codepoint);
+typedef bool    (*CheckGlyphFunc)(void *font_priv, uint32_t codepoint);
 
 /**
  * Destroy a font's private data.
@@ -243,7 +243,7 @@ ass_create_font_provider(ASS_Renderer *priv, ASS_FontProviderFuncs *funcs,
  * \return success
  *
  */
-int
+bool
 ass_font_provider_add_font(ASS_FontProvider *provider,
                            ASS_FontProviderMetaData *meta, const char *path,
                            int index, void *data);