Vscode-cpptools: Add some intelliSenseUpdateDelay setting

Created on 15 Sep 2020  路  9Comments  路  Source: microsoft/vscode-cpptools

VS uses 1 second. We currently use 3 seconds, but our next release may change to 2 seconds.

Based on feedback at https://github.com/microsoft/vscode-cpptools/issues/6129 .

Language Service enhancement fixed (release pending) more votes needed quick fix

Most helpful comment

@sean-mcmanus as another point of reference, clion uses 500ms as the default. The 1s value from VS is probably a reasonable upper bound for the default value.

Personally, I set the delay down to 250. I'm not sure squiggles as you type is an issue people care about versus feeling snappy when you want it to.

The most important thing is having a setting though.

All 9 comments

@sean-mcmanus as another point of reference, clion uses 500ms as the default. The 1s value from VS is probably a reasonable upper bound for the default value.

Personally, I set the delay down to 250. I'm not sure squiggles as you type is an issue people care about versus feeling snappy when you want it to.

The most important thing is having a setting though.

How long will it take to start fix?

The 3 seconds delay is too long anyhow. I really can't stand the slow respond. Especially when I fix a syntax error, I can't see the error squiggle disappear immediately.

We made the change from 3 to 2 seconds, but if anyone wants to make a PR on the TypeScript side for a new setting, we might be able to make the changes to the cpptools side to read that setting. See https://github.com/microsoft/vscode-cpptools/pull/5730 as an example.

In fact, I have made some vscode extensions, but not familiar with lsp.

I don't think you wouldn't need to change LSP stuff in order to add an intelliSenseUpdateDelay setting...the setting value just needs to be sent along with the other settings we send.

I don't think you wouldn't need to change LSP stuff in order to add an intelliSenseUpdateDelay setting...the setting value just needs to be sent along with the other settings we send.

I will make a pr about this today or tomorrow.

Was this page helpful?
0 / 5 - 0 ratings