Update cpp property per platform

[seanyen]

On Windows, the cpp property should be using the following settings:

"name": "Windows",
"intelliSenseMode": "msvc-x64",

Also, we should not add /usr/include when it is Windows.
(This fix should be able to address part of #72.)

[kejxu]

let's wait until #74 gets merged into mainline

[kejxu]

I think a better solution would be to leverage api exposed from vscode-cpptools for c++ language configuration; however, that is likely going to take time. Let's stay simple for this change, and I will open a to-do ticket for that.

[Brabalawuka]

On Windows, the cpp property should be using the following settings:

"name": "Windows",
"intelliSenseMode": "msvc-x64",

Also, we should not add /usr/include when it is Windows.
(This fix should be able to address part of #72.)

I am still getting intellisense error after updating the preperties json file

include errors detected. Please update your includePath. Squiggles are disabled for this translation unit (C:\test\src\helloworld\src\listener.cpp). cannot open source file "boost/math/special_functions/round.hpp" (dependency of "ros/ros.h")

My json file is like:

{
"configurations": [
    {
        "browse": {
            "databaseFilename": "",
            "limitSymbolsToIncludedHeaders": true
        },
        "includePath": [
            "C:/opt/ros/melodic/x64\\include",
            "C:/opt/rosdeps/x64\\include",
            "${workspaceFolder}/**"
        ],
        "name": "Windows",
        "intelliSenseMode": "msvc-x64",
        "compilerPath": "C:/Program Files (x86)/Microsoft Visual Studio/2019/Preview/VC/Tools/MSVC/14.22.27812/bin/Hostx64/x64/cl.exe",
        "windowsSdkVersion": "10.0.18362.0",
        "cStandard": "c11",
        "cppStandard": "c++17"
    }
],
"version": 4

}

[kejxu]

I am still getting intellisense error after updating the preperties json file

thanks for sharing! could you try "C:/opt/rosdeps/x64\\include\\**"?

[Brabalawuka]

On Windows, the cpp property should be using the following settings:

"name": "Windows",
"intelliSenseMode": "msvc-x64",

Also, we should not add /usr/include when it is Windows.
(This fix should be able to address part of #72.)

I am still getting intellisense error after updating the preperties json file
include errors detected. Please update your includePath. Squiggles are disabled for this translation unit (C:\test\src\helloworld\src\listener.cpp). cannot open source file "boost/math/special_functions/round.hpp" (dependency of "ros/ros.h")
My json file is like:

{
"configurations": [
    {
        "browse": {
            "databaseFilename": "",
            "limitSymbolsToIncludedHeaders": true
        },
        "includePath": [
            "C:/opt/ros/melodic/x64\\include",
            "C:/opt/rosdeps/x64\\include",
            "${workspaceFolder}/**"
        ],
        "name": "Windows",
        "intelliSenseMode": "msvc-x64",
        "compilerPath": "C:/Program Files (x86)/Microsoft Visual Studio/2019/Preview/VC/Tools/MSVC/14.22.27812/bin/Hostx64/x64/cl.exe",
        "windowsSdkVersion": "10.0.18362.0",
        "cStandard": "c11",
        "cppStandard": "c++17"
    }
],
"version": 4

}

thanks for sharing! could you try "C:/opt/rosdeps/x64\\include\\**"?

Thanks for your help! The previos error has gone but onw it becomes

#include errors detected. Please update your includePath. Squiggles are disabled for this translation unit (C:\test\src\helloworld\src\listener.cpp). cannot open source file "sys/time.h" (dependency of "ros/ros.h")

Does that mean I hav anything that is not installed? I installed C++ envrionment in Visual studio so it should be Ok right?

[kejxu]

Thanks for your help! The previos error has gone but onw it becomes

#include errors detected. Please update your includePath. Squiggles are disabled for this translation unit (C:\test\src\helloworld\src\listener.cpp). cannot open source file "sys/time.h" (dependency of "ros/ros.h")

Does that mean I hav anything that is not installed? I installed C++ envrionment in Visual studio so it should be Ok right?

Thanks for reporting this!

This sounds similar to ros/roscpp_core#108. sys/time.h is a Linux-only library, thus generating this error on Windows. This error could be ignored if IntelliSense works for data types that your code base consumes =)

[Brabalawuka]

Thanks for your help! The previos error has gone but onw it becomes
#include errors detected. Please update your includePath. Squiggles are disabled for this translation unit (C:\test\src\helloworld\src\listener.cpp). cannot open source file "sys/time.h" (dependency of "ros/ros.h")
Does that mean I hav anything that is not installed? I installed C++ envrionment in Visual studio so it should be Ok right?

Thanks for reporting this!

This sounds similar to ros/roscpp_core#108. sys/time.h is a Linux-only library, thus generating this error on Windows. This error could be ignored if IntelliSense works for data types that your code base consumes =)

Thanks a lot! SO I would just ognore it sinces it compiles Ok. !