From: | Zdenek Kotala <Zdenek(dot)Kotala(at)Sun(dot)COM> |
---|---|
To: | Kris Jurka <books(at)ejurka(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Stefan Kaltenbrunner <stefan(at)kaltenbrunner(dot)cc>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Gorm(dot)Andersen(at)Sun(dot)COM |
Subject: | Re: compiler warnings on the buildfarm |
Date: | 2007-07-13 13:35:14 |
Message-ID: | 46977F92.7000000@sun.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Kris Jurka wrote:
>
>
> On Fri, 13 Jul 2007, Zdenek Kotala wrote:
>
>> Tom Lane wrote:
>>> Kris Jurka <books(at)ejurka(dot)com> writes:
>>>> So pam_message ** isn't const.
>>>
>>> Ah, thanks. I see luna_moth is giving the same warning, so it's still
>>> not const in Solaris 11 either.
>>>
>>> Is it worth working around this? It's strictly cosmetic AFAICS.
>>>
>>> The main issue in my mind would be how to determine whether to use
>>> const or not. If all Solaris releases are like this, and can be
>>> expected to stay that way,
>>
>> I think yes. It is defined as X/Open standard says.
>>
>
> Not according to the link you sent earlier. My reading says that
> Solaris has it defined wrong and pg has it right.
If I look there
http://www.opengroup.org/onlinepubs/008329799/chap5.htm#tagcjh_06
in "Call Back Information" section. The structure is defined as
struct pam_conv{ int (*conv) (int, struct pam_message **, struct
pam_response **, void *); void *appdata_ptr; };
I don't see any "const" keyword there.
Zdenek
From | Date | Subject | |
---|---|---|---|
Next Message | Heikki Linnakangas | 2007-07-13 13:40:53 | Re: compiler warnings on the buildfarm |
Previous Message | Kris Jurka | 2007-07-13 13:22:29 | Re: compiler warnings on the buildfarm |