Fix compiler warnings (including a seriously bogus elog call); minor

code beautification.

diff --git a/src/backend/utils/adt/trigfuncs.c b/src/backend/utils/adt/trigfuncs.c
index 2f3692ec431f06755f209a7f8034325b038fb957..633f3d20dc5b46039b1c7216b3345482f3b97021 100644 (file)
--- a/src/backend/utils/adt/trigfuncs.c
+++ b/src/backend/utils/adt/trigfuncs.c
@@ -7,25 +7,23 @@
  * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/backend/utils/adt/trigfuncs.c,v 1.1 2008/11/03 20:17:20 adunstan Exp $
+ * $PostgreSQL: pgsql/src/backend/utils/adt/trigfuncs.c,v 1.2 2008/11/04 00:29:39 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
-
-
-
 #include "postgres.h"
-#include "commands/trigger.h"
+
 #include "access/htup.h"
+#include "commands/trigger.h"
+#include "utils/builtins.h"
+
 
 /*
  * suppress_redundant_updates_trigger
  *
  * This trigger function will inhibit an update from being done
  * if the OLD and NEW records are identical.
- *
  */
-
 Datum
 suppress_redundant_updates_trigger(PG_FUNCTION_ARGS)
 {
@@ -35,41 +33,47 @@ suppress_redundant_updates_trigger(PG_FUNCTION_ARGS)
 
     /* make sure it's called as a trigger */
     if (!CALLED_AS_TRIGGER(fcinfo))
-        elog(ERROR, (errcode(ERRCODE_E_R_I_E_TRIGGER_PROTOCOL_VIOLATED),
-                    errmsg("suppress_redundant_updates_trigger: must be called as trigger")));
+        ereport(ERROR,
+               (errcode(ERRCODE_E_R_I_E_TRIGGER_PROTOCOL_VIOLATED),
+                errmsg("suppress_redundant_updates_trigger: must be called as trigger")));
    
     /* and that it's called on update */
     if (! TRIGGER_FIRED_BY_UPDATE(trigdata->tg_event))
-        ereport(ERROR, (errcode(ERRCODE_E_R_I_E_TRIGGER_PROTOCOL_VIOLATED),
-                       errmsg( "suppress_redundant_updates_trigger: may only be called on update")));
+        ereport(ERROR,
+               (errcode(ERRCODE_E_R_I_E_TRIGGER_PROTOCOL_VIOLATED),
+                errmsg("suppress_redundant_updates_trigger: must be called on update")));
 
     /* and that it's called before update */
     if (! TRIGGER_FIRED_BEFORE(trigdata->tg_event))
-        ereport(ERROR, (errcode(ERRCODE_E_R_I_E_TRIGGER_PROTOCOL_VIOLATED),
-                       errmsg( "suppress_redundant_updates_trigger: may only be called before update")));
+        ereport(ERROR,
+               (errcode(ERRCODE_E_R_I_E_TRIGGER_PROTOCOL_VIOLATED),
+                errmsg("suppress_redundant_updates_trigger: must be called before update")));
 
     /* and that it's called for each row */
     if (! TRIGGER_FIRED_FOR_ROW(trigdata->tg_event))
-        ereport(ERROR, (errcode(ERRCODE_E_R_I_E_TRIGGER_PROTOCOL_VIOLATED),
-                       errmsg( "suppress_redundant_updates_trigger: may only be called for each row")));
+        ereport(ERROR,
+               (errcode(ERRCODE_E_R_I_E_TRIGGER_PROTOCOL_VIOLATED),
+                errmsg("suppress_redundant_updates_trigger: must be called for each row")));
 
-   /* get tuple data, set default return */
+   /* get tuple data, set default result */
    rettuple  = newtuple = trigdata->tg_newtuple;
    oldtuple = trigdata->tg_trigtuple;
 
    newheader = newtuple->t_data;
    oldheader = oldtuple->t_data;
 
+   /* if the tuple payload is the same ... */
     if (newtuple->t_len == oldtuple->t_len &&
        newheader->t_hoff == oldheader->t_hoff &&
        (HeapTupleHeaderGetNatts(newheader) == 
-        HeapTupleHeaderGetNatts(oldheader) ) &&
+        HeapTupleHeaderGetNatts(oldheader)) &&
        ((newheader->t_infomask & ~HEAP_XACT_MASK) == 
-        (oldheader->t_infomask & ~HEAP_XACT_MASK) )&&
+        (oldheader->t_infomask & ~HEAP_XACT_MASK)) &&
        memcmp(((char *)newheader) + offsetof(HeapTupleHeaderData, t_bits),
               ((char *)oldheader) + offsetof(HeapTupleHeaderData, t_bits),
               newtuple->t_len - offsetof(HeapTupleHeaderData, t_bits)) == 0)
    {
+       /* ... then suppress the update */
        rettuple = NULL;
    }