{
struct strmap *n;
- /* Empty map? */
- if (!map->u.n)
- return NULL;
- n = closest((struct strmap *)map, member);
- if (streq(member, n->u.s))
- return n->v;
+ /* Not empty map? */
+ if (map->u.n) {
+ n = closest((struct strmap *)map, member);
+ if (streq(member, n->u.s))
+ return n->v;
+ }
+ errno = ENOENT;
return NULL;
}
u8 direction = 0; /* prevent bogus gcc warning. */
/* Empty map? */
- if (!map->u.n)
+ if (!map->u.n) {
+ errno = ENOENT;
return NULL;
+ }
/* Find closest, but keep track of parent. */
n = map;
}
/* Did we find it? */
- if (!streq(member, n->u.s))
+ if (!streq(member, n->u.s)) {
+ errno = ENOENT;
return NULL;
+ }
ret = n->u.s;
if (valuep)
* @map: the typed strmap to search.
* @member: the string to search for.
*
- * Returns the value, or NULL if it isn't in the map.
+ * Returns the value, or NULL if it isn't in the map (and sets errno = ENOENT).
*
* Example:
* int *val = strmap_get(&map, "hello");
* @member: the string to remove from the map.
* @valuep: the value (if non-NULL)
*
- * This returns the string which was passed to strmap_map(), or NULL.
+ * This returns the string which was passed to strmap_map(), or NULL if
+ * it was not in the map (and sets errno = ENOENT).
+ *
* This means that if you allocated a string (eg. using strdup()), you
* can free it here. Similarly, the value is returned in @valuep if
* @valuep is not NULL.
char *v;
/* This is how many tests you plan to run */
- plan_tests(32);
+ plan_tests(42);
strmap_init(&map);
ok1(!strmap_get(&map, str));
+ ok1(errno == ENOENT);
ok1(!strmap_get(&map, none));
+ ok1(errno == ENOENT);
ok1(!strmap_del(&map, str, NULL));
+ ok1(errno == ENOENT);
ok1(!strmap_del(&map, none, NULL));
+ ok1(errno == ENOENT);
ok1(strmap_add(&map, str, val));
ok1(strmap_get(&map, str) == val);
/* We compare the string, not the pointer. */
ok1(strmap_get(&map, dup) == val);
ok1(!strmap_get(&map, none));
+ ok1(errno == ENOENT);
/* Add a duplicate should fail. */
ok1(!strmap_add(&map, dup, val));
ok1(strmap_del(&map, dup, &v) == str);
ok1(v == val);
ok1(!strmap_get(&map, str));
+ ok1(errno == ENOENT);
ok1(!strmap_get(&map, none));
+ ok1(errno == ENOENT);
/* Try insert and delete of empty string. */
ok1(strmap_add(&map, none, none));
ok1(strmap_get(&map, none) == none);
ok1(!strmap_get(&map, str));
+ ok1(errno == ENOENT);
/* Delete should return original string. */
ok1(strmap_del(&map, "", &v) == none);
ok1(v == none);
ok1(!strmap_get(&map, str));
+ ok1(errno == ENOENT);
ok1(!strmap_get(&map, none));
+ ok1(errno == ENOENT);
/* Both at once... */
ok1(strmap_add(&map, none, none));