Replace gratuitous memmove() with memcpy()
authorPeter Eisentraut <peter@eisentraut.org>
Wed, 11 Sep 2024 13:15:49 +0000 (15:15 +0200)
committerPeter Eisentraut <peter@eisentraut.org>
Wed, 11 Sep 2024 13:21:36 +0000 (15:21 +0200)
commit8b5c6a54c4396bb9daeb9ec5d9cbb0d3deedcbe3
tree437b1aaa5d0ee2b9c7a5aae05657a0acfcbaa0cb
parent842265631dfd9e5583d934e2ae328b09bc4b2f8b
Replace gratuitous memmove() with memcpy()

The index access methods all had similar code that copied the
passed-in scan keys to local storage.  They all used memmove() for
that, which is not wrong, but it seems confusing not to use memcpy()
when that would work.  Presumably, this was all once copied from
ancient code and never adjusted.

Discussion: https://www.postgresql.org/message-id/flat/f8c739d9-f48d-4187-b214-df3391ba41ab@eisentraut.org
contrib/bloom/blscan.c
src/backend/access/brin/brin.c
src/backend/access/gin/ginscan.c
src/backend/access/gist/gistscan.c
src/backend/access/hash/hash.c
src/backend/access/nbtree/nbtree.c
src/backend/access/spgist/spgscan.c