From def75761019620d78d6b206f8b8d56b38cb040f8 Mon Sep 17 00:00:00 2001 From: Abdelrahman Date: Mon, 5 May 2025 18:57:11 +0100 Subject: [PATCH] Use asserts in arena --- src/core/mem/arena/mem_arena.c | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/src/core/mem/arena/mem_arena.c b/src/core/mem/arena/mem_arena.c index ddac6a8..6313432 100644 --- a/src/core/mem/arena/mem_arena.c +++ b/src/core/mem/arena/mem_arena.c @@ -1,10 +1,12 @@ #include "mem_arena.h" +#include "../utils/mem_utils.h" #include "../../../common/aliases/aliases.h" #include "../../../common/misc/misc_utils.h" #include "../../os/mem/mem_os.h" #include #include #include +#include #ifndef DEFAULT_ALIGNMENT // Why 2 * sizeof(void *) instead of sizeof(void *) @@ -62,9 +64,7 @@ void *wapp_mem_arena_alloc(Arena *arena, u64 size) { } void *wapp_mem_arena_alloc_aligned(Arena *arena, u64 size, u64 alignment) { - if (!arena) { - return NULL; - } + assert(arena != NULL && "arena argument shouldn't be NULL"); u8 *alloc_start = arena->offset; @@ -123,18 +123,14 @@ void *wapp_mem_arena_realloc_aligned(Arena *arena, void *ptr, u64 old_size, u64 } void wapp_mem_arena_clear(Arena *arena) { - if (!arena) { - return; - } + assert(arena != NULL && "arena argument shouldn't be NULL"); memset(arena->buf, 0, arena->offset - arena->buf); arena->offset = arena->buf; } void wapp_mem_arena_destroy(Arena **arena) { - if (!arena) { - return; - } + assert(arena != NULL && (*arena) != NULL && "arena argument shouldn't be NULL"); Arena *arena_ptr = *arena; if (arena_ptr->buf) {