fuzzy-matcher

fuzzy-matcher copied to clipboard

Unicode Matching Indices are potentially incorrect

1

```rust fn main() { let matcher = SkimMatcherV2::default(); let text = " üäö "; // [(0, ' '), (1, 'ü'), (3, 'ä'), (5, 'ö'), (7, ' ')] println!("{:?}", text.char_indices().collect::()); println!("{:?}",...

MinusGix

Refactor/clippy warnings

1

mainrs

[skim v2] avoid panic when cache is disabled

1

when i set `.use_cache(false)` i got a panic: ``` thread 'my-thread' panicked at /HOME/.cargo/registry/src/index.crates.io-6f17d22bba15001f/fuzzy-matcher-0.3.7/src/skim.rs:932:48: already borrowed: BorrowMutError note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace ``` i think...

cdmistman

Hey there, First time using fuzzy-matcher and I noticed that `ScoreType` isn't `pub`. This is a bit of a pain if you want to store the scores anywhere, as it...

Grimeh

Public structs should implement Debug

1

Can you please derive Debug for public structs like SkimMatcher, SkimMatcherV2, and related structs. Thanks.

markus-bauer

Missing documentation of SkimMatcherV2's methods.

1

It is not easy to see what is: * element limit * How the cache is handled * What does "debug" do * What does `with_pos` mean in `fuzzy`. `SkimScoreConfig`...

vi

The main problem with the old code was that all of the cache RefCells were held for too long because of the huge scope of the `fuzzy` function. The problem...

RobWalt