Re: cross-platform pg_basebackup

Поиск
Список
Период
Сортировка
Искать
От
Andrew Dunstan
Тема
Re: cross-platform pg_basebackup
Дата
Msg-id
ffd7d092-1954-cbac-11f3-cd3239a92bfb@dunslane.net
Ответ на
Re: cross-platform pg_basebackup (Robert Haas)
Список
Дерево обсуждения
cross-platform pg_basebackup Robert Haas <robertmhaas@gmail.com>
Re: cross-platform pg_basebackup Tom Lane <tgl@sss.pgh.pa.us>
Re: cross-platform pg_basebackup Robert Haas <robertmhaas@gmail.com>
Re: cross-platform pg_basebackup Tom Lane <tgl@sss.pgh.pa.us>
Re: cross-platform pg_basebackup Robert Haas <robertmhaas@gmail.com>
Re: cross-platform pg_basebackup Andrew Dunstan <andrew@dunslane.net>
Re: cross-platform pg_basebackup davinder singh <davindersingh2692@gmail.com>
Re: cross-platform pg_basebackup Robert Haas <robertmhaas@gmail.com>
Re: cross-platform pg_basebackup Julien Rouhaud <rjuju123@gmail.com>
Re: cross-platform pg_basebackup Robert Haas <robertmhaas@gmail.com> 

On 2022-10-20 Th 14:47, Robert Haas wrote:
> On Thu, Oct 20, 2022 at 1:28 PM Tom Lane  wrote:
>> Robert Haas  writes:
>>> Cool. Here's a patch.
>> LGTM, except I'd be inclined to ensure that all the macros
>> are function-style, ie
>>
>> +#define IS_DIR_SEP(ch) IS_NONWINDOWS_DIR_SEP(ch)
>>
>> not just
>>
>> +#define IS_DIR_SEP IS_NONWINDOWS_DIR_SEP
>>
>> I don't recall the exact rules, but I know that the second style
>> can lead to expanding the macro in more cases, which we likely
>> don't want.  It also seems like better documentation to show
>> the expected arguments.
> OK, thanks. v2 attached.
>


Looks good.


cheers


andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com
В списке pgsql-hackers по дате отправления
Предыдущее
От: Peter Geoghegan
Дата:
Следующее
От: Peter Eisentraut
Дата:
FAQ