On Sun, May 2, 2021 at 9:37 PM Tom Lane <[email protected]> wrote: > Alexander Korotkov <[email protected]> writes: > > On Sun, May 2, 2021 at 9:04 PM Tom Lane <[email protected]> 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
