[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [bug-gawk] [PATCH] gawk: use dfa matcher for regex with anchor
From: |
arnold |
Subject: |
Re: [bug-gawk] [PATCH] gawk: use dfa matcher for regex with anchor |
Date: |
Wed, 21 Sep 2016 02:15:26 -0600 |
User-agent: |
Heirloom mailx 12.4 7/29/08 |
Hi. Thanks for this patch. I've applied it and pushed to the repo.
For future reference, when adding tests, besides including the files in
EXTRA_DIST in Makefile.am, you need to list the test's name in one of
the macros that defines the categories of tests. I took care of that for
you for this test. :-)
Thanks for including a test case with your patch; that really helps improve
the test suite and I appreciate it. I did verify that without the
DFA_ANCHOR flag the test doesn't pass.
And, thanks once more for caring about gawk in your work on dfa; I really
appreciate it.
Arnold
Norihiro Tanaka <address@hidden> wrote:
> Hi,
>
> Recently, dfa module started to support that newline_anchor option is
> set to false.
>
> gawk could not use dfa fully up to now, as gawk sets newline_anchor to
> false in many cases.
>
> This change is that gawk uses dfa even if newline_anchor option is false.
> No longer gawk check whether a pattern has anchor or not in pattern.
>
> Norihiro