diff options
| author | John Naylor | 2024-01-21 12:19:14 +0000 |
|---|---|---|
| committer | John Naylor | 2024-02-06 07:39:36 +0000 |
| commit | 9ed3ee5001b6a2d4cb0166eb8f12a457f30aaca4 (patch) | |
| tree | 67ea542a24cea71cad13cc64ca36bd425d7685f7 /src/backend | |
| parent | 1f61680327c962d5bafdbf6a04ad9bb0f7232f92 (diff) | |
Simplify initialization of incremental hash state
The standalone functions fasthash{32,64} use length for two purposes:
how many bytes to hash, and how to perturb the internal seed.
Developers using the incremental interface may not know the length
ahead of time (e.g. for C strings). In this case, it's advised to
pass length to the finalizer, but initialization still needed some
length up front, in the form of a placeholder macro.
Separate the concerns by having the standalone functions perturb the
internal seed themselves from their own length parameter, allowing
to remove "len" from fasthash_init(), as well as the placeholder macro.
Discussion: https://postgr.es/m/CANWCAZbTUk2LOyhsFo33gjLyLAHZ7ucXCi5K9u%3D%2BPtnTShDKtw%40mail.gmail.com
Diffstat (limited to 'src/backend')
| -rw-r--r-- | src/backend/catalog/namespace.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/catalog/namespace.c b/src/backend/catalog/namespace.c index b610aa62423..8df30b24401 100644 --- a/src/backend/catalog/namespace.c +++ b/src/backend/catalog/namespace.c @@ -256,7 +256,7 @@ spcachekey_hash(SearchPathCacheKey key) fasthash_state hs; int sp_len; - fasthash_init(&hs, FH_UNKNOWN_LENGTH, 0); + fasthash_init(&hs, 0); hs.accum = key.roleid; fasthash_combine(&hs); |