vmod_selector.c was over 1000 lines.

.compile() is still legal, but unnecessary and will be deprecated.

.create_stats() just sets a flag in the PRIV_TASK that stats should
be created by the fini method. If .compile() is called, .create_stats()
is now legal before or after .compile().

Ref:
https://github.com/varnishcache/varnish-cache/commit/681c11998ec56dc4460a30585210f95a7850ea3b

Currently to document the recent breaking changes and refactoring.

The complexity analysis can be highly nuanced. Sets with many strings
and/or long strings can in fact require more time for matching due
to locality effects, which depends highly on usage patterns. Prefix
matches can be longer for large sets since the depth of branching
in the trie becomes larger.

But these are considerations at the level of micro-optimization.
It's the difference between two- or three-digit nanosecond mean
times for the matches.

So it suffices to claim that matching is fast and scales well.

Benchmarks have shown that (collision buckets)/(set size) ranges
from 10%-25%, so there's no need to tweak the hash table size.