-
Notifications
You must be signed in to change notification settings - Fork 3
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.
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
Optimize loading time for "welspell_config" #16
Comments
One of the options would be to use See awalker89/openxlsx#440 for details. |
What do you think about c3cfb3e? I wanted to do a PR, but my fingers where faster than my brain. |
The result of Note. If a method, which permanently saves the environment variable, is implemented, there should be a way to update the list of the languages if it changes. E.g., on the installation of the language tool. A new button could also be created for this purpose. |
When I try to open "welspell_config" window, on my PC it takes several seconds to open. Sometimes I start doubting if I even pressed the hot-key combination to open the tool.
I did some quick profiling. One of the bottlenecks on LanguageToolR side is function "lato_list_languages"
Next,
lato_test_setup()
is called several times.Is it possible to optimize these processes?
Some ideas would be:
The text was updated successfully, but these errors were encountered: