On Sun, May 2, 2021 at 9:37 PM Tom Lane <t...@sss.pgh.pa.us> wrote: > Alexander Korotkov <aekorot...@gmail.com> writes: > > On Sun, May 2, 2021 at 9:04 PM Tom Lane <t...@sss.pgh.pa.us> wrote: > >> - state.in_quotes = false; > >> > >> This change seems wrong/unsafe too. > > > It seems OK, because this patch removes in_quotes field altogether. > > Oh, sorry, I misread the patch --- I thought that earlier hunk > was removing a local variable. Agreed, if you can do without this > state field altogether, that's fine.
OK, thank you for review! ------ Regards, Alexander Korotkov
