Replies: 3 comments 1 reply
-
This explains so much weird behavior I was running into, thanks for spotting this! Fixed in Saghen/frizbee@d96d203 but not yet pulled into this repo |
Beta Was this translation helpful? Give feedback.
-
Since I'm good at finding differences, Saghen/frizbee@d96d203#diff-4c62c83a36f3fd726be3053c2c1ec18263870543e5be32267fdc0f342f4589ecR36-R38 doesn't have the |
Beta Was this translation helpful? Give feedback.
-
I believe there is a similar issue with delimiter + capitals. Using the current latest blink.cmp Rust fuzzy matcher: This is with the buffer source for reference, and I'd expect the prefix match to beat the after-delimiter bonus. I believe this is because of this logic that explicitly disallows the prefix to get a capitalization bonus https://github.com/Saghen/frizbee/blob/789d5e1a42172dd334ef795a75814720b9bb148f/src/smith_waterman/reference.rs#L56. As far as I can tell the Lua implementation has no such check - is this intentional? |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
I've noticed that the Lua matcher seem does exact better. I seems that the Rust matcher ignores exact.
Different but related this line of code https://github.com/Saghen/blink.cmp/blob/main/lua/blink/cmp/fuzzy/lua/match.lua#L49 will give a case bonus on a delimiter character. Didn't know if that was intentional.
Beta Was this translation helpful? Give feedback.
All reactions