From 591abe1b727a70e1ae8dcfa6e384a41bd40367ed Mon Sep 17 00:00:00 2001
From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Sat, 29 Sep 2007 23:32:42 +0000
Subject: [PATCH] Remove bogus commutator marking --- the module doesn't
 actually supply any commutator operator for =(chkpass,text), so this was
 creating a shell operator that would fail on use.  Found by opr_sanity
 testing.

---
 contrib/chkpass/chkpass.sql.in        | 10 +++-------
 contrib/chkpass/uninstall_chkpass.sql |  2 --
 2 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/contrib/chkpass/chkpass.sql.in b/contrib/chkpass/chkpass.sql.in
index 68cd1e27659..40a0d722c90 100644
--- a/contrib/chkpass/chkpass.sql.in
+++ b/contrib/chkpass/chkpass.sql.in
@@ -4,7 +4,7 @@
 --  darcy@druid.net
 --  http://www.druid.net/darcy/
 -- 
---  $PostgreSQL: pgsql/contrib/chkpass/chkpass.sql.in,v 1.6 2005/01/29 22:35:01 tgl Exp $
+--  $PostgreSQL: pgsql/contrib/chkpass/chkpass.sql.in,v 1.7 2007/09/29 23:32:42 tgl Exp $
 --
 --  best viewed with tabs set to 4
 --
@@ -53,17 +53,13 @@ CREATE FUNCTION ne(chkpass, text)
 	LANGUAGE C STRICT;
 
 --
---	Now the operators.  Note how some of the parameters to some
---	of the 'create operator' commands are commented out.  This
---	is because they reference as yet undefined operators, and
---	will be implicitly defined when those are, further down.
+--	Now the operators.
 --
 
 CREATE OPERATOR = (
 	leftarg = chkpass,
 	rightarg = text,
-	commutator = =,
---	negator = <>,
+	negator = <>,
 	procedure = eq
 );
 
diff --git a/contrib/chkpass/uninstall_chkpass.sql b/contrib/chkpass/uninstall_chkpass.sql
index 4a4737a1d55..34ad3fde8fb 100644
--- a/contrib/chkpass/uninstall_chkpass.sql
+++ b/contrib/chkpass/uninstall_chkpass.sql
@@ -4,8 +4,6 @@ DROP OPERATOR <>(chkpass, text);
 
 DROP OPERATOR =(chkpass, text);
 
-DROP OPERATOR =(text, chkpass);
-
 DROP FUNCTION ne(chkpass, text);
 
 DROP FUNCTION eq(chkpass, text);
-- 
GitLab