Skip to content

Commit

Permalink
Update VSCode settings and CMakeLists for improved linting and includ…
Browse files Browse the repository at this point in the history
…e paths
  • Loading branch information
timetravelCat committed Jan 18, 2025
1 parent 7214747 commit 92e164f
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .vscode/settings.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
"c-cpp-flylint.undefines": [ ],
"c-cpp-flylint.includePaths": [
"${workspaceFolder}/include",
// "/usr/local/cuda/include"
],
"c-cpp-flylint.excludeFromWorkspacePaths": [
"/usr/include/**",
Expand All @@ -28,6 +29,7 @@
"c-cpp-flylint.cppcheck.suppressions": [
// "missingIncludeSystem",
// "constParameter",
"noExplicitConstructor",
"unusedFunction",
// "preprocessorErrorDirective",
// "constParameterPointer",
Expand Down Expand Up @@ -90,4 +92,5 @@
},

"C_Cpp.default.configurationProvider": "ms-vscode.cmake-tools",
// "C_Cpp.default.compileCommands": "${workspaceFolder}/build/compile_commands.json",
}
3 changes: 3 additions & 0 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,9 @@ add_library(${PROJECT_NAME}::${PROJECT_NAME} ALIAS ${PROJECT_NAME})
target_include_directories(${PROJECT_NAME} PUBLIC
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
$<INSTALL_INTERFACE:include>)

# target_include_directories(${PROJECT_NAME} SYSTEM PUBLIC
# ${CMAKE_CUDA_TOOLKIT_INCLUDE_DIRECTORIES})
target_compile_options(${PROJECT_NAME} PUBLIC
-Wall # Enable all common warnings.
-Wextra # Enable additional warnings not covered by -Wall.
Expand Down

0 comments on commit 92e164f

Please sign in to comment.