Do not check for valid utf8 in set text, render the invalid utf8 instead. Mmap fonts instead of loading the whole font

2 files changed, 20 insertions, 26 deletions

diff --git a/src/graphics/font.c b/src/graphics/font.c
index 175c571..dfffc7e 100644
--- a/src/graphics/font.c
+++ b/src/graphics/font.c
@@ -26,16 +26,16 @@ int mgl_font_load_from_file(mgl_font *self, const char *filepath, unsigned int c
     self->packed_chars = NULL;
     self->num_packed_chars = 0;
 
-    mgl_filedata filedata;
-    if(mgl_load_file(filepath, &filedata, NULL) != 0) {
+    mgl_memory_mapped_file mapped_file;
+    if(mgl_mapped_file_load(filepath, &mapped_file, &(mgl_memory_mapped_file_load_options){ .readable = true, .writable = false }) != 0) {
         fprintf(stderr, "Error: failed to load font %s, error: mgl_load_file failed\n", filepath);
         return -1;
     }
 
     stbtt_fontinfo font;
-    if(!stbtt_InitFont(&font, filedata.data, stbtt_GetFontOffsetForIndex(filedata.data, 0))) {
+    if(!stbtt_InitFont(&font, mapped_file.data, stbtt_GetFontOffsetForIndex(mapped_file.data, 0))) {
         fprintf(stderr, "Error: failed to load font %s, error: stbtt_InitFont failed\n", filepath);
-        mgl_filedata_free(&filedata);
+        mgl_mapped_file_unload(&mapped_file);
         return -1;
     }
 
@@ -43,9 +43,7 @@ int mgl_font_load_from_file(mgl_font *self, const char *filepath, unsigned int c
     self->packed_chars = malloc(self->num_packed_chars * sizeof(stbtt_packedchar));
     if(!self->packed_chars) {
         fprintf(stderr, "Error: failed to load font %s, error: out of memory\n", filepath);
-        mgl_filedata_free(&filedata);
-        mgl_font_unload(self);
-        return -1;
+        goto error;
     }
 
     bool atlas_created = false;
@@ -62,9 +60,7 @@ int mgl_font_load_from_file(mgl_font *self, const char *filepath, unsigned int c
         unsigned char *new_atlas = realloc(self->font_atlas.atlas, self->font_atlas.width * self->font_atlas.height);
         if(!new_atlas) {
             fprintf(stderr, "Error: failed to load font %s, error: out of memory\n", filepath);
-            mgl_filedata_free(&filedata);
-            mgl_font_unload(self);
-            return -1;
+            goto error;
         }
         self->font_atlas.atlas = new_atlas;
 
@@ -72,19 +68,17 @@ int mgl_font_load_from_file(mgl_font *self, const char *filepath, unsigned int c
 
         if(!stbtt_PackBegin(&pc, self->font_atlas.atlas, self->font_atlas.width, self->font_atlas.height, self->font_atlas.width, 1, NULL)) {
             fprintf(stderr, "Error: failed to load font %s, error: stbtt_PackBegin failed\n", filepath);
-            mgl_filedata_free(&filedata);
-            mgl_font_unload(self);
-            return -1;
+            goto error;
         }
 
         stbtt_PackSetOversampling(&pc, 2, 2);
 
-        if(!stbtt_PackFontRange(&pc, filedata.data, 0, self->character_size, 0, self->num_packed_chars, self->packed_chars)) {
+        if(!stbtt_PackFontRange(&pc, mapped_file.data, 0, self->character_size, 0, self->num_packed_chars, self->packed_chars)) {
             stbtt_PackEnd(&pc);
             continue;
         }
 
-        /*if(!stbtt_PackFontRange(&pc, filedata.data, 0, self->character_size, 0x00004E00, self->num_packed_chars, self->packed_chars)) {
+        /*if(!stbtt_PackFontRange(&pc, mapped_file.data, 0, self->character_size, 0x00004E00, self->num_packed_chars, self->packed_chars)) {
             stbtt_PackEnd(&pc);
             continue;
         }*/
@@ -96,22 +90,23 @@ int mgl_font_load_from_file(mgl_font *self, const char *filepath, unsigned int c
 
     if(!atlas_created) {
         fprintf(stderr, "Error: failed to load font %s, error: failed to create atlas\n", filepath);
-        mgl_filedata_free(&filedata);
-        mgl_font_unload(self);
-        return -1;
+        goto error;
     }
 
     if(mgl_texture_load_from_memory(&self->texture, self->font_atlas.atlas, self->font_atlas.width, self->font_atlas.height, MGL_IMAGE_FORMAT_ALPHA, NULL) != 0) {
         fprintf(stderr, "Error: failed to load font %s, error: mgl_texture_load_from_memory failed\n", filepath);
-        mgl_filedata_free(&filedata);
-        mgl_font_unload(self);
-        return -1;
+        goto error;
     }
 
     /* TODO: Use stbtt_GetCodepointSDF */
     
-    mgl_filedata_free(&filedata);
+    mgl_mapped_file_unload(&mapped_file);
     return 0;
+
+    error:
+    mgl_mapped_file_unload(&mapped_file);
+    mgl_font_unload(self);
+    return -1;
 }
 
 void mgl_font_unload(mgl_font *self) {
diff --git a/src/graphics/text.c b/src/graphics/text.c
index 28840ce..d330dc0 100644
--- a/src/graphics/text.c
+++ b/src/graphics/text.c
@@ -47,13 +47,13 @@ static mgl_vec2f mgl_text_calculate_bounds(mgl_text *self) {
     return bounds;
 }
 
-int mgl_text_init(mgl_text *self, mgl_font *font, const char *str, size_t str_size) {
+void mgl_text_init(mgl_text *self, mgl_font *font, const char *str, size_t str_size) {
     self->font = font;
     self->color = (mgl_color){ 255, 255, 255, 255 };
     self->position = (mgl_vec2f){ 0.0f, 0.0f };
     self->text = NULL;
     self->text_size = 0;
-    return mgl_text_set_string(self, str, str_size);
+    mgl_text_set_string(self, str, str_size);
 }
 
 void mgl_text_deinit(mgl_text *self) {
@@ -64,14 +64,13 @@ void mgl_text_deinit(mgl_text *self) {
     self->bounds = (mgl_vec2f){ 0.0f, 0.0f };
 }
 
-int mgl_text_set_string(mgl_text *self, const char *str, size_t str_size) {
+void mgl_text_set_string(mgl_text *self, const char *str, size_t str_size) {
     self->text = str;
     self->text_size = str_size;
     if(self->text && self->text_size > 0 && self->font)
         self->bounds = mgl_text_calculate_bounds(self);
     else
         self->bounds = (mgl_vec2f){ 0.0f, 0.0f };
-    return 0;
 }
 
 void mgl_text_set_font(mgl_text *self, mgl_font *font) {