# HG changeset patch # User Bruno Haible # Date 1544556995 -3600 # Node ID 5361c721aa56fe849fd8e9d6cd738cbb6cb451e8 # Parent d8b00b7465aa27b2cbf30d1b18fdcae2905adf29 array-set, linkedhash-set, hash-set: Fix tests. * tests/test-array_set.c (cmp_objects_in_array): New function. (check_equals): Use it. * tests/test-hash_set.c: Likewise. * tests/test-linkedhash_set.c: Likewise. diff -r d8b00b7465aa -r 5361c721aa56 ChangeLog --- a/ChangeLog Sat Dec 08 15:25:51 2018 +0100 +++ b/ChangeLog Tue Dec 11 20:36:35 2018 +0100 @@ -1,3 +1,11 @@ +2018-12-11 Bruno Haible + + array-set, linkedhash-set, hash-set: Fix tests. + * tests/test-array_set.c (cmp_objects_in_array): New function. + (check_equals): Use it. + * tests/test-hash_set.c: Likewise. + * tests/test-linkedhash_set.c: Likewise. + 2018-12-08 Bruno Haible Fix comments. diff -r d8b00b7465aa -r 5361c721aa56 tests/test-array_set.c --- a/tests/test-array_set.c Sat Dec 08 15:25:51 2018 +0100 +++ b/tests/test-array_set.c Tue Dec 11 20:36:35 2018 +0100 @@ -36,6 +36,14 @@ #define RANDOM(n) (rand () % (n)) #define RANDOM_OBJECT() objects[RANDOM (SIZEOF (objects))] +static int +cmp_objects_in_array (const void *objptr1, const void *objptr2) +{ + const void *obj1 = *(const void * const *)objptr1; + const void *obj2 = *(const void * const *)objptr2; + return strcmp ((const char *) obj1, (const char *) obj2); +} + static void check_equals (gl_set_t set1, gl_oset_t set2) { @@ -65,10 +73,8 @@ if (n > 0) { - qsort (elements_of_set1, n, sizeof (const void *), - (int (*) (const void *, const void *)) strcmp); - qsort (elements_of_set2, n, sizeof (const void *), - (int (*) (const void *, const void *)) strcmp); + qsort (elements_of_set1, n, sizeof (const void *), cmp_objects_in_array); + qsort (elements_of_set2, n, sizeof (const void *), cmp_objects_in_array); } for (i = 0; i < n; i++) ASSERT (elements_of_set1[i] == elements_of_set2[i]); diff -r d8b00b7465aa -r 5361c721aa56 tests/test-hash_set.c --- a/tests/test-hash_set.c Sat Dec 08 15:25:51 2018 +0100 +++ b/tests/test-hash_set.c Tue Dec 11 20:36:35 2018 +0100 @@ -35,6 +35,14 @@ #define RANDOM(n) (rand () % (n)) #define RANDOM_OBJECT() objects[RANDOM (SIZEOF (objects))] +static int +cmp_objects_in_array (const void *objptr1, const void *objptr2) +{ + const void *obj1 = *(const void * const *)objptr1; + const void *obj2 = *(const void * const *)objptr2; + return strcmp ((const char *) obj1, (const char *) obj2); +} + static void check_equals (gl_set_t set1, gl_set_t set2) { @@ -64,10 +72,8 @@ if (n > 0) { - qsort (elements_of_set1, n, sizeof (const void *), - (int (*) (const void *, const void *)) strcmp); - qsort (elements_of_set2, n, sizeof (const void *), - (int (*) (const void *, const void *)) strcmp); + qsort (elements_of_set1, n, sizeof (const void *), cmp_objects_in_array); + qsort (elements_of_set2, n, sizeof (const void *), cmp_objects_in_array); } for (i = 0; i < n; i++) ASSERT (elements_of_set1[i] == elements_of_set2[i]); diff -r d8b00b7465aa -r 5361c721aa56 tests/test-linkedhash_set.c --- a/tests/test-linkedhash_set.c Sat Dec 08 15:25:51 2018 +0100 +++ b/tests/test-linkedhash_set.c Tue Dec 11 20:36:35 2018 +0100 @@ -35,6 +35,14 @@ #define RANDOM(n) (rand () % (n)) #define RANDOM_OBJECT() objects[RANDOM (SIZEOF (objects))] +static int +cmp_objects_in_array (const void *objptr1, const void *objptr2) +{ + const void *obj1 = *(const void * const *)objptr1; + const void *obj2 = *(const void * const *)objptr2; + return strcmp ((const char *) obj1, (const char *) obj2); +} + static void check_equals (gl_set_t set1, gl_set_t set2) { @@ -64,10 +72,8 @@ if (n > 0) { - qsort (elements_of_set1, n, sizeof (const void *), - (int (*) (const void *, const void *)) strcmp); - qsort (elements_of_set2, n, sizeof (const void *), - (int (*) (const void *, const void *)) strcmp); + qsort (elements_of_set1, n, sizeof (const void *), cmp_objects_in_array); + qsort (elements_of_set2, n, sizeof (const void *), cmp_objects_in_array); } for (i = 0; i < n; i++) ASSERT (elements_of_set1[i] == elements_of_set2[i]);