refactor: Changed Result types return sytaxis
This commit is contained in:
79
src/arena.c
79
src/arena.c
@@ -7,20 +7,12 @@
|
||||
|
||||
ArenaResult arena_init(size_t capacity) {
|
||||
if (capacity < 1) {
|
||||
ArenaResult err = {
|
||||
.is_valid = false,
|
||||
.err = ARENA_INVALID_SIZE,
|
||||
};
|
||||
return err;
|
||||
return (ArenaResult) {.is_valid = false, .err = ARENA_INVALID_SIZE};
|
||||
}
|
||||
|
||||
void *buffer = malloc(capacity);
|
||||
if (buffer == NULL) {
|
||||
ArenaResult err = {
|
||||
.is_valid = false,
|
||||
.err = ARENA_BAD_ALLOC,
|
||||
};
|
||||
return err;
|
||||
return (ArenaResult) {.is_valid = false, .err = ARENA_BAD_ALLOC};
|
||||
}
|
||||
|
||||
Arena new_arena = {
|
||||
@@ -28,12 +20,8 @@ ArenaResult arena_init(size_t capacity) {
|
||||
.capacity = capacity,
|
||||
.offset = 0,
|
||||
};
|
||||
ArenaResult val = {
|
||||
.is_valid = true,
|
||||
.arena = new_arena,
|
||||
};
|
||||
|
||||
return val;
|
||||
return (ArenaResult) {.is_valid = true, .arena = new_arena};
|
||||
}
|
||||
|
||||
void arena_destroy(Arena *arena) {
|
||||
@@ -45,47 +33,27 @@ void arena_destroy(Arena *arena) {
|
||||
|
||||
ArenaPointer arena_alloc(Arena *arena, size_t size, size_t alignment) {
|
||||
if (arena == NULL) {
|
||||
ArenaPointer err = {
|
||||
.is_valid = false,
|
||||
.err = ARENA_NULL_ARG,
|
||||
};
|
||||
return err;
|
||||
return (ArenaPointer) {.is_valid = false, .err = ARENA_NULL_ARG};
|
||||
}
|
||||
|
||||
SizeResult padding = get_arena_align_padding(arena, alignment);
|
||||
if (!padding.is_valid) {
|
||||
ArenaPointer err = {
|
||||
.is_valid = false,
|
||||
.err = padding.err,
|
||||
};
|
||||
return err;
|
||||
return (ArenaPointer) {.is_valid = false, .err = padding.err};
|
||||
}
|
||||
|
||||
|
||||
if (arena->offset + padding.val >= arena->capacity) {
|
||||
ArenaPointer err = {
|
||||
.is_valid = false,
|
||||
.err = ARENA_OUT_OF_SPACE,
|
||||
};
|
||||
return err;
|
||||
return (ArenaPointer) {.is_valid = false, .err = ARENA_OUT_OF_SPACE};
|
||||
}
|
||||
|
||||
if (arena->offset > SIZE_MAX - padding.val - size) {
|
||||
ArenaPointer err = {
|
||||
.is_valid = false,
|
||||
.err = ARENA_CAPACITY_OVERFLOW,
|
||||
};
|
||||
return err;
|
||||
return (ArenaPointer) {.is_valid = false, .err = ARENA_CAPACITY_OVERFLOW};
|
||||
}
|
||||
|
||||
size_t aligned_offset = arena->offset + padding.val;
|
||||
|
||||
arena->offset = aligned_offset + size;
|
||||
ArenaPointer val = {
|
||||
.is_valid = true,
|
||||
.address = arena->buffer + aligned_offset,
|
||||
};
|
||||
return val;
|
||||
return (ArenaPointer) {.is_valid = true, .address = arena->buffer + aligned_offset};
|
||||
}
|
||||
|
||||
ArenaErr arena_ensure_capacity(Arena *arena, size_t size, size_t alignment) {
|
||||
@@ -122,20 +90,12 @@ ArenaErr arena_ensure_capacity(Arena *arena, size_t size, size_t alignment) {
|
||||
|
||||
ArenaPointer arena_push(Arena *arena, void *data, size_t size, size_t alignment) {
|
||||
if (arena == NULL || data == NULL) {
|
||||
ArenaPointer bad_pointer = {
|
||||
.is_valid = false,
|
||||
.err = ARENA_NULL_ARG,
|
||||
};
|
||||
return bad_pointer;
|
||||
return (ArenaPointer) {.is_valid = false, .err = ARENA_NULL_ARG};
|
||||
}
|
||||
|
||||
ArenaErr err = arena_ensure_capacity(arena, size, alignment);
|
||||
if (err != ARENA_OK) {
|
||||
ArenaPointer bad_pointer = {
|
||||
.is_valid = false,
|
||||
.err = err,
|
||||
};
|
||||
return bad_pointer;
|
||||
return (ArenaPointer) {.is_valid = false, .err = err};
|
||||
}
|
||||
|
||||
ArenaPointer pointer = arena_alloc(arena, size, alignment);
|
||||
@@ -168,28 +128,17 @@ ArenaErr arena_realloc(Arena *arena, size_t new_capacity) {
|
||||
|
||||
SizeResult get_arena_align_padding(Arena *arena, size_t alignment) {
|
||||
if (arena == NULL) {
|
||||
SizeResult err = {
|
||||
.is_valid = false,
|
||||
.err = ARENA_NULL_ARG,
|
||||
};
|
||||
return err;
|
||||
return (SizeResult) {.is_valid = false, .err = ARENA_NULL_ARG};
|
||||
}
|
||||
|
||||
if (alignment < 1) {
|
||||
SizeResult err = {
|
||||
.is_valid = false,
|
||||
.err = ARENA_INVALID_ALIGN,
|
||||
};
|
||||
return err;
|
||||
return (SizeResult) {.is_valid = false, .err = ARENA_INVALID_ALIGN};
|
||||
}
|
||||
|
||||
uintptr_t current_address = (uintptr_t) (arena->buffer + arena->offset);
|
||||
uintptr_t aligned = ((current_address + (alignment - 1)) & ~(alignment - 1));
|
||||
SizeResult val = {
|
||||
.is_valid = true,
|
||||
.val = aligned - current_address,
|
||||
};
|
||||
return val;
|
||||
|
||||
return (SizeResult) {.is_valid = true, .err = aligned - current_address};
|
||||
}
|
||||
|
||||
bool mul_size_t_safe(size_t a, size_t b, size_t *out) {
|
||||
|
||||
Reference in New Issue
Block a user