Re: pg_tablespace_databases

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: pg_tablespace_databases
Дата
Msg-id 28599.1089119964@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: pg_tablespace_databases  (Bruce Momjian <pgman@candle.pha.pa.us>)
Ответы Re: pg_tablespace_databases  (Bruce Momjian <pgman@candle.pha.pa.us>)
Список pgsql-patches
Bruce Momjian <pgman@candle.pha.pa.us> writes:
> Tom Lane wrote:
>> [ shrug... ]  The name is not going to change again.  I have never cared
>> for the practice of writing strlen("foo") as if it were a compile-time
>> constant.

> I think with gcc strlen("foo") is a compile-time constant.

Portability is exactly the root of the problem.  If you are in the habit
of doing this then you get led into unportable behaviors like
    char    localarray[strlen(foo) + 1];
which no compiler except gcc will take.  (We just had to fix exactly
that mistake in someone's patch within the last week or two.)

> What do you prefer?

I use "3" ;-).  As long as the size calculation and the filling of the
string are immediately adjacent, the purpose of the code is clear
enough.

            regards, tom lane

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

Предыдущее
От: Gavin Sherry
Дата:
Сообщение: Re: pg_tablespace_databases
Следующее
От: Bruce Momjian
Дата:
Сообщение: Re: [PATCH] s_lock support for win32