Skip to content

update vscode config so that headers can be found#231

Merged
haochengxia merged 2 commits intodevelopfrom
config
Jun 25, 2025
Merged

update vscode config so that headers can be found#231
haochengxia merged 2 commits intodevelopfrom
config

Conversation

@1a1a11a
Copy link
Copy Markdown
Owner

@1a1a11a 1a1a11a commented Jun 25, 2025

update vscode config so that headers can be found

@1a1a11a 1a1a11a requested a review from Copilot June 25, 2025 15:52
@1a1a11a 1a1a11a requested a review from haochengxia as a code owner June 25, 2025 15:52
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request updates the VSCode configuration to improve header file detection.

  • Adds file association settings in .vscode/settings.json to map various header names to specific language modes.
  • Introduces platform-specific configurations in .vscode/c_cpp_properties.json for Linux, macOS, and Windows.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.vscode/settings.json Adds file association and various editor, cmake, and git settings.
.vscode/c_cpp_properties.json Introduces compiler configuration with include paths and build settings.
Comments suppressed due to low confidence (5)

.vscode/settings.json:4

  • The header 'cmath' is a standard C++ header and is typically associated with 'cpp' rather than 'c'. Consider updating its language association to 'cpp' for correct IntelliSense behavior.
        "cmath": "c",

.vscode/settings.json:5

  • The header 'complex' is a C++ header and should likely be associated with 'cpp' instead of 'c'. Consider changing it to 'cpp' to ensure correct syntax processing.
        "complex": "c",

.vscode/settings.json:19

  • The header 'string' is part of the C++ standard library and associating it with 'c' may lead to incorrect language features. Consider updating this association to 'cpp'.
        "string": "c",

.vscode/settings.json:20

  • The header 'istream' is intended for C++ code. Changing its association from 'c' to 'cpp' could improve language support in VSCode.
        "istream": "c",

.vscode/settings.json:21

  • As 'streambuf' is a C++ component, associating it with 'c' may result in improper handling by VSCode. It is recommended to update its association to 'cpp'.
        "streambuf": "c",

Comment thread .vscode/settings.json
"CMAKE_EXPORT_COMPILE_COMMANDS": "ON"
},
"cmake.generator": "Unix Makefiles",
"cmake.parallelJobs": 0,
Copy link

Copilot AI Jun 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Setting 'cmake.parallelJobs' to 0 disables parallel building, which might affect build performance. If parallel builds are desired, consider setting this to an appropriate positive integer.

Suggested change
"cmake.parallelJobs": 0,
"cmake.parallelJobs": 4,

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Collaborator

@haochengxia haochengxia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@haochengxia haochengxia merged commit 9d15b0f into develop Jun 25, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants