Merge branch 'dev' into npc_upgrade

.clang-format

@@ -10,3 +10,4 @@ IndentCaseBlocks: true
 IncludeBlocks: Regroup
 AllowShortBlocksOnASingleLine: Empty
 ColumnLimit: 80
+AllowShortEnumsOnASingleLine: false

CMakeLists.txt

@@ -34,6 +34,7 @@ set(SOURCES
   src/npc.c
   src/events.c
   src/animation.c
+  src/inventory.c
   # ...
 )
 # Shared assets, fx-9860G-only assets and fx-CG-50-only assets
@@ -56,6 +57,8 @@ set(ASSETS
 set(ASSETS_cg
   assets-cg/player_male.png
   assets-cg/player_female.png
+  assets-cg/player_male_inv.png
+  assets-cg/player_female_inv.png
   assets-cg/npc/char/npc_male.png
   assets-cg/npc/char/npc_female.png
   assets-cg/npc/char/npc_milkman.png
@@ -69,6 +72,10 @@ set(ASSETS_cg
   assets-cg/INFO_Icon.png
   assets-cg/player_face.png
   assets-cg/font.png
+  assets-cg/inventory.png
+  assets-cg/items.png
+  assets-cg/selection.png
+  assets-cg/selected.png
 )
 
 set(ASSETS_cg_EGA64
@@ -83,6 +90,8 @@ set(ASSETS_fx
   assets-fx/SGN_Icon.png
   assets-fx/player_face.png
   assets-fx/font.png
+  assets-fx/selection.png
+  assets-fx/selected.png
   # ...
 )
 
@@ -98,6 +107,8 @@ set(ASSETS_fx_1b
   assets-fx/1b/npc/face/npc_milkman.png
   assets-fx/1b/npc/face/npc_police.png
   assets-fx/1b/INFO_Icon.png
+  assets-fx/1b/inventory.png
+  assets-fx/1b/items.png
   # ...
 )
 
@@ -112,7 +123,9 @@ set(ASSETS_fx_2b
   assets-fx/2b/npc/face/npc_female.png
   assets-fx/2b/npc/face/npc_milkman.png
   assets-fx/2b/npc/face/npc_police.png
-  assets-fx/1b/INFO_Icon.png
+  assets-fx/2b/INFO_Icon.png
+  assets-fx/2b/inventory.png
+  assets-fx/2b/items.png
   # ...
 )
 

assets-fx/1b/fxconv-metadata.txt

@@ -1,3 +1,11 @@
 INFO_Icon.png:
   type: bopti-image
   name: INFO_Icon_img
+
+inventory.png:
+  type: bopti-image
+  name: inventory_img
+
+items.png:
+  type: bopti-image
+  name: items_img

assets-fx/2b/fxconv-metadata.txt

@@ -1,3 +1,11 @@
 INFO_Icon.png:
   type: bopti-image
   name: INFO_Icon_img
+
+inventory.png:
+  type: bopti-image
+  name: inventory_img
+
+items.png:
+  type: bopti-image
+  name: items_img

assets-fx/fxconv-metadata.txt

@@ -17,6 +17,14 @@ SGN_Icon.png:
   type: bopti-image
   name: SGN_Icon_img
 
+selection.png:
+  type: bopti-image
+  name: selection_img
+
+selected.png:
+  type: bopti-image
+  name: selected_img
+
 font.png:
   name: fontRPG
   type: font

assets/converters.py

@@ -142,7 +142,7 @@ def convert_map(input: str, output: str, params: dict, target):
         indoor = int(input_map.get_property("indoor"))
     except Exception as e:
         # Show a warning
-        print(f"WARNING: Indoor property not found.\n")
+        print(f"WARNING: Indoor property not found.")
 
     if indoor:
         # Get the indoor tileset

src/config.h

@@ -12,6 +12,9 @@
 #define GRAYMODEOK 1
 #endif
 
+#define SLOT_NUM 9
+#define SLOT_COLUMNS 3
+
 #if GINT_RENDER_RGB
 /* The tile size */
 #define T_HEIGHT 16
@@ -22,9 +25,15 @@
 /* The size of the player */
 #define P_WIDTH 16
 #define P_HEIGHT 16
-/*Max number of dynamic NPCs.*/
+/* Max number of dynamic NPCs. */
 #define NPC_STACK_SIZE 256
-
+/* The position of the slots in the inventory */
+#define SLOT_Y 87
+#define SLOT_X_EQUIPPED 222
+#define SLOT_X 272
+#define SLOT_W 28
+#define SLOT_H 27
+#define SLOT_SPACING 32
 #else
 /* The tile size */
 #define T_HEIGHT 8
@@ -35,9 +44,15 @@
 /* The player size */
 #define P_WIDTH 8
 #define P_HEIGHT 8
-/*Max number of "dynamic" NPCs. We are starved for static ram on fx !*/
+/* Max number of "dynamic" NPCs. We are starved for static ram on fx ! */
 #define NPC_STACK_SIZE 32
-
+/* The position of the slots in the inventory */
+#define SLOT_Y 24
+#define SLOT_X_EQUIPPED 72
+#define SLOT_X 88
+#define SLOT_W 8
+#define SLOT_H 8
+#define SLOT_SPACING 12
 #endif
 
 /* SPEED should NOT be 8 or bigger: it may cause bugs when handling

src/events.c

@@ -48,7 +48,7 @@ int (*_operations[OP_AMOUNT])(int, int) = {_op_null, _op_set, _op_add, _op_sub,
 
 #define MIN(a, b) a < b ? a : b
 
-char _message_buffer[MESSAGE_BUFFER_SZ];
+char *_message_buffer;
 char *events_parse_string(EventHandler *handler, char *message) {
     size_t message_pos = 0;
     char in_token = 0;

src/events.h

@@ -15,7 +15,11 @@ typedef struct {
     unsigned int vars;
 } EventHandler;
 
-typedef enum { T_NULL, T_VAR_EDIT, T_AMOUNT } Token;
+typedef enum {
+    T_NULL,
+    T_VAR_EDIT,
+    T_AMOUNT
+} Token;
 
 typedef enum {
     OP_NULL,

src/game.c

@@ -1,6 +1,7 @@
 #include "game.h"
 
 #include "config.h"
+#include "inventory.h"
 #include "map.h"
 #include "mapdata.h"
 #include "npc.h"
@@ -35,6 +36,11 @@ void game_init(Game *game) {
     events_init_handler(&game->handler);
     events_bind_variable(&game->handler, (int *)&game->player.life, "life");
     events_bind_variable(&game->handler, &game->mana, "mana");
+    inventory_init(&game->inventory);
+    /* For debugging */
+    game->inventory.slots[5].i = I_GLOVE;
+    game->inventory.slots[1].i = I_ARMOR;
+    game->inventory.slots[8].i = I_TALISMAN;
 
     // reload_npc(&game);
 }
@@ -121,8 +127,10 @@ void game_render_indicator(Game *game) {
 
 /* Draw everything. */
 void game_draw(Game *game) {
-    /*Only clear if we are inside, the screen is guaranteed to be filled
+    /* Only clear if we are inside, the screen is guaranteed to be filled
      * otherwise */
+    /* (Mibi88) if we do so, we should only use opaque tiles in the background
+    layer (it's currently not the case), to artefacts when rendering dialogs. */
     if(game->map_level->indoor)
         dclear(C_WHITE);
     map_render_by_layer(game, BACKGROUND);
@@ -135,6 +143,7 @@ void game_draw(Game *game) {
     dprint(8, 8, C_BLACK, "npc_count: %d", npc_count);
     dprint(8, 16, C_BLACK, "Mana: %d", game->mana);
     dprint(8, 24, C_BLACK, "X: %d Y: %d", game->player.x, game->player.y);
+    inventory_draw(&game->inventory, &game->player);
 }
 
 /* Key management */
@@ -149,7 +158,67 @@ void game_get_inputs(Game *game) {
     if(keydown(KEY_EXIT))
         game->exittoOS = true;
 
-    /* Player actions - Prototypes in player.h and implementation in player.c */
+    /* Inventory */
+    if(keydown(KEY_ALPHA)) {
+        game->inventory.open = !game->inventory.open;
+        /* TODO: Make something cleaner */
+        while(keydown(KEY_ALPHA)) {
+            clearevents();
+            sleep();
+        }
+    } else if(game->inventory.open) {
+        if(keydown(KEY_LEFT))
+            inventory_move_selection(&game->inventory, D_LEFT);
+        /* TODO: Make something cleaner */
+        while(keydown(KEY_LEFT)) {
+            clearevents();
+            sleep();
+        }
+        if(keydown(KEY_RIGHT))
+            inventory_move_selection(&game->inventory, D_RIGHT);
+        /* TODO: Make something cleaner */
+        while(keydown(KEY_RIGHT)) {
+            clearevents();
+            sleep();
+        }
+        if(keydown(KEY_UP))
+            inventory_move_selection(&game->inventory, D_UP);
+        /* TODO: Make something cleaner */
+        while(keydown(KEY_UP)) {
+            clearevents();
+            sleep();
+        }
+        if(keydown(KEY_DOWN))
+            inventory_move_selection(&game->inventory, D_DOWN);
+        /* TODO: Make something cleaner */
+        while(keydown(KEY_DOWN)) {
+            clearevents();
+            sleep();
+        }
+        if(keydown(KEY_OPTN))
+            inventory_move_from_selected(&game->inventory);
+        /* TODO: Make something cleaner */
+        while(keydown(KEY_OPTN)) {
+            clearevents();
+            sleep();
+        }
+        if(keydown(KEY_SQUARE)) {
+            inventory_use(&game->inventory, &game->player);
+        }
+        if(keydown(KEY_F1)) {
+            inventory_unequip(&game->inventory, IT_TALISMAN);
+        }
+        if(keydown(KEY_F2)) {
+            inventory_unequip(&game->inventory, IT_ARMOR);
+        }
+        if(keydown(KEY_F3)) {
+            inventory_unequip(&game->inventory, IT_WEAPON);
+        }
+        if(keydown(KEY_SHIFT))
+            game->inventory.selected = game->inventory.selection;
+    } else {
+        /* Player actions - Prototypes in player.h and implementation in
+         * player.c */
         if(keydown(KEY_LEFT))
             player_move(game, D_LEFT);
         if(keydown(KEY_RIGHT))
@@ -180,8 +249,10 @@ void game_get_inputs(Game *game) {
                     Portal *dest_portal = (Portal *)portal->portal;
                     Collider dest_collider = dest_portal->collider;
                     Map *dest_map = (Map *)portal->map;
-                player->x = (dest_collider.x1 + dest_collider.x2) / 2 * PXSIZE;
+                    player->x =
-                player->y = (dest_collider.y1 + dest_collider.y2) / 2 * PXSIZE;
+                        (dest_collider.x1 + dest_collider.x2) / 2 * PXSIZE;
+                    player->y =
+                        (dest_collider.y1 + dest_collider.y2) / 2 * PXSIZE;
                     game->map_level = dest_map;
                     /* TODO: Make something cleaner */
                     while(keydown(KEY_SHIFT)) {
@@ -241,6 +312,7 @@ void game_get_inputs(Game *game) {
             game->record = !game->record;
 
 #endif // USB_FEATURE
+    }
 }
 
 void game_update_animations(Game *game, unsigned char ms) {

src/game.h

@@ -8,14 +8,59 @@
 #include <stdint.h>
 
 /* The direction where the player is going to. */
-typedef enum { D_UP, D_DOWN, D_LEFT, D_RIGHT } Direction;
+typedef enum {
+    D_UP,
+    D_DOWN,
+    D_LEFT,
+    D_RIGHT
+} Direction;
 
-typedef enum { P_LEFTUP = -1, P_CENTER = 0, P_RIGHTDOWN = 1 } Checkpos;
+typedef enum {
+    P_LEFTUP = -1,
+    P_CENTER = 0,
+    P_RIGHTDOWN = 1
+} Checkpos;
+
+typedef enum {
+    I_NONE,
+    I_ARMOR,
+    I_GLOVE,
+    I_SWORD,
+    I_BEER,
+    I_MILK,
+    I_TALISMAN,
+    I_AMOUNT
+} Item;
+
+typedef enum {
+    IT_NONE,
+    IT_TALISMAN,
+    IT_ARMOR,
+    IT_WEAPON,
+    IT_FOOD,
+    IT_AMOUNT
+} ItemType;
+
+typedef struct {
+    Item i : 4;
+    unsigned char durability;
+} Slot;
+
+typedef struct {
+    /* Backpack slots. */
+    Slot slots[SLOT_NUM];
+    /* Equipped items: first slot: talisman, second slot: armor, last slot:
+    weapon. */
+    Slot equipped[3];
+    /* 1 if the inventory is open. */
+    char open : 1;
+    char selected;
+    char selection;
+} Inventory;
 
 typedef struct {
     uint32_t x1, y1;
     uint32_t x2, y2;
-
 } Collider;
 
 /* Struct that define player parameters */
@@ -109,6 +154,7 @@ typedef struct {
     uint8_t current_group;
     uint8_t hostile_to_group;
 
+    /* uint16_t to keep the struct aligned */
     uint16_t __padding;
 } NPC;
 
@@ -178,6 +224,7 @@ typedef struct {
     bool debug_extra;
 
     Animation npc_animation;
+    Inventory inventory;
 
     int mana; /* Only for testing events TODO: Remove this! */
 } Game;

src/inventory.c

@@ -0,0 +1,151 @@
+#include "inventory.h"
+
+#include <gint/display.h>
+#include <string.h>
+
+extern bopti_image_t inventory_img;
+extern bopti_image_t items_img;
+extern bopti_image_t selection_img;
+extern bopti_image_t selected_img;
+extern bopti_image_t player_male_inv_img;
+extern bopti_image_t player_female_inv_img;
+
+char item_types[I_AMOUNT] = {IT_NONE, IT_ARMOR, IT_WEAPON,  IT_WEAPON,
+                             IT_FOOD, IT_FOOD,  IT_TALISMAN};
+
+void inventory_init(Inventory *inventory) {
+    inventory->open = 0;
+    memset(inventory->slots, 0, sizeof(Slot) * SLOT_NUM);
+    memset(inventory->equipped, 0, sizeof(Slot) * 3);
+    inventory->selected = -1;
+    inventory->selection = 0;
+}
+
+void inventory_draw(Inventory *inventory, Player *player) {
+    size_t i;
+    if(inventory->open) {
+        dimage(0, 0, &inventory_img);
+        for(i = 0; i < SLOT_NUM; i++) {
+            dsubimage(SLOT_X + (i % SLOT_COLUMNS) * SLOT_SPACING,
+                      SLOT_Y + (i / SLOT_COLUMNS) * SLOT_SPACING, &items_img,
+                      inventory->slots[i].i * SLOT_W, 0, SLOT_W, SLOT_H,
+                      DIMAGE_NONE);
+            if(i == inventory->selection) {
+                dimage(SLOT_X + (i % SLOT_COLUMNS) * SLOT_SPACING,
+                       SLOT_Y + (i / SLOT_COLUMNS) * SLOT_SPACING,
+                       &selection_img);
+            }
+            if(i == inventory->selected) {
+                dimage(SLOT_X + (i % SLOT_COLUMNS) * SLOT_SPACING,
+                       SLOT_Y + (i / SLOT_COLUMNS) * SLOT_SPACING,
+                       &selected_img);
+            }
+        }
+        for(i = 0; i < 3; i++) {
+            dsubimage(SLOT_X_EQUIPPED, SLOT_Y + i * SLOT_SPACING, &items_img,
+                      inventory->equipped[i].i * SLOT_W, 0, SLOT_W, SLOT_H,
+                      DIMAGE_NONE);
+        }
+#if GINT_RENDER_RGB
+        /* Render the player between the two swords if we are on cg. */
+        dimage(183, 20,
+               player->is_male ? &player_male_inv_img : &player_female_inv_img);
+#endif
+    }
+    /* TODO: Stats. */
+}
+
+char inventory_add(Inventory *inventory, Item item) {
+    size_t i;
+    for(i = 0; i < SLOT_NUM; i++) {
+        if(!inventory->slots[i].i) {
+            inventory->slots[i].i = item;
+            inventory->slots[i].durability = 255;
+            return 0;
+        }
+    }
+    return 1;
+}
+
+void inventory_move_from_selected(Inventory *inventory) {
+    if(inventory->selected < 0)
+        return;
+    Slot current = inventory->slots[inventory->selection];
+    Slot new = inventory->slots[inventory->selected];
+    inventory->slots[inventory->selection] = new;
+    inventory->slots[inventory->selected] = current;
+}
+
+void inventory_use(Inventory *inventory, Player *player) {
+    Item item = inventory->slots[inventory->selection].i;
+    switch(item_types[item]) {
+    case IT_TALISMAN:
+        inventory->equipped[0] = inventory->slots[inventory->selection];
+        break;
+    case IT_ARMOR:
+        inventory->equipped[1] = inventory->slots[inventory->selection];
+        break;
+    case IT_WEAPON:
+        inventory->equipped[2] = inventory->slots[inventory->selection];
+        break;
+    case IT_FOOD:
+        /* TODO */
+        break;
+    default:
+        break;
+    }
+    inventory->slots[inventory->selection].i = I_NONE;
+    inventory->slots[inventory->selection].durability = 255;
+}
+
+void inventory_unequip(Inventory *inventory, ItemType type) {
+    if(inventory->slots[inventory->selection].i)
+        return;
+    switch(type) {
+    case IT_TALISMAN:
+        inventory->slots[inventory->selection] = inventory->equipped[0];
+        inventory->equipped[0].i = I_NONE;
+        inventory->equipped[0].durability = 255;
+        break;
+    case IT_ARMOR:
+        inventory->slots[inventory->selection] = inventory->equipped[1];
+        inventory->equipped[1].i = I_NONE;
+        inventory->equipped[1].durability = 255;
+        break;
+    case IT_WEAPON:
+        inventory->slots[inventory->selection] = inventory->equipped[2];
+        inventory->equipped[2].i = I_NONE;
+        inventory->equipped[2].durability = 255;
+        break;
+    default:
+        break;
+    }
+}
+
+void inventory_move_selection(Inventory *inventory, Direction direction) {
+    switch(direction) {
+    case D_UP:
+        inventory->selection -= SLOT_COLUMNS;
+        if(inventory->selection < 0)
+            inventory->selection = 0;
+        break;
+    case D_DOWN:
+        inventory->selection += SLOT_COLUMNS;
+        if(inventory->selection >= SLOT_NUM) {
+            inventory->selection = SLOT_NUM - 1;
+        }
+        break;
+    case D_LEFT:
+        if(inventory->selection > 0) {
+            inventory->selection--;
+        }
+        break;
+    case D_RIGHT:
+        if(inventory->selection < SLOT_NUM - 1) {
+            inventory->selection++;
+        }
+        break;
+    default:
+        break;
+    }
+}

src/inventory.h

@@ -0,0 +1,15 @@
+#ifndef INVENTORY_H
+#define INVENTORY_H
+
+/* The structs related to the inventory are defined in game.h */
+#include "game.h"
+
+void inventory_init(Inventory *inventory);
+void inventory_draw(Inventory *inventory, Player *player);
+char inventory_add(Inventory *inventory, Item item);
+void inventory_move_from_selected(Inventory *inventory);
+void inventory_use(Inventory *inventory, Player *player);
+void inventory_unequip(Inventory *inventory, ItemType type);
+void inventory_move_selection(Inventory *inventory, Direction direction);
+
+#endif

src/main.c

@@ -32,7 +32,8 @@
 extern Map *worldRPG[];
 
 /* Game data (defined in "game.h")*/
-Game game = {NULL,
+Game game = {
+    NULL,
     {12 * PXSIZE,
      36 * PXSIZE,
      0,
@@ -56,7 +57,9 @@ Game game = {NULL,
     false,
     false,
     {},
-             100};
+    {},
+    100,
+};
 
 /* screen capture management code. TODO: Clean this up! */
 
@@ -117,6 +120,17 @@ int main(void) {
     }
     timer_start(timer);
 
+    extern char *_message_buffer;
+    _message_buffer = NULL;
+    _message_buffer = malloc(MESSAGE_BUFFER_SZ);
+    if(!_message_buffer) {
+        dtext(64, 64, C_BLACK,
+              "Failed to allocate the message buffer: not "
+              "enough RAM available. Press any key to quit.");
+        dupdate();
+        getkey();
+        return 0;
+    }
     game_init(&game);
 
 #if USB_FEATURE
@@ -213,5 +227,6 @@ int main(void) {
 #endif
 
     timer_stop(timer);
+    free(_message_buffer);
     return 1;
 }

src/npc.h

@@ -13,7 +13,6 @@ enum {
     NPC_FRIENDLY = 1, // The player's team
     NPC_HOSTILE = 2,  // to the player
     NPC_ALL = 3
-
 };
 
 /* /!\ Warning /!\