- [NEW] Gestió més eficient de la memòria que usen els GIFs

data/gifs.txt

@@ -3,6 +3,7 @@ caixes.gif
 doors.gif
 floor.gif
 font.gif
+font2.gif
 gat.gif
 objectes.gif
 obrer.gif

source/actor.cpp

@@ -46,7 +46,7 @@ namespace actor
         strcpy(act->bmp, bmp.c_str());
         act->pos = p;
         act->size = s;
-        act->surface = draw::loadSurface(bmp.c_str());
+        act->surface = draw::getSurface(bmp.c_str());
         act->bmp_rect = r;
         act->bmp_offset = o;
         act->anim_cycle = act->orient = 0;
@@ -82,7 +82,7 @@ namespace actor
     {
         actor_t *new_act = createEmptyActor();
         actor::templates::copy(new_act, act);
-        new_act->surface = draw::loadSurface(new_act->bmp);
+        new_act->surface = draw::getSurface(new_act->bmp);
         new_act->prev = new_act->next = new_act->above = new_act->below = nullptr;
         return new_act;
     }
@@ -109,7 +109,7 @@ namespace actor
                 } else if (util::strcomp(key, "bmp:")) {
                     const char *val = file::readString(buffer);
                     strcpy(t->bmp, val);
-                    t->surface = draw::loadSurface(t->bmp);
+                    t->surface = draw::getSurface(t->bmp);
                 } else if (util::strcomp(key, "bmp-rect:")) {
                     t->bmp_rect.x = file::readInt(buffer);
                     t->bmp_rect.y = file::readInt(buffer);
@@ -1076,7 +1076,7 @@ namespace actor
         if (act==first) first = act->next;
         if (act==dirty) dirty = act->next;
         if (act->next) act->next->prev = act->prev;
-        draw::freeSurface(act->surface);
+        //draw::freeSurface(act->surface);
         if (act==selected) selected = nullptr;
         free(act);
     }
@@ -1109,7 +1109,7 @@ namespace actor
             if (act->flags & FLAG_HERO) {
                 hero = act;
             } else {
-                draw::freeSurface(act->surface);
+                //draw::freeSurface(act->surface);
                 free(act);
             }
             act = tmp;
@@ -1122,7 +1122,7 @@ namespace actor
             if (act->flags & FLAG_HERO) {
                 hero = act;
             } else {
-                draw::freeSurface(act->surface);
+                //draw::freeSurface(act->surface);
                 free(act);
             }
             act = tmp;

source/jdraw.cpp

@@ -2,6 +2,7 @@
 #include "SDL2/SDL.h"
 #include "gif.c"
 #include "jfile.h"
+#include <vector>
 
 namespace draw
 {
@@ -31,6 +32,9 @@ namespace draw
 
     SDL_Rect viewport;
 
+    surface* managed[50];
+    int num_managed = 0;
+
     // Inicialització de tot el que fa falta per a carregar gràfics i pintar en pantalla
     void init(const std::string &titol, const uint16_t width, const uint16_t height, const int zoom)
     {
@@ -52,15 +56,27 @@ namespace draw
         sel_color = transparent = 0;
         for (int i=0;i<256;++i) color_indices[i] = i;
 
-        textsurf = loadSurface("font.gif");
+        int size;
-        textsurf2 = loadSurface("font2.gif");
+        char *buffer = file::getFileBuffer("gifs.txt", size, true);
+        char *p = buffer;
+        char *n = buffer;
+        while (*n!=0) {
+            while (*n!='\n') n++; *n=0;
+            loadSurface(p);
+            p=++n;
+        }
+        free(buffer);
+
+        textsurf = getSurface("font.gif");
+        textsurf2 = getSurface("font2.gif");
     }
 
     // Finalització del sistema
     void quit()
     {
-        if (textsurf) freeSurface(textsurf);
+        for (int i=0; i<num_managed; ++i) if (managed[i]) freeSurface(managed[i]);
-        if (textsurf2) freeSurface(textsurf2);
+        //if (textsurf) freeSurface(textsurf);
+        //if (textsurf2) freeSurface(textsurf2);
 
         // Si la superficie "screen" existia, alliberem la seua memòria
         if (screen != nullptr)
@@ -99,12 +115,15 @@ namespace draw
         surf->w = w;
         surf->h = h;
 
+        // Es una surface nova, no ve de arxiu ni tindrà varies referències
+        surf->filename = nullptr;
+
         // ...i tornem la superficie creada, clar
         return surf;
     }
 
     // Carrega un gràfic d'un arxiu (en format GIF) a una nova superficie, i torna un punter a ella
-    surface *loadSurface(const std::string &filename)
+    void loadSurface(const std::string &filename)
     {
         // Agafem un buffer de bytes de l'arxiu especificat
         // getFileBuffer() simplement ens torna el arxiu sencer dins de un array de char
@@ -114,7 +133,8 @@ namespace draw
         // Si ens ha tornat nullptr, es que no l'ha trobat, tornem nosaltres també nullptr ja que no s'ha pogut crear la superficie
         if (buffer == nullptr)
         {
-            return nullptr;
+            printf("ERROR: GIF no trobat: '%s'\n", filename.c_str());
+            exit(1);
         }
 
         // Primer reservem memòria per a la estructura "surface"
@@ -129,8 +149,23 @@ namespace draw
         // Com ja no ens fa falta, alliberem la memòria del buffer del arxiu
         free(buffer);
 
-        // I finalment tornem la superficie
+        // Especifiquem el nom de l'arxiu del que vé i el guardem a la llista
-        return surf;
+        surf->filename = (char*)malloc(filename.length()+1);
+        strcpy(surf->filename, filename.c_str());
+        managed[num_managed++] = surf;
+    }
+
+    surface *getSurface(const std::string &filename)
+    {
+        for(int i=0; i<num_managed; ++i)
+        {
+            // Si ja existia, pujem el nombre de referències i tornem eixa surface, i au
+            if (strcmp(managed[i]->filename, filename.c_str())==0)
+            {
+                return managed[i];
+            }
+        }
+        return nullptr;
     }
 
     // Allibera la memòria d'una superficie, els seus pixels inclosos
@@ -145,7 +180,12 @@ namespace draw
                 free(surf->pixels);
             }
 
-            // ... alliberem la superficie
+            // Si el nom de l'arxiu existeix, l'alliberem
+            if (surf->filename != nullptr)
+            {
+                free(surf->filename);
+            }
+
             free(surf);
         }
     }

source/jdraw.h

@@ -22,6 +22,7 @@ namespace draw
         uint16_t w;      // Ample de la superficie
         uint16_t h;      // Alt de la superficie
         uint8_t *pixels; // pixels de la superficie
+        char *filename;
     };
 
     /// @brief Inicialització de tot el que fa falta per a carregar gràfics i pintar en pantalla.
@@ -46,7 +47,10 @@ namespace draw
     /// @brief Carrega un gràfic d'un arxiu (en format GIF) a una nova superficie, i torna un punter a ella
     /// @param filename nom de l'arxiu GIF d'on carregar la superficie
     /// @return un punter a una nova superficie
-    surface *loadSurface(const std::string &filename);
+    /// ATENCIÓ: EN THEPOOL AÇÒ VAL PER A LA CÀRREGA INCIAL DE TOTS ELS GIFs
+    void loadSurface(const std::string &filename);
+
+    surface *getSurface(const std::string &filename);
 
     /// @brief Allibera la memòria d'una superficie, els seus pixels inclosos
     /// @param surf punter a la superficie a alliberar

source/main.cpp

@@ -55,7 +55,7 @@ void game::init()
     draw::init("The Pool", 520, 240, 3);
 
     //room::init();
-    surf = draw::loadSurface("test.gif");
+    surf = draw::getSurface("test.gif");
     draw::setSource(surf);
     draw::loadPalette("test.gif");
     game::setUpdateTicks(64);
@@ -384,12 +384,6 @@ bool game::loop()
     draw::print2(actor::hero::getBoostGod()/2, 2, 6, 28, col1, FONT_ZOOM_NONE);
     draw::print2(actor::hero::getBoostRun()/2, 2, 9, 28, col2, FONT_ZOOM_NONE);
 
-    //print(10, 200, actor::hero::getBoostJump());
-    //print(30, 200, actor::hero::getBoostGod()/2);
-    //print(50, 200, actor::hero::getBoostRun()/2);
-
-    //draw::draw(148+sx*2-sy*2, 67+sx+sy,24,24,24,0);
-
 /*
     print(0,0,input::mouseX());
     print(0,20,input::mouseY());
@@ -669,8 +663,8 @@ switch (section)
             ui::label("BITMAP", 2, line, 96, 11, GRAY); line+=10;
 
             if (btn_opt2("FILE", 2, line, act->bmp, gifs)) {
-                draw::freeSurface(act->surface);
+                //draw::freeSurface(act->surface);
-                act->surface = draw::loadSurface(act->bmp);
+                act->surface = draw::getSurface(act->bmp);
                 changed = true;
             }
             line+=10;

source/room.cpp

@@ -40,17 +40,10 @@ namespace room
     {
         actor::clear();
 
-        // [TODO][2024-06-05] Açò es una xorrada. Si al final sempre son els mateixos arxius,
+        floor_surf = draw::getSurface("floor.gif");
-        // carregar açò al principi del joc i au
+        walls_surf = draw::getSurface("walls.gif");
-        if (floor_surf) draw::freeSurface(floor_surf);
+        doors_surf = draw::getSurface("doors.gif");
-        if (walls_surf) draw::freeSurface(walls_surf);
+        aux_surf = draw::getSurface("roomaux.gif");
-        if (doors_surf) draw::freeSurface(doors_surf);
-        if (aux_surf) draw::freeSurface(aux_surf);
-
-        floor_surf = draw::loadSurface("floor.gif");
-        walls_surf = draw::loadSurface("walls.gif");
-        doors_surf = draw::loadSurface("doors.gif");
-        aux_surf = draw::loadSurface("roomaux.gif");
 
         editor::updateRoomList();
     }