cleanup

story645 · story645 · commit 33c986b1ecbb · 2023-10-04T18:36:09.000-04:00
diff --git a/src/ft2font.cpp b/src/ft2font.cpp
@@ -506,19 +506,23 @@ void FT2Font::set_text(
     FT_UInt previous = 0;
     FT2Font *previous_ft_object = NULL;
 
+    FT_WarnCache warn_cache;
+
     for (size_t n = 0; n < N; n++) {
         FT_UInt glyph_index = 0;
         FT_BBox glyph_bbox;
         FT_Pos last_advance;
 
         FT_Error charcode_error, glyph_error;
         FT2Font *ft_object_with_glyph = this;
+
         bool was_found = load_char_with_fallback(ft_object_with_glyph, glyph_index, glyphs,
                                                  char_to_font, glyph_to_font, codepoints[n], flags,
-                                                 charcode_error, glyph_error, false);
+                                                 charcode_error, glyph_error, &warn_cache, false);
         if (!was_found) {
             // render missing glyph tofu
             // come back to top-most font
+            warn_cache.warn();
             ft_object_with_glyph = this;
             char_to_font[codepoints[n]] = ft_object_with_glyph;
             glyph_to_font[glyph_index] = ft_object_with_glyph;
@@ -579,9 +583,13 @@ void FT2Font::load_char(long charcode, FT_Int32 flags, FT2Font *&ft_object, bool
         FT_UInt final_glyph_index;
         FT_Error charcode_error, glyph_error;
         FT2Font *ft_object_with_glyph = this;
-        bool was_found = load_char_with_fallback(ft_object_with_glyph, final_glyph_index, glyphs, char_to_font,
-                                glyph_to_font, charcode, flags, charcode_error, glyph_error, true);
+        FT_WarnCache warn_cache;
+        bool was_found = load_char_with_fallback(ft_object_with_glyph,
+                                final_glyph_index, glyphs, char_to_font,
+                                glyph_to_font, charcode, flags, charcode_error,
+                                glyph_error, warn_cache, true);
         if (!was_found) {
+            warn_cache.warn();
             if (charcode_error) {
                 throw_ft_error("Could not load charcode", charcode_error);
             }
@@ -639,9 +647,12 @@ bool FT2Font::load_char_with_fallback(FT2Font *&ft_object_with_glyph,
                                       FT_Int32 flags,
                                       FT_Error &charcode_error,
                                       FT_Error &glyph_error,
+                                      std::vector<FT_WarnCache> &warn_cache,
                                       bool override = false)
 {
-    FT_UInt glyph_index = ft_get_char_index_or_warn(face, charcode);
+    FT_UInt glyph_index = ft_get_char_index_or_warn(face, charcode, warn=false);
+    if (!glyph_index){
+        warn_cache.push_back(charcode, face->family_name)};
     if (glyph_index || override) {
         if (charcode_error=FT_Load_Glyph(face, glyph_index, flags)) {
             return false;
@@ -834,3 +845,9 @@ long FT2Font::get_name_index(char *name)
 {
     return FT_Get_Name_Index(face, (FT_String *)name);
 }
+
+void FT_WarnCache::warn(){
+    for (int i=0; i<cache.size(); i++){
+        ft_glyph_warn(cache[i].charcode, cache[i].family_name);
+    }
+}
diff --git a/src/ft2font.h b/src/ft2font.h
@@ -66,6 +66,19 @@ class FT2Image
 
 extern FT_Library _ft2Library;
 
+class FT_WarnCache{
+    private:
+        struct entry{
+            FT_ULong charcode;
+            FT_String family_name;
+        };
+        std::vector<entry> cache;
+    public:
+        FT_WarnCache();
+        void warn();
+};
+
+#endif
 class FT2Font
 {
 
@@ -91,8 +104,8 @@ class FT2Font
                                  FT_Int32 flags,
                                  FT_Error &charcode_error,
                                  FT_Error &glyph_error,
-                                 bool override,
-                                 FT_WarnCache &warn_cache);
+                                 FT_WarnCache &warn_cache,
+                                 bool override);
     void load_glyph(FT_UInt glyph_index, FT_Int32 flags, FT2Font *&ft_object, bool fallback);
     void load_glyph(FT_UInt glyph_index, FT_Int32 flags);
     void get_width_height(long *width, long *height);
@@ -156,17 +169,3 @@ class FT2Font
     FT2Font(const FT2Font &);
     FT2Font &operator=(const FT2Font &);
 };
-
-class FT_WarnCache{
-    private:
-        struct entry{
-            FT_ULong charcode;
-            FT_String family_name;
-        };
-        std::vector<entry> cache;
-    public:
-        FT_WarnCache();
-        void warn();
-};
-
-#endif
