On Wed, 2023-12-06 at 07:39 +0700, John Naylor wrote: > "git grep cstring_hash" found nothing, so not sure what you're > asking.
Sorry, I meant string_hash(). Your v5-0002 changes the way hashing works for cstrings, and that means it's no longer equivalent to hash_bytes with strlen. That's probably fine, but someone might assume that they are equivalent. > > In the abstract, I consider (b) to be a layering violation. As a > consequence, the cleverness in (b) is not confined to one or two > places, but is smeared over a whole bunch of places. I find it hard > to > follow. OK. I am fine with (a). Regards, Jeff Davis