Remove _PG_fini().

rjuju · rjuju · commit 609975ac219d · 2022-05-14T14:54:22.000+08:00
Postgres hasn't called this function for more than a decade (even before
extensions were introduced), and version 15 officially removed it so let's get
rid of it too.
diff --git a/pg_wait_sampling.c b/pg_wait_sampling.c
@@ -39,7 +39,6 @@
 PG_MODULE_MAGIC;
 
 void		_PG_init(void);
-void		_PG_fini(void);
 
 /* Global variables */
 bool					shmem_initialized = false;
@@ -359,22 +358,14 @@ _PG_init(void)
 	 */
 	prev_shmem_startup_hook = shmem_startup_hook;
 	shmem_startup_hook		= pgws_shmem_startup;
+	prev_shmem_startup_hook = shmem_startup_hook;
+	shmem_startup_hook		= pgws_shmem_startup;
 	planner_hook_next		= planner_hook;
 	planner_hook			= pgws_planner_hook;
 	prev_ExecutorEnd		= ExecutorEnd_hook;
 	ExecutorEnd_hook		= pgws_ExecutorEnd;
 }
 
-/*
- * Module unload callback
- */
-void
-_PG_fini(void)
-{
-	/* Uninstall hooks. */
-	shmem_startup_hook = prev_shmem_startup_hook;
-}
-
 /*
  * Find PGPROC entry responsible for given pid assuming ProcArrayLock was
  * already taken.
