search

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

Consider to use "<unnamed>" instead of "Default" #9

Closed ReinhardKeil closed 1 year ago

ReinhardKeil commented 1 year ago

In the CMSIS Context Window it seems the term "Default" is used when no build-type or target-type is specified in the csolution file.

As "Default" is a valid build-type or target-type name, this might get confusing. Instead I suggest to use the string "\<unnamed>"

See this picture image

JoeTheGuitarist commented 1 year ago

This suggestion makes sense, but I think the word "unnamed" could also be confusing here as the real meaning we want to convey is that the field is not specified, so perhaps we could use "undefined", which was suggested by @Elise-Kaminski

ReinhardKeil commented 1 year ago

Both \<undefined\> or \<unnamed\> would work for me. It should just be clear that this is not a standard build-type name.

mcgordonite commented 1 year ago

This was resolved in version 0.11.0.