Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Various fixes #170

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Various fixes #170

wants to merge 6 commits into from

Conversation

Mr-Bossman
Copy link

No description provided.

Run formater on extension.ts

Signed-off-by: Jesse Taube <Mr.Bossman075@gmail.com>
Add the ability to use
${workspaceRoot}
${workspaceFolder}
${cwd}
${env.VAR}
in the .clang-format path

Signed-off-by: Jesse Taube <Mr.Bossman075@gmail.com>
Fix most of the variables should be const.
Remove unused variables.
Move many declarations to the top of the function.

Signed-off-by: Jesse Taube <Mr.Bossman075@gmail.com>
`vscode.workspace.getConfiguration('clang-format')` is called in
almost every function often multiple times. This commit adds
`const config = ...` at the top of each function to reduce repetition.

Signed-off-by: Jesse Taube <Mr.Bossman075@gmail.com>
If vscode has no workspace the extension will fail.
Check if the styles are valid before using them.

Signed-off-by: Jesse Taube <Mr.Bossman075@gmail.com>
Some users may want to pass additional arguments to clang-format. This
commit adds a setting to allow users to do so.

Signed-off-by: Jesse Taube <Mr.Bossman075@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant