>From what I gather, this should be a little cleaner because the

triggered
function now returns the right datatype.

--
Peter Eisentraut                  Sernanders väg 10:115

diff --git a/src/backend/commands/user.c b/src/backend/commands/user.c
index 56dce87f7e001fa0dcb3cf6aeeb3426f05f26894..349a228872c42a685ad4d8fa277295bbf02268ca 100644 (file)
--- a/src/backend/commands/user.c
+++ b/src/backend/commands/user.c
@@ -5,7 +5,7 @@
  *
  * Copyright (c) 1994, Regents of the University of California
  *
- * $Id: user.c,v 1.42 1999/12/14 00:08:13 momjian Exp $
+ * $Id: user.c,v 1.43 1999/12/14 00:12:06 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -44,12 +44,20 @@ static void CheckPgUserAclNotNull(void);
  *---------------------------------------------------------------------
  */
 
-/* This is the old name. Now uses a lower case name to be able to call this
-   from SQL. */
-#define UpdatePgPwdFile() update_pg_pwd()
+static void UpdatePgPwdFile(void);
 
-void
-update_pg_pwd()
+/* This is a wrapper, so the below function can be called from a trigger
+   (used to update pg_pwd from pg_shadow) */
+HeapTuple update_pg_pwd()
+{
+    UpdatePgPwdFile();
+    return NULL;
+}
+
+
+
+static void
+UpdatePgPwdFile(void)
 {
        char       *filename,
                           *tempname;