From: | Kevin Grittner <kgrittn(at)ymail(dot)com> |
---|---|
To: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Fw: [COMMITTERS] pgsql: Fix blatantly broken record_image_cmp() logic for pass-by-value |
Date: | 2013-11-09 22:00:12 |
Message-ID: | 1384034412.80888.YahooMailNeo@web162901.mail.bf1.yahoo.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Forwarded to -hackers
----- Forwarded Message -----
> From: Kevin Grittner <kgrittn(at)ymail(dot)com>
> To: Kevin Grittner <kgrittn(at)ymail(dot)com>; Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
> Cc: "pgsql-committers(at)postgresql(dot)org" <pgsql-committers(at)postgresql(dot)org>
> Sent: Friday, November 8, 2013 4:33 PM
> Subject: Re: [COMMITTERS] pgsql: Fix blatantly broken record_image_cmp() logic for pass-by-value
>
> Kevin Grittner <kgrittn(at)ymail(dot)com> wrote:
>
>> I distilled it down to the simplest case I could find which
>> failed to produce the warning; attached.
>
> For a compiler which seems to like to generate warnings for really
> esoteric cases, clang falls down rather badly on uninitialized
> variables -- at least on the package for Ubuntu 12.10. It does not
> complain at all about this:
>
>
> int warning_test(int a);
> int warning_test(int a)
> {
> int result;
>
> if (a == 1)
> result = 1;
>
> return result;
> }
>
>
> I assume that everyone here agrees that merits a warning?
>
> Bug reports filed.
>
>
> --
> Kevin Grittner
> EDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2013-11-09 22:17:13 | Re: Fw: [COMMITTERS] pgsql: Fix blatantly broken record_image_cmp() logic for pass-by-value |
Previous Message | Kevin Grittner | 2013-11-09 21:58:53 | Fw: [COMMITTERS] pgsql: Fix blatantly broken record_image_cmp() logic for pass-by-value |
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2013-11-09 22:17:13 | Re: Fw: [COMMITTERS] pgsql: Fix blatantly broken record_image_cmp() logic for pass-by-value |
Previous Message | Kevin Grittner | 2013-11-09 21:58:53 | Fw: [COMMITTERS] pgsql: Fix blatantly broken record_image_cmp() logic for pass-by-value |