Re: Potentially misleading name of libpq pass phrase hook

Поиск
Список
Период
Сортировка
От Daniel Gustafsson
Тема Re: Potentially misleading name of libpq pass phrase hook
Дата
Msg-id 4885CB6A-ECA1-4752-A055-E32FBC62FC1C@yesql.se
обсуждение исходный текст
Ответ на Re: Potentially misleading name of libpq pass phrase hook  (Michael Paquier <michael@paquier.xyz>)
Ответы Re: Potentially misleading name of libpq pass phrase hook  ("Jonathan S. Katz" <jkatz@postgresql.org>)
Re: Potentially misleading name of libpq pass phrase hook  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
> On 16 May 2020, at 03:56, Michael Paquier <michael@paquier.xyz> wrote:
>
> On Fri, May 15, 2020 at 09:21:52PM -0400, Jonathan S. Katz wrote:
>> +1 on all of the above.
>>
>> I noticed this has been added to Open Items; I added a note that the
>> plan is to fix before the Beta 1 wrap.
>
> +1.  Thanks.
>
> Agreed.  PQsslKeyPassHook_<library>_type sounds fine to me as
> convention.  Wouldn't we want to also rename PQsetSSLKeyPassHook and
> PQgetSSLKeyPassHook, appending an "_OpenSSL" to both?

Yes, I think we should.  The attached performs the rename of the hook functions
and the type, and also fixes an off-by-one-'=' in a header comment which my OCD
couldn't unsee.

cheers ./daniel


Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: pgindent && weirdness
Следующее
От: Christopher Baines
Дата:
Сообщение: Re: [PATCH] Fix pg_dump --no-tablespaces for the custom format