Peter Eisentraut <peter@eisentraut.org> 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.
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.
But I don't mean that as carte blanche to make such changes
everywhere.
regards, tom lane