Re: compile warning

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Alvaro Herrera <alvherre(at)dcc(dot)uchile(dot)cl>
Cc: Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: compile warning
Date: 2003-10-10 03:51:09
Message-ID: 200310100351.h9A3p9A19412@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Alvaro Herrera wrote:
> I'm seeing this compile warning on today's CVS tip:
>
> $ make src/backend/commands/tablecmds.o
> gcc -O2 -g -Wall -Wmissing-prototypes -Wmissing-declarations -I./src/include -D_GNU_SOURCE -c -o src/backend/commands/tablecmds.o src/backend/commands/tablecmds.c
> src/backend/commands/tablecmds.c: In function `validateForeignKeyConstraint':
> src/backend/commands/tablecmds.c:3528: warning: dereferencing type-punned pointer will break strict-aliasing rules
>
> $ gcc --version
> gcc (GCC) 3.3.1 (Mandrake Linux 9.2 3.3.1-2mdk)
> Copyright (C) 2003 Free Software Foundation, Inc.
> This is free software; see the source for copying conditions. There is NO
> warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
>
> $ src/bin/pg_config/pg_config --configure
> '--enable-debug' '--enable-nls=es' '--enable-integer-datetimes'

If you change the offending line to:

fcinfo.context = (struct Node *) &trigdata;

I know it shouldn't make a difference, but it is worth a try.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Christopher Browne 2003-10-10 03:53:46 Re: 2-phase commit
Previous Message Bruce Momjian 2003-10-10 03:47:14 Re: [HACKERS] Cannot dump/restore text value \N