ARM-software / vscode-cmsis-csolution

Extension support for VS Code CMSIS Project Extension
https://marketplace.visualstudio.com/items?itemName=Arm.cmsis-csolution
Other
23 stars 5 forks source link

C/C++ language standard settings are not properly converted from Keil MDK project #15

Closed Overlap0190 closed 10 months ago

Overlap0190 commented 1 year ago

Type: Bug Report

Describe the bug

My settings in Keil MDK: grafik

After the conversion the *.csolution.yaml looks like this:

misc:
  - C*:
      - -fno-rtti
      - -funsigned-char
      - -fshort-enums
      - -fshort-wchar
      - -ffunction-sections
      - -std=c99
      ...

The C standard is applied to C/C++ files. The C++ standard is completely missing.

To Reproduce Steps to reproduce the behavior:

  1. Clone this repository https://github.com/RobertRostohar/Demo_EW/
  2. Open the EdgeImpulse folder in VS Code
  3. Convert the example.uvprojx to csolution
  4. Open the generated example.csolution.yaml and the misc segment should look like mentioned above.

Expected behavior I would expect something like this in the *.csolution.yaml:

misc:
  - C:
      - -std=c99
    CPP:
      - -std=c++14
    C*:
      - -fno-rtti
      - -funsigned-char
      - -fshort-enums
      - -fshort-wchar
      - -ffunction-sections
      ...
JoeTheGuitarist commented 1 year ago

@ducnguyenarm FYI

Overlap0190 commented 10 months ago

I think this issue is fixed in version 1.6.0. But I didn't try for some time so it could already be working in earlier versions.

mcgordonite commented 10 months ago

Closing this: C++ support was added to the µVision converter in version 1.6.0.