test: Added further testing and fixed functions for edge cases
This commit is contained in:
36
src/arena.c
36
src/arena.c
@@ -38,12 +38,20 @@ ArenaPointer arena_alloc(Arena *arena, size_t size, size_t alignment) {
|
||||
return (ArenaPointer) {.is_valid = false, .err = ARENA_NULL_ARG};
|
||||
}
|
||||
|
||||
SizeResult padding = get_arena_align_padding(arena, alignment);
|
||||
if (size < 1) {
|
||||
return (ArenaPointer) {.is_valid = false, .err = ARENA_INVALID_SIZE};
|
||||
}
|
||||
|
||||
if (alignment < 1) {
|
||||
return (ArenaPointer) {.is_valid = false, .err = ARENA_INVALID_ALIGN};
|
||||
}
|
||||
|
||||
SizeResult padding = arena_get_align_padding(arena, alignment);
|
||||
if (!padding.is_valid) {
|
||||
return (ArenaPointer) {.is_valid = false, .err = padding.err};
|
||||
}
|
||||
|
||||
if (arena->offset + padding.val >= arena->capacity) {
|
||||
if (arena->offset + padding.val + size > arena->capacity) {
|
||||
return (ArenaPointer) {.is_valid = false, .err = ARENA_OUT_OF_SPACE};
|
||||
}
|
||||
|
||||
@@ -62,7 +70,15 @@ ArenaErr arena_ensure_capacity(Arena *arena, size_t size, size_t alignment) {
|
||||
return ARENA_NULL_ARG;
|
||||
}
|
||||
|
||||
SizeResult padding = get_arena_align_padding(arena, alignment);
|
||||
if (size < 1) {
|
||||
return ARENA_INVALID_SIZE;
|
||||
}
|
||||
|
||||
if (alignment < 1) {
|
||||
return ARENA_INVALID_ALIGN;
|
||||
}
|
||||
|
||||
SizeResult padding = arena_get_align_padding(arena, alignment);
|
||||
if (!padding.is_valid) {
|
||||
return padding.err;
|
||||
}
|
||||
@@ -94,6 +110,14 @@ ArenaPointer arena_push(Arena *arena, void *data, size_t size, size_t alignment)
|
||||
return (ArenaPointer) {.is_valid = false, .err = ARENA_NULL_ARG};
|
||||
}
|
||||
|
||||
if (size < 1) {
|
||||
return (ArenaPointer) {.is_valid = false, .err = ARENA_INVALID_SIZE};
|
||||
}
|
||||
|
||||
if (size < 1) {
|
||||
return (ArenaPointer) {.is_valid = false, .err = ARENA_INVALID_ALIGN};
|
||||
}
|
||||
|
||||
ArenaErr err = arena_ensure_capacity(arena, size, alignment);
|
||||
if (err != ARENA_OK) {
|
||||
return (ArenaPointer) {.is_valid = false, .err = err};
|
||||
@@ -117,6 +141,10 @@ ArenaErr arena_realloc(Arena *arena, size_t new_capacity) {
|
||||
return ARENA_NULL_ARG;
|
||||
}
|
||||
|
||||
if (new_capacity < 1) {
|
||||
return ARENA_INVALID_SIZE;
|
||||
}
|
||||
|
||||
uint8_t *tmp = realloc(arena->buffer, new_capacity);
|
||||
if (tmp == NULL) {
|
||||
return ARENA_BAD_ALLOC;
|
||||
@@ -129,7 +157,7 @@ ArenaErr arena_realloc(Arena *arena, size_t new_capacity) {
|
||||
|
||||
|
||||
|
||||
SizeResult get_arena_align_padding(Arena *arena, size_t alignment) {
|
||||
SizeResult arena_get_align_padding(Arena *arena, size_t alignment) {
|
||||
if (arena == NULL) {
|
||||
return (SizeResult) {.is_valid = false, .err = ARENA_NULL_ARG};
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user