Re: New compile warnings

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: New compile warnings
Date: 2004-11-20 16:53:31
Message-ID: 21472.1100969611@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> I am seeing the following compile warnings in CVS. I am using for perl:
> Summary of my perl5 (5.0 patchlevel 5 subversion 3) configuration:

I believe these two:

> plperl.c:948: warning: `ret_hv' might be used uninitialized in this function
> plperl.c:949: warning: `ret_av' might be used uninitialized in this function

indicate an actual bug --- at least, it's far from clear that the code
can't try to use an uninitialized value. I trust that the authors of
plperl will step up and fix it; I'm not sufficiently clear on what cases
they are trying to support to want to touch it.

The others indicate sloppiness in the C code generated by perl's XS
functionality. There's nothing we can do about them. FWIW, less
obsolete versions of Perl generate fewer warnings --- the only one of
these that I see on 5.8.0 and up is

> SPI.c:158: warning: unused variable `items'

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2004-11-20 16:55:27 Re: Fwd: SELECT MAX with char => BUG?
Previous Message Katsaros Kwn/nos 2004-11-20 16:50:30 SRF related and other questions

Browse pgsql-patches by date

  From Date Subject
Next Message Andrew Dunstan 2004-11-21 02:02:56 Re: New compile warnings
Previous Message Bruce Momjian 2004-11-20 14:47:13 New compile warnings