Thanks for the review.

This is a potential bug fix.  However, match_mb_charset function isn't
used in grep and gawk, as DFA treats MBCSET as BACKREF by following code
if `backref' is provided.  Therefore the fix never induces any semantic
change in grep and gawk.

              if (d->states[s].has_mbcset && backref)
                {
                  *backref = 1;
                  goto done;
                }

Essentially, the function is able to be removed.  However, if we regard
DFA as a library,  we should keep it.




Reply via email to