| From: | Phil Sorber <phil(at)omniti(dot)com> |
|---|---|
| To: | Magnus Hagander <magnus(at)hagander(dot)net> |
| Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [PATCH] Add PQconninfoParseParams and PQconninfodefaultsMerge to libpq |
| Date: | 2013-02-03 07:23:05 |
| Message-ID: | CADAkt-jOHrWBR1YV3GoPNXJgGOyDC7e6_2k9Ho=9hgo3MWnCLg@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Sun, Feb 3, 2013 at 1:37 AM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>
> On Feb 3, 2013 4:16 AM, "Phil Sorber" <phil(at)omniti(dot)com> wrote:
>>
>> This patch came up from discussion about pg_isready.
>>
>> PQconninfoParseParams is similar to PQconninfoParse but takes two
>> arrays like PQconnectdbParams. It essentially exposes
>> conninfo_array_parse().
>>
>> PQconninfodefaultsMerge essentially exposes conninfo_add_defaults().
>> It allows you to pass a PQconninfoOption struct and it adds defaults
>> for all NULL values.
>>
>> There are no docs yet. I assumed I would let bikeshedding ensue, and
>> also debate on whether we even want these first.
>
> I think you forgot to attach the patch.
>
> /Magnus
/me hangs head in shame :-(
Here is is...
| Attachment | Content-Type | Size |
|---|---|---|
| libpq_params_parse_merge.diff | application/octet-stream | 4.7 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dean Rasheed | 2013-02-03 15:04:48 | Re: missing rename support |
| Previous Message | Amit kapila | 2013-02-03 07:19:14 | Re: Suggestion: Issue warning when calling SET TRANSACTION outside transaction block |