Re[2]: bool_plperl transform

Поиск
Список
Период
Сортировка
От Ivan Panchenko
Тема Re[2]: bool_plperl transform
Дата
Msg-id 1583105446.157155716@f330.i.mail.ru
обсуждение исходный текст
Ответ на Re: bool_plperl transform  (ilmari@ilmari.org (Dagfinn Ilmari Mannsåker))
Список pgsql-hackers


 
Понедельник, 2 марта 2020, 1:09 +03:00 от ilmari@ilmari.org:
 
Wao <wao@mail.ru> writes:
 
> +Datum
> +bool_to_plperl(PG_FUNCTION_ARGS)
> +{
> + dTHX;
> + bool in = PG_GETARG_BOOL(0);
> + SV *sv = newSVnv(SvNV(in ? &PL_sv_yes : &PL_sv_no));
> + return PointerGetDatum(sv);
> +}

Why is this only copying the floating point part of the built-in
booleans before returning them? I think this should just return
&PL_sv_yes or &PL_sv_no directly, like boolean expressions in Perl do,
and like what happens for NULL (&PL_sv_undef).
Thanks, I will fix this in the next version of the patch.
 
Regards,
Ivan

- ilmari
--
"A disappointingly low fraction of the human race is,
 at any given time, on fire." - Stig Sandbeck Mathisen

 
 
 
 
 

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

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Remove win32ver.rc from version_stamp.pl
Следующее
От: Rémi Lapeyre
Дата:
Сообщение: Re: [PATCH v1] Allow COPY "text" to output a header and add headermatching mode to COPY FROM