diff options
| author | EuAndreh <eu@euandre.org> | 2024-05-31 15:03:56 -0300 |
|---|---|---|
| committer | EuAndreh <eu@euandre.org> | 2024-05-31 15:07:51 -0300 |
| commit | ff6a6b13b6625f195404e4a738d60cbab465c7c4 (patch) | |
| tree | 2920b7b89855689f8007acae0138483040a6d4d0 /tests | |
| parent | src/vector.c: Make the "void **values" `const` (diff) | |
| download | pindaiba-ff6a6b13b6625f195404e4a738d60cbab465c7c4.tar.gz pindaiba-ff6a6b13b6625f195404e4a738d60cbab465c7c4.tar.xz | |
src/vector.h: Revert back to doing vector_free(&v)
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/vector.c | 65 |
1 files changed, 25 insertions, 40 deletions
diff --git a/tests/vector.c b/tests/vector.c index 607a46a..72d796e 100644 --- a/tests/vector.c +++ b/tests/vector.c @@ -43,8 +43,7 @@ test_vector_new_with(void) { assert(v->multiplier == multiplier); assert(v->value_size == size); - vector_free(v); - v = NULL; + vector_free(&v); test_ok(); } @@ -69,8 +68,7 @@ test_vector_new_with(void) { rc = 0; out: if (v != NULL) { - vector_free(v); - v = NULL; + vector_free(&v); } return rc; } @@ -98,8 +96,7 @@ test_vector_new(void) { assert(v->multiplier == GROWTH_MULTIPLIER); assert(v->value_size == size); - vector_free(v); - v = NULL; + vector_free(&v); test_ok(); } @@ -124,8 +121,7 @@ test_vector_new(void) { assert(v->multiplier == GROWTH_MULTIPLIER); assert(v->value_size == size); - vector_free(v); - v = NULL; + vector_free(&v); test_ok(); } @@ -133,8 +129,7 @@ test_vector_new(void) { rc = 0; out: if (v != NULL) { - vector_free(v); - v = NULL; + vector_free(&v); } return rc; } @@ -148,6 +143,7 @@ test_vector_free(void) { test_start("vector_free()"); { testing("*v becomes NULL again after vector_free(&v)"); + assert(v == NULL); if (vector_new(sizeof(char), &v)) { logerr("vector_new()"); @@ -155,14 +151,16 @@ test_vector_free(void) { } assert(v != NULL); - vector_free(v); - v = NULL; + vector_free(&v); test_ok(); } rc = 0; out: + if (v != NULL) { + vector_free(&v); + } return rc; } @@ -190,8 +188,7 @@ test_vector_count(void) { assert(vector_count(v) == 1); assert(v->count == 1); - vector_free(v); - v = NULL; + vector_free(&v); test_ok(); } @@ -199,8 +196,7 @@ test_vector_count(void) { rc = 0; out: if (v != NULL) { - vector_free(v); - v = NULL; + vector_free(&v); } return rc; } @@ -319,8 +315,7 @@ test_vector_nth(void) { assert(vector_nth(v, 2U, (void *)&nth) == 0); assert(nth == 555); - vector_free(v); - v = NULL; + vector_free(&v); test_ok(); } @@ -341,8 +336,7 @@ test_vector_nth(void) { assert(vector_nth(v, 4U, (void *)&nth) != 0); assert(nth == 222); - vector_free(v); - v = NULL; + vector_free(&v); test_ok(); } @@ -350,8 +344,7 @@ test_vector_nth(void) { rc = 0; out: if (v != NULL) { - vector_free(v); - v = NULL; + vector_free(&v); } return rc; } @@ -377,8 +370,7 @@ test_vector_push_back(void) { assert(vector_push_back(v, &after) == 0); assert(v->values[0] == (void *)after); - vector_free(v); - v = NULL; + vector_free(&v); test_ok(); } @@ -414,8 +406,7 @@ test_vector_push_back(void) { assert(v->capacity == new_capacity); assert(v->count == capacity + 2U); - vector_free(v); - v = NULL; + vector_free(&v); test_ok(); } @@ -423,8 +414,7 @@ test_vector_push_back(void) { rc = 0; out: if (v != NULL) { - vector_free(v); - v = NULL; + vector_free(&v); } return rc; } @@ -448,8 +438,7 @@ test_vector_pop_back(void) { assert(vector_pop_back(v, (void *)&val) != 0); assert(val == 222); - vector_free(v); - v = NULL; + vector_free(&v); test_ok(); } @@ -489,16 +478,15 @@ test_vector_pop_back(void) { assert(vector_pop_back(v, (void *)&val) != 0); - vector_free(v); - v = NULL; + vector_free(&v); + test_ok(); } rc = 0; out: if (v != NULL) { - vector_free(v); - v = NULL; + vector_free(&v); } return rc; } @@ -523,8 +511,7 @@ test_vector_contains(void) { assert(!vector_contains(v, &value)); } - vector_free(v); - v = NULL; + vector_free(&v); test_ok(); } @@ -552,8 +539,7 @@ test_vector_contains(void) { assert(vector_contains(v, &i)); } - vector_free(v); - v = NULL; + vector_free(&v); test_ok(); } @@ -561,8 +547,7 @@ test_vector_contains(void) { rc = 0; out: if (v != NULL) { - vector_free(v); - v = NULL; + vector_free(&v); } return rc; } |