RFC: remove configuration knobs about aho-corasick HOT 6 CLOSED

Sure, just removing it in the next breaking release also works.

I just thought that deprecating would be the most reliable way to get potential objections. FWIW, I cannot find any code on GitHub setting premultiply(false). For bytes_classes(false), I only found one use in tokei.

from aho-corasick.

@vks Yeah, there are some cases in which disabling byte classes can make the resulting search faster, but I've only ever seen slight improvements. (I've also seen regressions due to the increased size in the automaton.)

from aho-corasick.

In the next semver incompatible release, I am also planning to remove the ability to construct automatons using a smaller state ID representation. Instead, it will be fixed to u32 with no way to change it. See the corresponding comment on the regex-automata issue for more details: BurntSushi/regex-automata#7 (comment)

from aho-corasick.

I'm not sure how many users of aho-corasick will read this here. Maybe just go ahead and deprecate it? If there are a lot of objections, you can always revert the deprecation.

from aho-corasick.

Yeah I should have updated this. I'm not holding myself up here as a result of the lack of feedback. My plan is to just to go forward with this. Just haven't got a chance to do it. It's not super high priority.

from aho-corasick.

Deprecating them sounds smart though. Might be a good forcing function.

from aho-corasick.