On Fri, Feb 26, 2016 at 10:53 AM, Kyotaro HORIGUCHI <horiguchi.kyot...@lab.ntt.co.jp> wrote: > At Fri, 26 Feb 2016 10:38:22 +0900 (Tokyo Standard Time), Kyotaro HORIGUCHI > <horiguchi.kyot...@lab.ntt.co.jp> wrote in > <20160226.103822.12680005.horiguchi.kyot...@lab.ntt.co.jp> >> Hello, Thanks for the new patch. >> >> >> At Fri, 26 Feb 2016 08:52:54 +0900, Masahiko Sawada <sawada.m...@gmail.com> >> wrote in <CAD21AoAZKFVu8-MVhkJ3ywAiJmb=P-HSbJTGi=gk1la73kj...@mail.gmail.com> >> > Previous patch could not parse one character standby name correctly. >> > Attached latest patch. >> >> I haven't looked it in detail but it won't work as you >> expected. flex compains as the following for v12 patch. >> >> syncgroup_scanner.l:80: warning, rule cannot be matched >> syncgroup_scanner.l:84: warning, rule cannot be matched > > Making it independent from postgres body then compile it with > -DYYDEBUG and set yydebug = 1 would give you valuable information > and make testing of the parser far easier.
Thank you for your suggestion. Attached latest version patch. The changes from previous version are, - Fix parser, lexer bugs. - Add regression test patch based on patch Suraji submitted. Please review it. Regards, -- Masahiko Sawada
Description: Binary data
-- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers