On 23.12.25 16:07, Tom Lane wrote:
Peter Eisentraut <[email protected]> writes:
Here, Datum was used to pass around an opaque pointer between a group of
functions. But one might as well use void * for that; the use of Datum
doesn't achieve anything here and is just distracting.
We have some APIs where Datum is used to be agnostic about whether
a pointer or an integer is being passed through. on_shmem_exit()
callers make use of both cases, for example.
Agreed, those are different situations.
These tsearch functions don't need that, and they aren't adjacent
to places using such an API, so I'm fine with this patch as written.
Ok, committed.
