Re: pgsql: Beautify initialization of JsonValueList and JsonLikeRegexContex

From: Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Alexander Korotkov <akorotkov(at)postgresql(dot)org>, pgsql-committers <pgsql-committers(at)lists(dot)postgresql(dot)org>
Subject: Re: pgsql: Beautify initialization of JsonValueList and JsonLikeRegexContex
Date: 2019-03-18 06:56:29
Message-ID: CAPpHfdtOQ9kqr9yJp+wo+-uNQQZx_D9K6YP2Wxpe+d4FWGwbJA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Mon, Mar 18, 2019 at 7:12 AM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> On Sun, Mar 17, 2019 at 03:09:07PM +0300, Alexander Korotkov wrote:
> > Anyway, I thought about this as about just beautification. If it's
> > bad idea, let's revert this.
>
> I agree with Peter's point here that having {0} as initial state is
> cleaner instead of adding an extra level of abstraction.

OK! So, reverted.

------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2019-03-18 08:14:52 pgsql: Remove unused macro
Previous Message Alexander Korotkov 2019-03-18 06:55:21 pgsql: Revert 4178d8b91c