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

triggered
> function now returns the right datatype.

Oops, I got crossed up with Jan's improvements. Ignore this.

--
Peter Eisentraut                  Sernanders väg 10:115
peter_e@gmx.net                   75262 Uppsala

1 files changed, 6 insertions, 14 deletions

diff --git a/src/backend/commands/user.c b/src/backend/commands/user.c
index 349a228872c..de156171a5f 100644
--- 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.43 1999/12/14 00:12:06 momjian Exp $
+ * $Id: user.c,v 1.44 1999/12/14 00:17:33 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -44,20 +44,12 @@ static void CheckPgUserAclNotNull(void);
  *---------------------------------------------------------------------
  */
 
-static void UpdatePgPwdFile(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()
 
-/* 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)
+void
+update_pg_pwd()
 {
 	char	   *filename,
 			   *tempname;