-
Notifications
You must be signed in to change notification settings - Fork 335
[VSC-1630 | 1527] Feature clang project configuration #1489
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
Conversation
|
Download the artifacts for this pull request: |
|
Thank you for your patch. |
radurentea
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
d66fd9d to
62fd9ac
Compare
62fd9ac to
a10249e
Compare
Fabricio-ESP
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Confirmed working.
Re-adding IDF to the extension or installing a new version will add esp-clang tool to the toolset.
Description
Add
ESP-IDF: Configure project for ESP-Clangcommand to configure the current workspace project for LLVM Clang extension language server, to allow the user to configure esp-idf project syntax highlighting, go to declaration, etc.Fixes #1352
Fixes #1484
Type of change
Steps to test this pull request
Provide a list of steps to test changes in this PR and required output
ESP-IDF: Configure project for ESP-Clangcommand.idf.customExtraVarsshould have IDF_TOOLCHAIN andclang.pathandclang.argumentsshould be added to project's .vscode/settings.json. Add.clangdfile to project.ESP-IDF: Configure ESP-IDF extensionand the LLVM Clang extension provide C/C++ language support after running step 1.User can use the esp-clang toolchain and the LLVM Clang extension to provide C/C++ language support (syntax highlight, code navigation, etc.)
The
clang.pathandclang.argumentsshould be added to project's .vscode/settings.jsonTest Configuration:
Checklist