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 to set default includePath and browse.path in settings.json #1379

Closed
rongjiecomputer opened this issue Dec 28, 2017 · 1 comment
Closed

Comments

@rongjiecomputer
Copy link

Problem:

Currently includePath and browse.path can only be set in project-specific .vscode/c_cpp_properties.json and this leads to a lot of problems:

  1. Have to copy the c_cpp_properties.json in every project.
  2. Single file cannot have c_cpp_properties.json, so no intelliSense at all.
  3. A git repo cannot share c_cpp_properties.json because some users have very different includePath and browse.path due to non-default header/library installation.

Feature request:

Allow to set default includePath and browse.path in VSCode settings.json.

If includePath and browse.path are set in both settings.json and project-specific c_cpp_properties.json, includePath and browse.path in c_cpp_properties.json should have higher precedence, just like -I and /I flags have higher precedence than -isysroot flag and INCLUDE environment variable.

It might also make sense to have a separate symbol database for includePath and browse.path from settings.json to avoid re-parsing.

@sean-mcmanus
Copy link
Collaborator

Yes, we were already planning to do this. ETA maybe February (not sure yet). I think we're tracking this via #368 .

@github-actions github-actions bot locked and limited conversation to collaborators Oct 16, 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

2 participants