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

Non-recursive includePath does not work #546

Closed
janjongboom opened this issue Mar 10, 2017 · 6 comments
Closed

Non-recursive includePath does not work #546

janjongboom opened this issue Mar 10, 2017 · 6 comments

Comments

@janjongboom
Copy link

According to #475 you can define includePaths to be non-recursive by adding /* at the end of the folder name. This does not work for me on VsCode 1.10.2 with C/C++ plugin 0.10.3.

My vscode configuration is here: https://gist.github.com/janjongboom/13e96a66b5b167597c70070b15c71696

And when I type D5 in my editor it shows a symbol from a file which is not in this list...

screen shot 2017-03-10 at 09 37 25

/cc @bobbrow @sg-

@bobbrow
Copy link
Member

bobbrow commented Mar 10, 2017

Did you modify the includePath for a folder that was already open in VS Code? We don't currently remove symbols for directories after they've already been parsed - even if you change the includePath. You might need to delete the DB and have the extension rescan your folder using the new includePath. Sorry for the annoyance. We're working on making this better.

Based on your gist, it looks like you are developing on Mac. So you should be able to find the DB at this location:

~/Library/Application Support/Code/User/workspaceStorage/{hash}/ms-vscode.cpptools/.browse.VC.db

@sean-mcmanus
Copy link
Contributor

Unfortunately, the "*" is not working on Linux/Mac -- it works on Windows. The bug should be easy to fix for the next release.

@bobbrow bobbrow added the bug label Mar 10, 2017
@janjongboom
Copy link
Author

@sean-mcmanus @bobbrow I have the same issue on Windows... Also get symbols that are not in my includePaths even though all my paths are postfixed with /*. I have removed my DB... Any clue?

@sean-mcmanus
Copy link
Contributor

That's strange. I'm not able to repro the issue on Windows. There are additional scenarios where files/symbols will get added to the database, such as when a file is opened or when files are created in the workspace. Also, when you remove paths from the includePath, we don't remove the files/symbols from the database, which we plan to fix soon. Can you give an example of an include path and file/symbol that is reproducing the error, such as from the MSVC or Windows SDKs?

@janjongboom
Copy link
Author

@sean-mcmanus I was on an old version of C/C++ extension on my Windows installation that didn't have the addWorkspaceRootToIncludePath setting. Now fixed (on Windows, not on macOS).

@bobbrow
Copy link
Member

bobbrow commented Jul 3, 2017

This issue should have been fixed in 0.11.1. Please let us know if you are still seeing this on the latest version of the extension. Thanks!

@bobbrow bobbrow closed this as completed Jul 3, 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.
Projects
None yet
Development

No branches or pull requests

4 participants