Jim Meyering <[email protected]> wrote:

> Hi Paul,
>
> In case your bug fix looks safe/small, and is ready, ...
>
> I'm hoping to release 2.18 today, with the attached commits.
> Changes since yesterday: comment/log tweaks, and I've hoisted the using_utf8
> test in trivial_case_ignore to precede the two memchr tests.

Hi Jim.

Why copy the using_utf8() routine out of dfa.c?  Why not just link
to it instead?  If it's static, make it extern... That way if the
logic ever changes then it only has to be changed in one place.

Just a thought. :-)

Arnold



Reply via email to