Re: Transform for pl/perl

Поиск
Список
Период
Сортировка
От Pavel Stehule
Тема Re: Transform for pl/perl
Дата
Msg-id CAFj8pRCYBPFGZG5LHdfY1Ej+a_9sb3ugyDkUaJGD37UjpRpA8Q@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Transform for pl/perl  (Anthony Bykov <a.bykov@postgrespro.ru>)
Ответы Re: Transform for pl/perl  (Anthony Bykov <a.bykov@postgrespro.ru>)
Список pgsql-hackers
Hi

I am looking on this patch. I found few issues:

1. compile warning

I../../src/include  -D_GNU_SOURCE -I/usr/include/libxml2   -I/usr/lib64/perl5/CORE  -c -o jsonb_plperl.o jsonb_plperl.c
jsonb_plperl.c: In function ‘SV_FromJsonbValue’:
jsonb_plperl.c:69:9: warning: ‘result’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  return result;
         ^~~~~~
jsonb_plperl.c: In function ‘SV_FromJsonb’:
jsonb_plperl.c:142:9: warning: ‘result’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  return result;
         ^~~~~~

2. bad comment

/*
 * SV_ToJsonbValue
 *
 * Transform Jsonb into SV --- propably reverse direction
 */


/*
 * HV_ToJsonbValue
 *
 * Transform Jsonb into SV
 */

/*
 * plperl_to_jsonb(SV *in)
 *
 * Transform Jsonb into SV
 */

3. Do we need two identical tests fro PLPerl and PLPerlu? Why?

Regards

Pavel


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

Предыдущее
От: Etsuro Fujita
Дата:
Сообщение: Re: inserts into partitioned table may cause crash
Следующее
От: Curt Tilmes
Дата:
Сообщение: Re: [PATCH] Find additional connection service files inpg_service.conf.d directory