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.

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

allow intellisense to have a delay under 500ms #13076

Open
TheBearodactyl opened this issue Dec 20, 2024 · 2 comments
Open

allow intellisense to have a delay under 500ms #13076

TheBearodactyl opened this issue Dec 20, 2024 · 2 comments
Labels
enhancement Improvement to an existing feature Language Service

Comments

@TheBearodactyl
Copy link

Feature Request

C_Cpp.intelliSenseUpdateDelay should have a minimum of at most 50ms, 500 is much too restrictive. a default of 500ms makes sense, but not a minimum

@sean-mcmanus
Copy link
Contributor

sean-mcmanus commented Dec 20, 2024

I think VS uses 1 second and has no setting to change it.

The minimum of 500ms is also enforced in the cpptools process.

There is a problem with lowering it too much -- if you're using a TU that takes many seconds to update, then when you type completions work if the characters aren't typed before an update is queued, so although users can enter multiple characters for a completion in < 500 ms, at 50ms, it becomes harder and could make it appear that completions are always delayed behind an IntelliSense update (which I think is a more important scenario than slightly faster error squiggles).

And the speed at which IntelliSense errors would be reported would only be at best 450ms faster and it's usually not that critical for IntelliSense errors to be appear 450ms sooner.

Having a lower setting would enable users to optin to that if they really want, but we were worried about users not understanding the completion problem they could start having.

@sean-mcmanus sean-mcmanus added Language Service enhancement Improvement to an existing feature labels Dec 20, 2024
@TheBearodactyl
Copy link
Author

TheBearodactyl commented Dec 20, 2024

maybe there could be a warning that shows in the logs if it's set under 500 then?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement to an existing feature Language Service
Projects
Status: No status
Development

No branches or pull requests

2 participants