Re: clang's static checker report.

From: Greg Stark <gsstark(at)mit(dot)edu>
To: Grzegorz Jaskiewicz <gj(at)pointblue(dot)com(dot)pl>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Alex Hunsaker <badalex(at)gmail(dot)com>, pgsql-hackers Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: clang's static checker report.
Date: 2009-08-30 15:10:49
Message-ID: 407d949e0908300810g524ed757sea76163839024e33@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Aug 30, 2009 at 3:57 PM, Grzegorz Jaskiewicz<gj(at)pointblue(dot)com(dot)pl> wrote:
>
> On 30 Aug 2009, at 15:46, Greg Stark wrote:
>
>> So three of the four dead initialization warnings are legitimate -- if
>> minor -- errors. Attached is a patch to remove the redundant
>> initializations.
>
>
> well, all I can do is this: http://llvm.org/bugs/show_bug.cgi?id=4832
>
> I find it hard to belive tho, that it only found 4 possible positives ;)

You might try something like the attached.

--
greg
http://mit.edu/~gsstark/resume.pdf

Attachment Content-Type Size
clang-palloc-nonulls.patch text/x-patch 1.5 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2009-08-30 16:09:48 Re: PQexecPrepared() behavior
Previous Message Grzegorz Jaskiewicz 2009-08-30 14:57:22 Re: clang's static checker report.