On 20/08/2024 07:46, Michael Paquier wrote:
Hi all,
While hacking on a different thing, I've noticed that
ldap_password_func had the idea to define _PG_fini(). This is
pointless, as library unloading is not supported in the backend, and
something that has been cleaned up from the tree in ab02d702ef08.
That's not something to encourage, perhaps, as well..
How about removing it like in the attached to be consistent?
+1. There's also a prototype for _PG_fini() in fmgr.h, let's remove that
too.
--
Heikki Linnakangas
Neon (https://neon.tech)