Thread: Re: [HACKERS] merge psql ef/ev sf/sv handling functions

Re: [HACKERS] merge psql ef/ev sf/sv handling functions

From
Masahiko Sawada
Date:
On Sat, Apr 1, 2017 at 3:04 AM, Fabien COELHO <coelho@cri.ensmp.fr> wrote:
>
> Hello,
>
> While reviewing Corey's \if patch, I complained that there was some amount
> of copy-paste in "psql/command.c".
>
> Here is an attempt at merging some functions which removes 160 lines of
> code.
>

Thank you for the patch. Is this an item for PG11?

Regards,

--
Masahiko Sawada
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center



Re: [HACKERS] merge psql ef/ev sf/sv handling functions

From
Fabien COELHO
Date:
>> While reviewing Corey's \if patch, I complained that there was some amount
>> of copy-paste in "psql/command.c".
>>
>> Here is an attempt at merging some functions which removes 160 lines of
>> code.
>
> Thank you for the patch. Is this an item for PG11?

Yep.

-- 
Fabien.



Re: [HACKERS] merge psql ef/ev sf/sv handling functions

From
Fabien COELHO
Date:
>> While reviewing Corey's \if patch, I complained that there was some amount
>> of copy-paste in "psql/command.c".
>>
>> Here is an attempt at merging some functions which removes 160 lines of
>> code.
>
> Thank you for the patch. Is this an item for PG11?

Yes, as it is submitted to CF 2017-09.

-- 
Fabien.



Re: [HACKERS] merge psql ef/ev sf/sv handling functions

From
Masahiko Sawada
Date:
On Wed, Jul 19, 2017 at 2:41 PM, Fabien COELHO <coelho@cri.ensmp.fr> wrote:
>
>>> While reviewing Corey's \if patch, I complained that there was some
>>> amount
>>> of copy-paste in "psql/command.c".
>>>
>>> Here is an attempt at merging some functions which removes 160 lines of
>>> code.
>>
>>
>> Thank you for the patch. Is this an item for PG11?
>
>
> Yes, as it is submitted to CF 2017-09.
>

Thank!
It is already registered to next CF. I missed it, sorry.

Regards,

--
Masahiko Sawada
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center