diff --git a/src/primitives/strings/str8/str8.c b/src/primitives/strings/str8/str8.c index 9b268e3..f940cd4 100644 --- a/src/primitives/strings/str8/str8.c +++ b/src/primitives/strings/str8/str8.c @@ -6,17 +6,14 @@ #include #include #include +#include #define STR8_BUF_ALLOC_SIZE(CAPACITY) (sizeof(Str8) + sizeof(c8) * CAPACITY) Str8 *wapp_str8_alloc_buf(const Allocator *allocator, u64 capacity) { - Str8 *str = NULL; + assert(allocator != NULL && "allocator argument shouldn't be NULL"); - if (!allocator) { - goto RETURN_STR8; - } - - str = wapp_mem_allocator_alloc(allocator, STR8_BUF_ALLOC_SIZE(capacity)); + Str8 *str = wapp_mem_allocator_alloc(allocator, STR8_BUF_ALLOC_SIZE(capacity)); if (!str) { goto RETURN_STR8; } @@ -30,14 +27,10 @@ RETURN_STR8: } Str8 *wapp_str8_alloc_cstr(const Allocator *allocator, const char *str) { - Str8 *output = NULL; - - if (!allocator || !str) { - goto RETURN_ALLOC_CSTR; - } + assert(allocator != NULL && str != NULL && "Arguments passed shouldn't be NULL"); u64 length = strlen(str); - output = wapp_str8_alloc_buf(allocator, length * 2); + Str8 *output = wapp_str8_alloc_buf(allocator, length * 2); if (!output) { goto RETURN_ALLOC_CSTR; } @@ -50,13 +43,9 @@ RETURN_ALLOC_CSTR: } Str8 *wapp_str8_alloc_str8(const Allocator *allocator, Str8RO *str) { - Str8 *output = NULL; + assert(allocator != NULL && str != NULL && "Arguments passed shouldn't be NULL"); - if (!allocator || !str) { - goto RETURN_ALLOC_STR8; - } - - output = wapp_str8_alloc_buf(allocator, str->capacity); + Str8 *output = wapp_str8_alloc_buf(allocator, str->capacity); if (!output) { goto RETURN_ALLOC_STR8; } @@ -69,10 +58,9 @@ RETURN_ALLOC_STR8: } Str8 *wapp_str8_alloc_substr(const Allocator *allocator, Str8RO *str, u64 start, u64 end) { + assert(allocator != NULL && str != NULL && "Arguments passed shouldn't be NULL"); + Str8 *output = NULL; - if (!allocator || !str) { - goto RETURN_ALLOC_SUBSTR; - } if (start >= str->size || start >= end) { goto RETURN_ALLOC_SUBSTR; @@ -95,10 +83,7 @@ RETURN_ALLOC_SUBSTR: } void wapp_str8_dealloc_buf(const Allocator *allocator, Str8 **str) { - if (!allocator || !str || !(*str)) { - return; - } - + assert(allocator != NULL && str != NULL && (*str) != NULL && "Arguments passed shouldn't be NULL"); wapp_mem_allocator_free(allocator, (void **)str, STR8_BUF_ALLOC_SIZE((*str)->capacity)); } @@ -136,11 +121,11 @@ bool wapp_str8_equal(Str8RO *s1, Str8RO *s2) { } bool wapp_str8_equal_to_count(Str8RO* s1, Str8RO* s2, u64 count) { - if (!s1 || !s2) { - return false; - } + if (!s1 || !s2) { + return false; + } - return memcmp(s1->buf, s2->buf, count) == 0; + return memcmp(s1->buf, s2->buf, count) == 0; } Str8 wapp_str8_slice(Str8RO *str, u64 start, u64 end) { @@ -161,9 +146,7 @@ Str8 wapp_str8_slice(Str8RO *str, u64 start, u64 end) { } Str8 *wapp_str8_alloc_concat(const Allocator *allocator, Str8 *dst, Str8RO *src) { - if (!allocator || !dst || !src) { - return NULL; - } + assert(allocator != NULL && dst != NULL && src != NULL && "Arguments passed shouldn't be NULL"); Str8 *output = NULL; u64 remaining = dst->capacity - dst->size; @@ -189,9 +172,7 @@ RETURN_STR8_CONCAT: } void wapp_str8_concat_capped(Str8 *dst, Str8RO *src) { - if (!dst || !src) { - return; - } + assert(dst != NULL && src != NULL && "Arguments passed shouldn't be NULL"); u64 remaining = dst->capacity - dst->size; u64 to_copy = remaining < src->size ? remaining : src->size; @@ -201,9 +182,7 @@ void wapp_str8_concat_capped(Str8 *dst, Str8RO *src) { } void wapp_str8_copy_cstr_capped(Str8 *dst, const char *src) { - if (!dst || !src) { - return; - } + assert(dst != NULL && src != NULL && "Arguments passed shouldn't be NULL"); u64 length = strlen(src); u64 to_copy = length <= dst->capacity ? length : dst->capacity; @@ -214,9 +193,7 @@ void wapp_str8_copy_cstr_capped(Str8 *dst, const char *src) { } void wapp_str8_copy_str8_capped(Str8 *dst, Str8RO *src) { - if (!dst || !src) { - return; - } + assert(dst != NULL && src != NULL && "Arguments passed shouldn't be NULL"); u64 to_copy = src->size <= dst->capacity ? src->size : dst->capacity; @@ -226,9 +203,7 @@ void wapp_str8_copy_str8_capped(Str8 *dst, Str8RO *src) { } void wapp_str8_copy_to_cstr(char *dst, Str8RO *src, u64 dst_capacity) { - if (!dst || !src) { - return; - } + assert(dst != NULL && src != NULL && "Arguments passed shouldn't be NULL"); u64 to_copy = src->size < dst_capacity ? src->size : dst_capacity - 1; @@ -237,9 +212,7 @@ void wapp_str8_copy_to_cstr(char *dst, Str8RO *src, u64 dst_capacity) { } void wapp_str8_format(Str8 *dst, const char *format, ...) { - if (!dst || !format) { - return; - } + assert(dst != NULL && format != NULL && "Arguments passed shouldn't be NULL"); va_list args1; va_list args2; @@ -301,9 +274,7 @@ i64 wapp_str8_rfind(Str8RO *str, Str8RO substr) { } Str8List *wapp_str8_split_with_max(const Allocator *allocator, Str8RO *str, Str8RO *delimiter, i64 max_splits) { - if (!allocator || !str || !delimiter) { - return NULL; - } + assert(allocator != NULL && str != NULL && delimiter != NULL && "Arguments passed shouldn't be NULL"); Str8List *output = wapp_mem_allocator_alloc(allocator, sizeof(Str8List)); @@ -356,9 +327,7 @@ RETURN_STR8_SPLIT: } Str8List *wapp_str8_rsplit_with_max(const Allocator *allocator, Str8RO *str, Str8RO *delimiter, i64 max_splits) { - if (!allocator || !str || !delimiter) { - return NULL; - } + assert(allocator != NULL && str != NULL && delimiter != NULL && "Arguments passed shouldn't be NULL"); Str8List *output = wapp_mem_allocator_alloc(allocator, sizeof(Str8List)); @@ -408,9 +377,7 @@ RETURN_STR8_SPLIT: } Str8 *wapp_str8_join(const Allocator *allocator, const Str8List *list, Str8RO *delimiter) { - if (!allocator || !list || !delimiter) { - return NULL; - } + assert(allocator != NULL && list != NULL && delimiter != NULL && "Arguments passed shouldn't be NULL"); u64 capacity = wapp_str8_list_total_size(list) + (delimiter->size * (list->node_count - 1)); Str8 *output = wapp_str8_alloc_buf(allocator, capacity * 2);