Re: BUG #6216: Calling PQconnectdbParams from C++ with a char**

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Craig Ringer <ringerc(at)ringerc(dot)id(dot)au>
Cc: Lionel Elie Mamane <lionel(at)mamane(dot)lu>, pgsql-bugs(at)postgresql(dot)org
Subject: Re: BUG #6216: Calling PQconnectdbParams from C++ with a char**
Date: 2011-09-20 23:43:25
Message-ID: 8393.1316562205@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

Craig Ringer <ringerc(at)ringerc(dot)id(dot)au> writes:
> As for wording: my *personal* preference is "const char * const" but I
> don't know what the opinions of those who work with the code day-to-day are.

+1. Isn't the other ordering deprecated by recent C standards?
(Or maybe I'm just thinking of where you're supposed to put "static",
but in any case "char const *" looks pretty weird to me.) Also,
the existing usages in libpq-fe.h look like that, and there's no good
reason for these to be randomly different.

regards, tom lane

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message pratikchirania 2011-09-21 07:16:58 Re: Timezone issues with Postrres
Previous Message Craig Ringer 2011-09-20 23:29:23 Re: BUG #6216: Calling PQconnectdbParams from C++ with a char**