EasyClangComplete icon indicating copy to clipboard operation
EasyClangComplete copied to clipboard

Add support for replacement of vscode variables

Open seanmlyons22 opened this issue 5 years ago • 3 comments

Convert Variables such as ${workspaceFolder} so they can be understood by LLVM

Close #741

seanmlyons22 avatar Apr 24 '21 16:04 seanmlyons22

@seanmlyons22 thanks for the contribution. I might be missing a point a bit though. This is already done by using other variables. For example ${project_base_path} will expand to a full path to the project root. How would this variable be different?

niosus avatar Apr 26 '21 17:04 niosus

@niosus, sorry this got lost in my inbox. This would be exactly the same as the other variables. I will update this to be replaced in the same manner.

Edit: Looked at project_base_path, the difference between that variable and ${workspaceFolder} is that the latter is a VSCode variable, so it should only be replaced for c_cpp_properties.json, not for the others.

seanmlyons22 avatar May 18 '21 12:05 seanmlyons22

Got it. Makes sense to me!

niosus avatar May 18 '21 13:05 niosus