clean up

story645 · story645 · commit d0908d8d27ad · 2023-10-04T18:44:18.000-04:00
diff --git a/src/ft2font.cpp b/src/ft2font.cpp
@@ -506,23 +506,19 @@ 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, &warn_cache, false);
+                                                 charcode_error, glyph_error, 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;
@@ -583,13 +579,9 @@ 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;
-        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);
+        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);
         if (!was_found) {
-            warn_cache.warn();
             if (charcode_error) {
                 throw_ft_error("Could not load charcode", charcode_error);
             }
@@ -647,12 +639,9 @@ 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, warn=false);
-    if (!glyph_index){
-        warn_cache.push_back(charcode, face->family_name)};
+    FT_UInt glyph_index = ft_get_char_index_or_warn(face, charcode);
     if (glyph_index || override) {
         if (charcode_error=FT_Load_Glyph(face, glyph_index, flags)) {
             return false;
@@ -845,9 +834,3 @@ 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,19 +66,6 @@ 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
 {
 
@@ -104,7 +91,6 @@ class FT2Font
                                  FT_Int32 flags,
                                  FT_Error &charcode_error,
                                  FT_Error &glyph_error,
-                                 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);
@@ -169,3 +155,5 @@ class FT2Font
     FT2Font(const FT2Font &);
     FT2Font &operator=(const FT2Font &);
 };
+
+#endif
