It seems redundant: strmap_get() and strmap_del() can only return NULL
when the string is not a member. However, it became clear in writing
ccan/tsort that it's much more convenient for callers if we set errno
in this case too, so they can pass it up.