lib/gnulib
1
0
Fork 0
mirror of https://git.savannah.gnu.org/git/gnulib.git synced 2026-10-24 00:48:56 +03:00
Code Activity

tests: Support CC="clang -flto -fsanitize=cfi -fvisibility=hidden".

Browse Source 
Reported by Lasse Collin in
<https://lists.gnu.org/archive/html/bug-gnulib/2026-08/msg00190.html>.

* tests/test-array_map.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-array_omap.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-avltree_omap.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-rbtree_omap.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-array_set.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-array_oset.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-avltree_oset.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-rbtree_oset.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-oset-update.h (test_update): Likewise.
This commit is contained in:
Bruno Haible
2026-08-15 17:46:48 +02:00
parent cec6411b0e
commit 9388edc18a
10 changed files with 127 additions and 40 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
ChangeLog
View File

@@ -1,3 +1,26 @@
2026-08-15 Bruno Haible <bruno@clisp.org>
tests: Support CC="clang -flto -fsanitize=cfi -fvisibility=hidden".
Reported by Lasse Collin in
<https://lists.gnu.org/archive/html/bug-gnulib/2026-08/msg00190.html>.
* tests/test-array_map.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-array_omap.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-avltree_omap.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-rbtree_omap.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-array_set.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-array_oset.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-avltree_oset.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-rbtree_oset.c (string_compare): New function.
(main): Use instead of casting strcmp to a different function prototype.
* tests/test-oset-update.h (test_update): Likewise.
2026-08-15 Bruno Haible <bruno@clisp.org>
memmove: Avoid undefined behaviour upon memmove (NULL, ptr, 0).

24
tests/test-array_map.c
View File

@@ -34,6 +34,14 @@ static const char *objects[30] =
"p", "q", "r", "s", "t", "u", "v", "w", "x", "y", "z", "<", ">", "[", "]"
};
static int
string_compare (const void *elt1, const void *elt2)
{
const char *s1 = elt1;
const char *s2 = elt2;
return strcmp (s1, s2);
}
static bool
string_equals (const void *x1, const void *x2)
{
@@ -143,12 +151,12 @@ main (int argc, char *argv[])
const char *key = RANDOM_OBJECT ();
const char *value = RANDOM_OBJECT ();
bool added = gl_map_nx_put (map1, key, value);
size_t index = gl_sortedlist_indexof (keys, (gl_listelement_compar_fn)strcmp, key);
size_t index = gl_sortedlist_indexof (keys, string_compare, key);
ASSERT (added == (index == (size_t)(-1)));
if (added)
{
gl_sortedlist_add (keys, (gl_listelement_compar_fn)strcmp, key);
index = gl_sortedlist_indexof (keys, (gl_listelement_compar_fn)strcmp, key);
gl_sortedlist_add (keys, string_compare, key);
index = gl_sortedlist_indexof (keys, string_compare, key);
gl_list_add_at (values, index, value);
}
else
@@ -166,7 +174,7 @@ main (int argc, char *argv[])
const char *key = RANDOM_OBJECT ();
const void *ret = gl_map_get (map1, key);
size_t index =
gl_sortedlist_indexof (keys, (gl_listelement_compar_fn)strcmp, key);
gl_sortedlist_indexof (keys, string_compare, key);
ASSERT (ret
== (index != (size_t)(-1) ? gl_list_get_at (values, index) : NULL));
}
@@ -177,12 +185,12 @@ main (int argc, char *argv[])
const char *value = RANDOM_OBJECT ();
bool added = gl_map_nx_put (map1, key, value);
size_t index =
gl_sortedlist_indexof (keys, (gl_listelement_compar_fn)strcmp, key);
gl_sortedlist_indexof (keys, string_compare, key);
ASSERT (added == (index == (size_t)(-1)));
if (added)
{
gl_sortedlist_add (keys, (gl_listelement_compar_fn)strcmp, key);
index = gl_sortedlist_indexof (keys, (gl_listelement_compar_fn)strcmp, key);
gl_sortedlist_add (keys, string_compare, key);
index = gl_sortedlist_indexof (keys, string_compare, key);
gl_list_add_at (values, index, value);
}
else
@@ -194,7 +202,7 @@ main (int argc, char *argv[])
const char *key = RANDOM_OBJECT ();
bool removed = gl_map_remove (map1, key);
size_t index =
gl_sortedlist_indexof (keys, (gl_listelement_compar_fn)strcmp, key);
gl_sortedlist_indexof (keys, string_compare, key);
ASSERT (removed == (index != (size_t)(-1)));
if (removed)
{

26
tests/test-array_omap.c
View File

@@ -73,6 +73,14 @@ check_all (gl_omap_t map1, gl_list_t keys, gl_list_t values)
check_equals (map1, keys, values);
}
static int
string_compare (const void *elt1, const void *elt2)
{
const char *s1 = elt1;
const char *s2 = elt2;
return strcmp (s1, s2);
}
int
main (int argc, char *argv[])
{
@@ -88,7 +96,7 @@ main (int argc, char *argv[])
size_t initial_size = RANDOM (20);
/* Create map1. */
map1 = gl_omap_nx_create_empty (GL_ARRAY_OMAP, (gl_mapkey_compar_fn) strcmp, NULL, NULL);
map1 = gl_omap_nx_create_empty (GL_ARRAY_OMAP, string_compare, NULL, NULL);
ASSERT (map1 != NULL);
/* Create keys and values. */
@@ -103,12 +111,12 @@ main (int argc, char *argv[])
const char *key = RANDOM_OBJECT ();
const char *value = RANDOM_OBJECT ();
bool added = gl_omap_nx_put (map1, key, value);
size_t index = gl_sortedlist_indexof (keys, (gl_listelement_compar_fn)strcmp, key);
size_t index = gl_sortedlist_indexof (keys, string_compare, key);
ASSERT (added == (index == (size_t)(-1)));
if (added)
{
gl_sortedlist_add (keys, (gl_listelement_compar_fn)strcmp, key);
index = gl_sortedlist_indexof (keys, (gl_listelement_compar_fn)strcmp, key);
gl_sortedlist_add (keys, string_compare, key);
index = gl_sortedlist_indexof (keys, string_compare, key);
gl_list_add_at (values, index, value);
}
else
@@ -126,7 +134,7 @@ main (int argc, char *argv[])
const char *key = RANDOM_OBJECT ();
const void *ret = gl_omap_get (map1, key);
size_t index =
gl_sortedlist_indexof (keys, (gl_listelement_compar_fn)strcmp, key);
gl_sortedlist_indexof (keys, string_compare, key);
ASSERT (ret
== (index != (size_t)(-1) ? gl_list_get_at (values, index) : NULL));
}
@@ -137,12 +145,12 @@ main (int argc, char *argv[])
const char *value = RANDOM_OBJECT ();
bool added = gl_omap_nx_put (map1, key, value);
size_t index =
gl_sortedlist_indexof (keys, (gl_listelement_compar_fn)strcmp, key);
gl_sortedlist_indexof (keys, string_compare, key);
ASSERT (added == (index == (size_t)(-1)));
if (added)
{
gl_sortedlist_add (keys, (gl_listelement_compar_fn)strcmp, key);
index = gl_sortedlist_indexof (keys, (gl_listelement_compar_fn)strcmp, key);
gl_sortedlist_add (keys, string_compare, key);
index = gl_sortedlist_indexof (keys, string_compare, key);
gl_list_add_at (values, index, value);
}
else
@@ -154,7 +162,7 @@ main (int argc, char *argv[])
const char *key = RANDOM_OBJECT ();
bool removed = gl_omap_remove (map1, key);
size_t index =
gl_sortedlist_indexof (keys, (gl_listelement_compar_fn)strcmp, key);
gl_sortedlist_indexof (keys, string_compare, key);
ASSERT (removed == (index != (size_t)(-1)));
if (removed)
{

26
tests/test-array_oset.c
View File

@@ -27,8 +27,6 @@
#include "gl_array_list.h"
#include "macros.h"
#include "test-oset-update.h"
static const char *objects[30] =
{
"a", "b", "c", "d", "e", "f", "g", "h", "i", "j", "k", "l", "m", "n", "o",
@@ -68,6 +66,14 @@ check_all (gl_oset_t set1, gl_list_t set2)
check_equals (set1, set2);
}
static int
string_compare (const void *elt1, const void *elt2)
{
const char *s1 = elt1;
const char *s2 = elt2;
return strcmp (s1, s2);
}
static bool
is_at_least (const void *elt, const void *threshold)
{
@@ -88,6 +94,8 @@ gl_sortedlist_indexof_atleast (gl_list_t set,
return (size_t)(-1);
}
#include "test-oset-update.h"
int
main (int argc, char *argv[])
{
@@ -102,7 +110,7 @@ main (int argc, char *argv[])
size_t initial_size = RANDOM (20);
/* Create set1. */
set1 = gl_oset_nx_create_empty (GL_ARRAY_OSET, (gl_setelement_compar_fn) strcmp, NULL);
set1 = gl_oset_nx_create_empty (GL_ARRAY_OSET, string_compare, NULL);
ASSERT (set1 != NULL);
/* Create set2. */
@@ -115,9 +123,9 @@ main (int argc, char *argv[])
{
const char *obj = RANDOM_OBJECT ();
ASSERT (gl_oset_nx_add (set1, obj)
== (gl_sortedlist_search (set2, (gl_listelement_compar_fn)strcmp, obj) != NULL
== (gl_sortedlist_search (set2, string_compare, obj) != NULL
? false
: (gl_sortedlist_add (set2, (gl_listelement_compar_fn)strcmp, obj), true)));
: (gl_sortedlist_add (set2, string_compare, obj), true)));
check_all (set1, set2);
}
@@ -130,23 +138,23 @@ main (int argc, char *argv[])
{
const char *obj = RANDOM_OBJECT ();
ASSERT (gl_oset_search (set1, obj)
== (gl_sortedlist_search (set2, (gl_listelement_compar_fn)strcmp, obj) != NULL));
== (gl_sortedlist_search (set2, string_compare, obj) != NULL));
}
break;
case 1:
{
const char *obj = RANDOM_OBJECT ();
ASSERT (gl_oset_nx_add (set1, obj)
== (gl_sortedlist_search (set2, (gl_listelement_compar_fn)strcmp, obj) != NULL
== (gl_sortedlist_search (set2, string_compare, obj) != NULL
? false
: (gl_sortedlist_add (set2, (gl_listelement_compar_fn)strcmp, obj), true)));
: (gl_sortedlist_add (set2, string_compare, obj), true)));
}
break;
case 2:
{
const char *obj = RANDOM_OBJECT ();
ASSERT (gl_oset_remove (set1, obj)
== gl_sortedlist_remove (set2, (gl_listelement_compar_fn)strcmp, obj));
== gl_sortedlist_remove (set2, string_compare, obj));
}
break;
case 3:

10
tests/test-array_set.c
View File

@@ -88,6 +88,14 @@ check_all (gl_set_t set1, gl_oset_t set2)
check_equals (set1, set2);
}
static int
string_compare (const void *elt1, const void *elt2)
{
const char *s1 = elt1;
const char *s2 = elt2;
return strcmp (s1, s2);
}
int
main (int argc, char *argv[])
{
@@ -106,7 +114,7 @@ main (int argc, char *argv[])
ASSERT (set1 != NULL);
/* Create set2. */
set2 = gl_oset_create_empty (GL_ARRAY_OSET, (gl_setelement_compar_fn) strcmp, NULL);
set2 = gl_oset_create_empty (GL_ARRAY_OSET, string_compare, NULL);
check_all (set1, set2);

12
tests/test-avltree_omap.c
View File

@@ -69,6 +69,14 @@ check_all (gl_omap_t map1, gl_omap_t map2)
check_equals (map1, map2);
}
static int
string_compare (const void *key1, const void *key2)
{
const char *s1 = key1;
const char *s2 = key2;
return strcmp (s1, s2);
}
int
main (int argc, char *argv[])
{
@@ -82,11 +90,11 @@ main (int argc, char *argv[])
size_t initial_size = RANDOM (20);
/* Create map1. */
map1 = gl_omap_nx_create_empty (GL_ARRAY_OMAP, (gl_mapkey_compar_fn) strcmp, NULL, NULL);
map1 = gl_omap_nx_create_empty (GL_ARRAY_OMAP, string_compare, NULL, NULL);
ASSERT (map1 != NULL);
/* Create map2. */
map2 = gl_omap_nx_create_empty (GL_AVLTREE_OMAP, (gl_mapkey_compar_fn) strcmp, NULL, NULL);
map2 = gl_omap_nx_create_empty (GL_AVLTREE_OMAP, string_compare, NULL, NULL);
ASSERT (map2 != NULL);
check_all (map1, map2);

16
tests/test-avltree_oset.c
View File

@@ -26,8 +26,6 @@
#include "gl_array_oset.h"
#include "macros.h"
#include "test-oset-update.h"
extern void gl_avltree_oset_check_invariants (gl_oset_t set);
static const char *objects[30] =
@@ -68,12 +66,22 @@ check_all (gl_oset_t set1, gl_oset_t set2)
check_equals (set1, set2);
}
static int
string_compare (const void *elt1, const void *elt2)
{
const char *s1 = elt1;
const char *s2 = elt2;
return strcmp (s1, s2);
}
static bool
is_at_least (const void *elt, const void *threshold)
{
return strcmp ((const char *) elt, (const char *) threshold) >= 0;
}
#include "test-oset-update.h"
int
main (int argc, char *argv[])
{
@@ -87,11 +95,11 @@ main (int argc, char *argv[])
size_t initial_size = RANDOM (20);
/* Create set1. */
set1 = gl_oset_nx_create_empty (GL_ARRAY_OSET, (gl_setelement_compar_fn) strcmp, NULL);
set1 = gl_oset_nx_create_empty (GL_ARRAY_OSET, string_compare, NULL);
ASSERT (set1 != NULL);
/* Create set2. */
set2 = gl_oset_nx_create_empty (GL_AVLTREE_OSET, (gl_setelement_compar_fn) strcmp, NULL);
set2 = gl_oset_nx_create_empty (GL_AVLTREE_OSET, string_compare, NULL);
ASSERT (set2 != NULL);
check_all (set1, set2);

2
tests/test-oset-update.h
View File

@@ -30,7 +30,7 @@ test_update (gl_oset_implementation_t implementation)
char D[2] = "D";
gl_oset_t set1 =
gl_oset_nx_create_empty (implementation, (gl_setelement_compar_fn) strcmp, NULL);
gl_oset_nx_create_empty (implementation, string_compare, NULL);
ASSERT (set1 != NULL);
/* Fill the set. */

12
tests/test-rbtree_omap.c
View File

@@ -69,6 +69,14 @@ check_all (gl_omap_t map1, gl_omap_t map2)
check_equals (map1, map2);
}
static int
string_compare (const void *key1, const void *key2)
{
const char *s1 = key1;
const char *s2 = key2;
return strcmp (s1, s2);
}
int
main (int argc, char *argv[])
{
@@ -82,11 +90,11 @@ main (int argc, char *argv[])
size_t initial_size = RANDOM (20);
/* Create map1. */
map1 = gl_omap_nx_create_empty (GL_ARRAY_OMAP, (gl_mapkey_compar_fn) strcmp, NULL, NULL);
map1 = gl_omap_nx_create_empty (GL_ARRAY_OMAP, string_compare, NULL, NULL);
ASSERT (map1 != NULL);
/* Create map2. */
map2 = gl_omap_nx_create_empty (GL_RBTREE_OMAP, (gl_mapkey_compar_fn) strcmp, NULL, NULL);
map2 = gl_omap_nx_create_empty (GL_RBTREE_OMAP, string_compare, NULL, NULL);
ASSERT (map2 != NULL);
check_all (map1, map2);

16
tests/test-rbtree_oset.c
View File

@@ -26,8 +26,6 @@
#include "gl_array_oset.h"
#include "macros.h"
#include "test-oset-update.h"
extern void gl_rbtree_oset_check_invariants (gl_oset_t set);
static const char *objects[30] =
@@ -68,12 +66,22 @@ check_all (gl_oset_t set1, gl_oset_t set2)
check_equals (set1, set2);
}
static int
string_compare (const void *elt1, const void *elt2)
{
const char *s1 = elt1;
const char *s2 = elt2;
return strcmp (s1, s2);
}
static bool
is_at_least (const void *elt, const void *threshold)
{
return strcmp ((const char *) elt, (const char *) threshold) >= 0;
}
#include "test-oset-update.h"
int
main (int argc, char *argv[])
{
@@ -87,11 +95,11 @@ main (int argc, char *argv[])
size_t initial_size = RANDOM (20);
/* Create set1. */
set1 = gl_oset_nx_create_empty (GL_ARRAY_OSET, (gl_setelement_compar_fn) strcmp, NULL);
set1 = gl_oset_nx_create_empty (GL_ARRAY_OSET, string_compare, NULL);
ASSERT (set1 != NULL);
/* Create set2. */
set2 = gl_oset_nx_create_empty (GL_RBTREE_OSET, (gl_setelement_compar_fn) strcmp, NULL);
set2 = gl_oset_nx_create_empty (GL_RBTREE_OSET, string_compare, NULL);
ASSERT (set2 != NULL);
check_all (set1, set2);