(This is mostly a TODO for me, unless someone else would like to follow up on it.)
[ ] Currently, there are places in the documentation that reference the memmap crate, for example here (where the link also is broken). Given that fstnow uses memmap2, these references should be updated. Furthermore, note the RustSec advisory around memmap. Therefore, examples that use memmap should also be updated to use memmap2.
[ ] Highlight whether or not OpBuilder provides the ability to efficiently merge/split existing Fsts versus simply rebuilding the Fst from scratch. Update:OpBuilder [does not "fuse"] existing, lexicographically disjoint FSTs together.
[ ] In the subsection Quirks, provide a link to merge.rs
(This is mostly a TODO for me, unless someone else would like to follow up on it.)
[ ] Currently, there are places in the documentation that reference the
memmapcrate, for example here (where the link also is broken). Given thatfstnow usesmemmap2, these references should be updated. Furthermore, note the RustSec advisory aroundmemmap. Therefore, examples that usememmapshould also be updated to usememmap2.[ ] Highlight whether or not
OpBuilderprovides the ability to efficiently merge/split existingFsts versus simply rebuilding theFstfrom scratch. Update:OpBuilder[does not "fuse"] existing, lexicographically disjoint FSTs together.[ ] In the subsection Quirks, provide a link to
merge.rs