New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Exact words #22

Merged

subins2000 merged 5 commits into master from exact-words

Feb 5, 2022

Member

subins2000 commented Feb 5, 2022 •

edited

Fixes #21

Adds a new higher prioirty ExactWords result along with ExactMatches. How it differs:
- ExactWords - Exactly found words in dictionary if there is any.
- ExactMatches - Exactly starting word matches in dictionary if there is any. Not applicable for patterns dictionary.
Avoid item variable in range loops to save memory copying wherever possible. for i, item := range to for i := range

subins2000 added 5 commits

February 5, 2022 17:05


          Differentiate exact matches and exactWords result

cddc44b


          Move ExactWords logic to channelGetFromDictionary

6df5be0


          Update testcases for ExactWords, add test for ExactMatches

798be54


          Improved comments

716ae50


          Avoid item variable in range wherever possible

b9486a1

subins2000 merged commit 4d1021d into master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment