Skip to content
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

Performance - took to long to spell check file - utf_info.hxx #1775

Closed
3 tasks done
Jason3S opened this issue Sep 27, 2021 · 1 comment · Fixed by #1800
Closed
3 tasks done

Performance - took to long to spell check file - utf_info.hxx #1775

Jason3S opened this issue Sep 27, 2021 · 1 comment · Fixed by #1800
Labels
bug Needs Investigating new issue Performance Run-time performance

Comments

@Jason3S
Copy link
Collaborator

Jason3S commented Sep 27, 2021

Info

It took nearly 30 seconds to spell check a single file:
nuspell/utf_info.hxx at master · nuspell/nuspell

Kind of Issue

  • runtime - command-line tools

Which Tool or library

  • cspell -- the command-line spelling tool
  • cspell-lib -- library that does the actual spell checking.

Which Version

Version: 5.10.1
Jason3S added a commit that referenced this issue Sep 28, 2021
@Jason3S
fix: Reduce the cost of regexp exclusions
a252ffe 
Fix #1775
@Jason3S Jason3S mentioned this issue Sep 28, 2021
Merged
Jason3S added a commit that referenced this issue Sep 28, 2021
@Jason3S
fix: Reduce the cost of regexp exclusions (#1800)
4544c25 
* fix: Reduce the cost of regexp exclusions
    Fix #1775
* Create TextRange.regexp.test.ts
* Update spellCheckFile.test.ts
@github-actions
Copy link
Contributor

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Needs Investigating new issue Performance Run-time performance
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Reduce the cost of regexp exclusions streetsidesoftware/cspell
1 participant
@Jason3S