assets/converters.py

@@ -314,10 +314,9 @@ def convert_map(input: str, output: str, params: dict, target):
         npc_struct += fxconv.u32(i["dialogID"])
         npc_struct += fxconv.u32(i["needAction"])
         npc_struct += fxconv.string(i["name"])
-        npc_struct += fxconv.u8(len(i["path"]) > 2)
+        npc_struct += fxconv.u32(len(i["path"]) > 2)
-        npc_struct += fxconv.u8(0)
+        npc_struct += fxconv.u32(len(i["path"])//2)
-        npc_struct += fxconv.u8(len(i["path"])//2)
+        npc_struct += fxconv.u32(0)
-        npc_struct += fxconv.u8(0)
 
         xpath = bytes()
         ypath = bytes()
@@ -333,7 +332,7 @@ def convert_map(input: str, output: str, params: dict, target):
         npc_struct += fxconv.u32(0) # TODO: Type
         npc_struct += fxconv.u8(0) # TODO: Group
         npc_struct += fxconv.u8(0) # TODO: Hostile to
-        npc_struct += fxconv.u16(0) # Padding
+        npc_struct += fxconv.u16(0) # TODO: Padding (what is it ?)
     map_struct += fxconv.ptr(npc_struct)
     # Load signs
     map_struct += fxconv.u32(len(signs))

src/config.h

@@ -2,7 +2,7 @@
 #define CONFIG_H
 
 #define USB_FEATURE 0
-#define DEBUGMODE 0
+#define DEBUGMODE 1
 #define PRECISION 8
 
 #include <gint/display.h>

src/game.c

@@ -201,21 +201,12 @@ void game_get_inputs(Game *game) {
             mynpc->x = player->x;
             mynpc->y = player->x;
             mynpc->hasPath = 0;
-            mynpc->owns_path = false;
+            mynpc->face = 0;
-            mynpc->face = 1;
             mynpc->paused = 0;
             mynpc->has_dialog = 0;
             mynpc->xpath = NULL;
             mynpc->ypath = NULL;
-        }
+            npc_alloc_path(mynpc);
-        while(keydown(KEY_F1)) {
-            clearevents();
-        }
-    }
-    if(keydown(KEY_F2)) {
-        npc_remove_pos(0);
-        while(keydown(KEY_F2)) {
-            clearevents();
         }
     }
 

src/game.h

@@ -97,10 +97,9 @@ typedef struct {
     char *name;
 
     /* data for NPC's trajectories */
-    uint8_t hasPath;
+    uint32_t hasPath;
-    uint8_t owns_path;
+    uint32_t path_length;
-    uint8_t path_length;
+    uint32_t currentPoint;
-    uint8_t currentPoint;
     int16_t *xpath;
     int16_t *ypath;
 

src/npc.c

@@ -30,53 +30,58 @@ NPC *npc_create() {
 void npc_remove(NPC *npc) {
     uint32_t pos = (uint32_t)npc - (uint32_t)npc_stack;
 
-    if(pos >= NPC_STACK_SIZE)
+    /*if(npc->xpath)
-        return;
-
-    if(npc->owns_path) {
         free(npc->xpath);
-        free(npc->ypath);
+    if(npc->ypath)
-    }
+        free(npc->ypath);*/
 
-    if(pos == NPC_STACK_SIZE) {
+    if(pos == npc_count) {
-        if(npc_count)
         npc_count--;
         return;
     }
-    uint32_t move_size = sizeof(NPC) * (npc_count - pos);
+    memmove(npc, npc + sizeof(NPC), sizeof(NPC) * (npc_count - pos));
-    if(move_size + pos > NPC_STACK_SIZE)
-        move_size = NPC_STACK_SIZE - pos;
 
-    memmove(npc, &npc_stack[++pos], move_size);
+    return;
-    if(npc_count)
-        npc_count--;
 }
 
-void npc_remove_pos(uint32_t pos) { npc_remove(&npc_stack[pos]); }
-
 float length(float x, float y) { return sqrtf(x * x + y * y); }
 
-int npc_clear_path(NPC *npc) {
+int npc_alloc_path(NPC *npc) {
-    npc->currentPoint = 0;
+    npc_clear_path(npc);
-    npc->hasPath = 0;
-    npc->path_length = 0;
 
-    if(npc->owns_path) {
+    /*if(npc->xpath)
         free(npc->xpath);
-        free(npc->ypath);
+    if(npc->ypath)
-    }
+        free(npc->ypath);*/
 
     npc->xpath = malloc(4);
     npc->ypath = malloc(4);
     if(npc->xpath == NULL || npc->ypath == NULL)
         return 1;
-    npc->owns_path = true;
+    return 0;
+}
+
+int npc_clear_path(NPC *npc) {
+    npc->currentPoint = 0;
+    npc->hasPath = 0;
+
+    if(npc->xpath == NULL || npc->ypath == NULL) {
+        npc->path_length = 0;
+        return 1;
+    }
+    memset(npc->xpath, 0, npc->path_length * sizeof(uint16_t));
+    memset(npc->ypath, 0, npc->path_length * sizeof(uint16_t));
+
+    npc->path_length = 0;
+
     return 0;
 }
 
 int npc_append_path(uint16_t x, uint16_t y, NPC *npc) {
-    npc->xpath = realloc(npc->xpath, npc->path_length * 2 + 2);
+    npc->xpath = realloc(npc->xpath, npc->path_length * sizeof(uint16_t) +
-    npc->ypath = realloc(npc->ypath, npc->path_length * 2 + 2);
+                                         sizeof(uint16_t));
+    npc->ypath = realloc(npc->ypath, npc->path_length * sizeof(uint16_t) +
+                                         sizeof(uint16_t));
     if(npc->xpath == NULL || npc->ypath == NULL)
         return 1;
     npc->xpath[npc->path_length] = x - npc->x;
@@ -93,7 +98,7 @@ void as_clean(uint8_t *visited, uint8_t *gscore, uint8_t *fscore) {
 
 // TODO : Fix
 int as_reconstruct_path(int16_t *came_from, int w, int h, int16_t spos,
-                        int16_t dest, NPC *npc) {
+                        int16_t dest, NPC *npc, bool is_alloc) {
     if(npc_clear_path(npc))
         goto as_recons_fail;
 
@@ -128,6 +133,7 @@ int as_reconstruct_path(int16_t *came_from, int w, int h, int16_t spos,
         npc->ypath[npc->path_length - i - 1] = ty;
     }
 
+    if(is_alloc)
         free(came_from);
 
     npc->hasPath = true;
@@ -136,6 +142,7 @@ int as_reconstruct_path(int16_t *came_from, int w, int h, int16_t spos,
 
 as_recons_fail:
 
+    if(is_alloc)
         free(came_from);
 
     return 1;
@@ -165,49 +172,42 @@ int npc_pathfind(int32_t dest_x, int32_t dest_y, Map *full_map, NPC *npc) {
     if(map[dest_y * w + dest_x])
         return 2;
 
-    npc_clear_path(npc);
+    uint8_t *visited, *gscore, *fscore;
-
-    uint8_t *visited;
     int16_t *came_from;
-    uint8_t *gscore;
+    bool is_alloc;
-    uint8_t *fscore;
 
+    /*if we can, take advantage of the on-chip memory*/
     if(isSH3() || w * h * 5 > 1024 * 15) {
+        is_alloc = true;
         visited = malloc(w * h);
         came_from = malloc(w * h * 2);
         gscore = malloc(w * h * 2);
         fscore = malloc(w * h * 2);
-        if(!visited || !came_from || !gscore || !fscore) {
+        /*If the buffers won't fit on xyram and we are on fx, alloc will likely
-            as_clean(visited, gscore, fscore);
+         * fail*/
-            free(came_from);
+        if(!visited || !came_from || !gscore || !fscore)
-            return 4;
+            return 1;
-        }
-        for(i = 0; i < w * h; i++)
-            visited[i] = 1;
-        for(i = 0; i < w * h; i++)
-            came_from[i] = -1;
-        for(i = 0; i < w * h; i++)
-            gscore[i] = 255;
-        for(i = 0; i < w * h; i++)
-            fscore[i] = 255;
     } else {
+        is_alloc = false;
         visited = (void *)xyram;
-        gscore = (void *)(xyram + w * h);
+        came_from = (void *)(xyram + w * h * sizeof(uint16_t));
-        fscore = (void *)(xyram + w * h * 2);
+        gscore = (void *)(xyram + 3 * (w * h));
-        came_from = (void *)(xyram + w * h * 3);
+        fscore = (void *)(xyram + 4 * (w * h));
-        for(i = 0; i < w * h; i++)
-            visited[i] = 1;
-        for(i = 0; i < w * h; i++)
-            came_from[i] = -1;
-        for(i = 0; i < w * h; i++)
-            gscore[i] = 255;
-        for(i = 0; i < w * h; i++)
-            fscore[i] = 255;
     }
 
-    fscore[spos] = length(dest_x - x, dest_y - y);
+    npc_clear_path(npc);
+
+    for(i = 0; i < w * h; i++)
+        visited[i] = 1;
     visited[spos] = 0;
+    for(i = 0; i < w * h; i++)
+        came_from[i] = -1;
+    for(i = 0; i < w * h; i++)
+        gscore[i] = 255;
     gscore[spos] = 0;
+    for(i = 0; i < w * h; i++)
+        fscore[i] = 255;
+    fscore[spos] = length(dest_x - x, dest_y - y);
 
     uint8_t bscore;
     int32_t bx = x;
@@ -229,10 +229,10 @@ int npc_pathfind(int32_t dest_x, int32_t dest_y, Map *full_map, NPC *npc) {
             bscore = fscore[i];
         }
         if(bx == dest_x && by == dest_y) {
+            if(is_alloc)
                 as_clean(visited, gscore, fscore);
-            return 1 /*as_reconstruct_path(came_from, w, h, spos,
+            return 0; /*as_reconstruct_path(came_from, w, h, spos,
-                                        dest_y * w + dest_x, npc)*/
+                                        dest_y * w + dest_x, npc, is_alloc);*/
-                ;
         }
 
         visited[by * w + bx] = 1;
@@ -262,9 +262,11 @@ int npc_pathfind(int32_t dest_x, int32_t dest_y, Map *full_map, NPC *npc) {
         }
     }
 
+    if(is_alloc){
         as_clean(visited, gscore, fscore);
-
         free(came_from);
+    }
+
     return 3;
 }
 
@@ -276,8 +278,8 @@ void update_npcs(Game *game) {
     }
     for(i = 0; i < npc_count; i++) {
         update_npc(&npc_stack[i]);
-        /*npc_pathfind(game->player.x, game->player.y, game->map_level,
+        npc_pathfind(game->player.x, game->player.y, game->map_level,
-                     &npc_stack[i]);*/
+                     &npc_stack[i]);
     }
 }
 

src/npc.h

@@ -28,15 +28,13 @@ NPC *npc_create();
  * Consider this as a free */
 void npc_remove(NPC *npc);
 
-/* Pops the NPC at pos from the NPC stack. You would most likely want to use
+/*Sets the adequate variables of npc so that it's path is cleared*/
- * npc_remove instead
- * Consider this as a free */
-void npc_remove_pos(uint32_t pos);
-
-/* Frees then malloc()s a new path to npc
- * Useful if you want to safely edit a path */
 int npc_clear_path(NPC *npc);
 
+/* Tries to free then malloc()s a new path to npc
+ * Useful if you want to safely edit a path */
+int npc_alloc_path(NPC *npc);
+
 /* Adds point x,y to the path of npc
  * Won't work on static NPCs, use npc_clear_path before or make them on the
  * heap */
@@ -47,8 +45,8 @@ int npc_append_path(uint16_t x, uint16_t y, NPC *npc);
  * Returns non-zero on failure */
 int npc_pathfind(int32_t dest_x, int32_t dest_y, Map *full_map, NPC *npc);
 
-/* Draws every NPC. This function should be called after drawing the
+/* Draws the player player. This function should be called after drawing the
- * background */
+ * map! */
 void npc_draw(Game *game);
 
 /* Updates the static NPCs and the NPC stack */