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

Trigger database rebuild #464

Closed
csholmq opened this issue Jan 26, 2017 · 6 comments
Closed

Trigger database rebuild #464

csholmq opened this issue Jan 26, 2017 · 6 comments
Labels
Feature Request fixed Check the Milestone for the release in which the fix is or will be available. Language Service

Comments

@csholmq
Copy link
Contributor

csholmq commented Jan 26, 2017

After looking at #415 (comment) I found the new location for the database file. But this leads me to two questions:

  1. What triggers a rebuild of the database? According to Browse database doesn't clear when there's a configuration change #262, it's not triggered by changes in c_cpp_properties.json. This is also the case for adding entries to files.exclude where I have to manually delete the database to have it take effect.
  2. How do I trigger a rebuild manually (besides deleting the database that is)? The only command I found was Unload Language Server which doesn't seem to do much.
@sean-mcmanus
Copy link
Collaborator

The database doesn't rebuild, other than via manually deleting or renaming it. It's a known problem we would like to fix. I've also never seen "Unload Language Server" have any effect.

@csholmq
Copy link
Contributor Author

csholmq commented Aug 12, 2017

It appears that modifying or deleting the c_cpp_properties.json triggers a rebuild. At least for me. Is this by design?

@sean-mcmanus
Copy link
Collaborator

It just re-does what it usually does when the workspace is first opened (searches for new files to add to the database, potentially parsing them), but it doesn't remove stuff from the database yet. Also, ideally it would only trigger a rebuild when a setting that requires a rebuild changes, i.e. changing the intelliSenseMode shouldn't trigger a rebuild. Was there a reason you did not want a "rebuild"?

@csholmq
Copy link
Contributor Author

csholmq commented Aug 12, 2017

Oh, I see. Because removing an include path did break stuff.

@csholmq
Copy link
Contributor Author

csholmq commented Aug 13, 2017

As per your question, I was positively surprised. Sorry if it sounded as if it was something negative.

@bobbrow bobbrow added the fixed Check the Milestone for the release in which the fix is or will be available. label Aug 16, 2017
@bobbrow
Copy link
Member

bobbrow commented Aug 17, 2017

We just released 0.12.3 which added a new command to reset the database.

@bobbrow bobbrow closed this as completed Aug 17, 2017
@github-actions github-actions bot locked and limited conversation to collaborators Oct 18, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Feature Request fixed Check the Milestone for the release in which the fix is or will be available. Language Service
Projects
None yet
Development

No branches or pull requests

3 participants