PROTO: Create text_objects and use them for rendering to improve text rendering performance

src/console/console.c

@@ -197,7 +197,7 @@ void console_output_render(void) {
             lines++;
         } else {
             // TODO add word wrapping?
-            text_render_char(&tconf, TEXT_DEFAULT, x, y + con->y_pos - 100, c);
+            text_render_char_uncached(&tconf, TEXT_DEFAULT, x, y + con->y_pos - 100, c, false);
             x += font_small.w;
         }
     }
@@ -268,6 +268,7 @@ void console_event(game_state *gs, SDL_Event *e) {
             if(isprint(c) && len < sizeof(con->input) - 1) {
                 con->input[len + 1] = '\0';
                 con->input[len] = tolower(c);
+                con->text_cache[0].dirty = true;
             }
         }
     } else if(e->type == SDL_KEYDOWN) {
@@ -293,11 +294,13 @@ void console_event(game_state *gs, SDL_Event *e) {
         } else if(scancode == SDL_SCANCODE_BACKSPACE || scancode == SDL_SCANCODE_DELETE) {
             if(len > 0) {
                 con->input[len - 1] = '\0';
+                con->text_cache[0].dirty = true;
             }
         } else if(scancode == SDL_SCANCODE_RETURN || scancode == SDL_SCANCODE_KP_ENTER) {
             // send the input somewhere and clear the input line
             console_handle_line(gs);
             con->input[0] = '\0';
+            con->text_cache[0].dirty = true;
         } else if(scancode == SDL_SCANCODE_PAGEUP) {
             console_output_scroll_up(1);
         } else if(scancode == SDL_SCANCODE_PAGEDOWN) {
@@ -307,6 +310,10 @@ void console_event(game_state *gs, SDL_Event *e) {
 }
 
 void console_render(void) {
+    if(con == NULL) {
+        return;
+    }
+
     if(con->y_pos > 0) {
         if(con->hist_pos != -1 && con->hist_pos_changed) {
             char *input = list_get(&con->history, con->hist_pos);
@@ -324,11 +331,12 @@ void console_render(void) {
         tconf.font = FONT_SMALL;
         // input line
         tconf.cforeground = TEXT_MEDIUM_GREEN;
-        text_render(&tconf, TEXT_DEFAULT, 0, con->y_pos - 7, 300, 6, con->input);
+        text_render(&con->text_cache[0], &tconf, TEXT_DEFAULT, 0, con->y_pos - 7, 300, 6, con->input);
 
         // cursor
         tconf.cforeground = TEXT_BLINKY_GREEN;
-        text_render(&tconf, TEXT_DEFAULT, strlen(con->input) * font_small.w, con->y_pos - 7, 6, 6, CURSOR_STR);
+        text_render(&con->text_cache[1], &tconf, TEXT_DEFAULT, strlen(con->input) * font_small.w, con->y_pos - 7, 6, 6,
+                    CURSOR_STR);
         console_output_render();
     }
 }

src/console/console_type.h

@@ -21,6 +21,7 @@ typedef struct console {
     bool is_open;
     bool owns_input;
     int y_pos;
+    text_object text_cache[2];
     hashmap cmds; // string -> command
 } console;
 

src/game/game_state_type.h

@@ -1,6 +1,8 @@
 #ifndef GAME_STATE_TYPE_H
 #define GAME_STATE_TYPE_H
 
+#include <stdbool.h>
+
 #include "engine.h"
 #include "game/protos/fight_stats.h"
 #include "utils/random.h"

src/game/gui/label.c

@@ -6,15 +6,17 @@
 typedef struct {
     char *text;
     text_settings tconf;
+    text_object text_cache[1];
 } label;
 
 static void label_render(component *c) {
     label *local = widget_get_obj(c);
-    text_render(&local->tconf, TEXT_DEFAULT, c->x, c->y, c->w, c->h, local->text);
+    text_render(&(local->text_cache[0]), &local->tconf, TEXT_DEFAULT, c->x, c->y, c->w, c->h, local->text);
 }
 
 static void label_free(component *c) {
     label *local = widget_get_obj(c);
+    text_objects_free(local->text_cache, 1);
     omf_free(local->text);
     omf_free(local);
 }
@@ -45,7 +47,7 @@ component *label_create_with_width(const text_settings *tconf, const char *text,
 
     int tsize = text_char_width(tconf);
     int longest = 0;
-    int h = text_find_line_count(tconf, max_width / tsize, 0, strlen(text), text, &longest);
+    int h = text_find_line_count(tconf, max_width / tsize, 0, text, &longest);
 
     // fonts are all 8 high?
     component_set_size_hints(c, longest * tsize, h * 8);
@@ -55,7 +57,7 @@ component *label_create_with_width(const text_settings *tconf, const char *text,
     widget_set_obj(c, local);
     widget_set_render_cb(c, label_render);
     widget_set_free_cb(c, label_free);
-
+    // local->text_cache[0].dynamic = true;
     return c;
 }
 

src/game/gui/menu.c

@@ -110,7 +110,11 @@ static void menu_render(component *c) {
             if(m->help_bg2) {
                 video_draw(m->help_bg2, m->help_x - 8, m->help_y - 8);
             }
-            text_render(&m->help_text_conf, TEXT_DEFAULT, m->help_x, m->help_y, m->help_w, m->help_h, (*tmp)->help);
+            // TODO: The text_cache should move into inner component. In this case the one of the s->objs.
+            // Right now it will get invalidated on every render. For help text it usually doesn't matter because there
+            // is usually only 1 help text on the screen per time.
+            text_render(&(m->text_cache[0]), &m->help_text_conf, TEXT_DEFAULT, m->help_x, m->help_y, m->help_w,
+                        m->help_h, (*tmp)->help);
         }
         i++;
     }
@@ -368,6 +372,7 @@ void menu_set_help_text_settings(component *c, text_settings *settings) {
 
 static void menu_free(component *c) {
     menu *m = sizer_get_obj(c);
+    text_objects_free(m->text_cache, 1);
     if(m->bg1) {
         surface_free(m->bg1);
         omf_free(m->bg1);
@@ -411,6 +416,11 @@ void menu_set_padding(component *c, int padding) {
     m->padding = padding;
 }
 
+void menu_invalidate_help_text_cache(component *c) {
+    menu *m = sizer_get_obj(c);
+    m->text_cache->dirty = true;
+}
+
 component *menu_create(int obj_h) {
     component *c = sizer_create();
 

src/game/gui/menu.h

@@ -38,6 +38,7 @@ typedef struct {
     void *userdata;
     menu_free_cb free;
     menu_tick_cb tick;
+    text_object text_cache[1];
 } menu;
 
 component *menu_create(int obj_h);
@@ -62,5 +63,6 @@ void menu_set_help_pos(component *c, int x, int y, int h, int w);
 void menu_set_help_text_settings(component *c, text_settings *settings);
 void menu_set_margin_top(component *c, int margin);
 void menu_set_padding(component *c, int padding);
+void menu_invalidate_help_text_cache(component *c);
 
 #endif // MENU_H

src/game/gui/spritebutton.c

@@ -21,6 +21,7 @@ typedef struct {
     spritebutton_focus_cb focus_cb;
     bool free_userdata;
     void *userdata;
+    text_object text_cache[1];
 } spritebutton;
 
 static void spritebutton_render(component *c) {
@@ -34,12 +35,13 @@ static void spritebutton_render(component *c) {
         mode = TEXT_SELECTED;
     }
     if(sb->text) {
-        text_render(&sb->tconf, mode, c->x, c->y, c->w, c->h, sb->text);
+        text_render(sb->text_cache, &sb->tconf, mode, c->x, c->y, c->w, c->h, sb->text);
     }
 }
 
 static void spritebutton_free(component *c) {
     spritebutton *sb = widget_get_obj(c);
+    text_objects_free(sb->text_cache, 1);
     if(sb->free_userdata) {
         omf_free(sb->userdata);
     }
@@ -107,7 +109,7 @@ component *spritebutton_create(const text_settings *tconf, const char *text, sur
     widget_set_focus_cb(c, spritebutton_focus);
     widget_set_tick_cb(c, spritebutton_tick);
     widget_set_free_cb(c, spritebutton_free);
-
+    sb->text_cache[0].dynamic = true;
     return c;
 }