Re: Miscellaneous changes to plperl [PATCH]

Поиск
Список
Период
Сортировка
От David E. Wheeler
Тема Re: Miscellaneous changes to plperl [PATCH]
Дата
Msg-id 63E5EAC5-9D15-4976-99E6-9E2E44AC9DC9@kineticode.com
обсуждение исходный текст
Ответ на Re: Miscellaneous changes to plperl [PATCH]  (Alex Hunsaker <badalex@gmail.com>)
Ответы Re: Miscellaneous changes to plperl [PATCH]  (Alex Hunsaker <badalex@gmail.com>)
Re: Miscellaneous changes to plperl [PATCH]  (Tim Bunce <Tim.Bunce@pobox.com>)
Список pgsql-hackers
On Jan 23, 2010, at 11:20 AM, Alex Hunsaker wrote:

> Well no, i suppose we could fix that via:
> $name =~ s/[:|']/_/g;
>
> Im betting that was the intent.

Doubtful. In Perl, the package separator is either `::` or `'` (for hysterical reasons). So the original code was
replacingany package separator with a single underscore. Your regex would change This::Module to This__Module, which
I'mcertain was not the intent. 

Best,

David

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

Предыдущее
От: Greg Smith
Дата:
Сообщение: Re: commit fests
Следующее
От: Andrew Dunstan
Дата:
Сообщение: Re: 8.5 vs. 9.0, Postgres vs. PostgreSQL