Re: Miscellaneous changes to plperl [PATCH]

Поиск
Список
Период
Сортировка
От Alex Hunsaker
Тема Re: Miscellaneous changes to plperl [PATCH]
Дата
Msg-id 34d269d41001231153k2140af24k17393bdae65780e@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Miscellaneous changes to plperl [PATCH]  ("David E. Wheeler" <david@kineticode.com>)
Ответы Re: Miscellaneous changes to plperl [PATCH]  (Tim Bunce <Tim.Bunce@pobox.com>)
Список pgsql-hackers
On Sat, Jan 23, 2010 at 12:42, David E. Wheeler <david@kineticode.com> wrote:
> 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. 

Haha, yep your right.  I could have sworn I tested it with a function
name with a ' and it broke.  But your obviously right :)


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

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