Thread: pgsql: suppress_redundant_updates_trigger function.

pgsql: suppress_redundant_updates_trigger function.

From
adunstan@postgresql.org (Andrew Dunstan)
Date:
Log Message:
-----------
suppress_redundant_updates_trigger function.

Modified Files:
--------------
    pgsql/doc/src/sgml:
        func.sgml (r1.452 -> r1.453)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/func.sgml?r1=1.452&r2=1.453)
    pgsql/src/backend/utils/adt:
        Makefile (r1.69 -> r1.70)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/Makefile?r1=1.69&r2=1.70)
    pgsql/src/include/catalog:
        pg_proc.h (r1.521 -> r1.522)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/catalog/pg_proc.h?r1=1.521&r2=1.522)
    pgsql/src/include/utils:
        builtins.h (r1.325 -> r1.326)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/utils/builtins.h?r1=1.325&r2=1.326)
    pgsql/src/test/regress/expected:
        triggers.out (r1.24 -> r1.25)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/expected/triggers.out?r1=1.24&r2=1.25)
    pgsql/src/test/regress/sql:
        triggers.sql (r1.13 -> r1.14)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/sql/triggers.sql?r1=1.13&r2=1.14)

Added Files:
-----------
    pgsql/src/backend/utils/adt:
        trigfuncs.c (r1.1)

(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/trigfuncs.c?rev=1.1&content-type=text/x-cvsweb-markup)

Re: pgsql: suppress_redundant_updates_trigger function.

From
"Alex Hunsaker"
Date:
On Mon, Nov 3, 2008 at 13:17, Andrew Dunstan <adunstan@postgresql.org> wrote:
> Log Message:
> -----------
> suppress_redundant_updates_trigger function.
>
> Modified Files:
> --------------
>    pgsql/doc/src/sgml:
>        func.sgml (r1.452 -> r1.453)
>        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/func.sgml?r1=1.452&r2=1.453)
>    pgsql/src/backend/utils/adt:
>        Makefile (r1.69 -> r1.70)
>        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/Makefile?r1=1.69&r2=1.70)
>    pgsql/src/include/catalog:
>        pg_proc.h (r1.521 -> r1.522)
>        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/catalog/pg_proc.h?r1=1.521&r2=1.522)
>    pgsql/src/include/utils:
>        builtins.h (r1.325 -> r1.326)
>        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/utils/builtins.h?r1=1.325&r2=1.326)
>    pgsql/src/test/regress/expected:
>        triggers.out (r1.24 -> r1.25)
>        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/expected/triggers.out?r1=1.24&r2=1.25)
>    pgsql/src/test/regress/sql:
>        triggers.sql (r1.13 -> r1.14)
>        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/sql/triggers.sql?r1=1.13&r2=1.14)
>
> Added Files:
> -----------
>    pgsql/src/backend/utils/adt:
>        trigfuncs.c (r1.1)
>
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/trigfuncs.c?rev=1.1&content-type=text/x-cvsweb-markup)
>

Hrm this generates 2 new compiler warnings for me

gcc -v
Using built-in specs.
Target: i686-pc-linux-gnu
Configured with: ../configure --prefix=/usr --enable-shared
--enable-languages=c,c++,fortran,objc,obj-c++,treelang
--enable-threads=posix --mandir=/usr/share/man
--infodir=/usr/share/info --enable-__cxa_atexit --disable-multilib
--libdir=/usr/lib --libexecdir=/usr/lib --enable-clocale=gnu
--disable-libstdcxx-pch --with-tune=generic
Thread model: posix
gcc version 4.3.2 (GCC)

trigfuncs.c:30: warning: no previous prototype for
'suppress_redundant_updates_trigger'
trigfuncs.c: In function 'suppress_redundant_updates_trigger':
trigfuncs.c:39: warning: passing argument 2 of 'elog_finish' makes
pointer from integer without a cast

Re: pgsql: suppress_redundant_updates_trigger function.

From
Tom Lane
Date:
"Alex Hunsaker" <badalex@gmail.com> writes:
> On Mon, Nov 3, 2008 at 13:17, Andrew Dunstan <adunstan@postgresql.org> wrote:
>> suppress_redundant_updates_trigger function.

> Hrm this generates 2 new compiler warnings for me

Yeah, I just ran into that too.  Cleanup committed.

            regards, tom lane