Re: FindDefinedSymbol() is subtly broken

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: FindDefinedSymbol() is subtly broken
Дата
Msg-id 3761.1526760348@sss.pgh.pa.us
обсуждение исходный текст
Ответ на FindDefinedSymbol() is subtly broken  (John Naylor <jcnaylor@gmail.com>)
Список pgsql-hackers
John Naylor <jcnaylor@gmail.com> writes:
> Long story short: It expects an array of paths, but will die if it
> can't find the symbol in the first path. Arguably it's a bug, but
> since the original use case for multiple paths is long gone anyway, it
> might never occur in practice. The attached patch changes it to expect
> a single path.

> Also attached: a few minor build script cleanups I've noticed along the way:
> -more accurate error message
> -s/print sprintf/printf/
> -a recent perltidy change made a couple multi-line strings look odd,
> so use heredocs (which other scripts use in this case anyway)
> -make generated file footers match project style
> -remove a duplicate comment

This all seemed like minor cleanup from v11 changes, so I went
ahead and pushed it rather than waiting for v12.

            regards, tom lane


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

Предыдущее
От: Tom Lane
Дата:
Сообщение: printf format selection vs. reality
Следующее
От: Tom Lane
Дата:
Сообщение: printf("%lf",...) isn't actually portable