From 00a97272f2fe00c09dd954c21fffb4efebc208af Mon Sep 17 00:00:00 2001 From: Andrei Listochkin Date: Tue, 17 May 2022 18:31:51 +0100 Subject: automate braceless return substitution for long lines Per [bjorn3][https://github.com/bjorn3] suggestion resolves cases where an early return is moved to a separate line due to line width formatting. This setting changes ``` if (a very long condition) return; ``` to ``` if (a very long condition) { return; } ``` while keeping ``` if (short) return; ``` as is. In pathological cases this may cause `npm run fix` not to fix formatting in one go and may require running it twice. --- editors/code/src/snippets.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'editors/code/src/snippets.ts') diff --git a/editors/code/src/snippets.ts b/editors/code/src/snippets.ts index 14c2121d08a..299d29c27ee 100644 --- a/editors/code/src/snippets.ts +++ b/editors/code/src/snippets.ts @@ -11,7 +11,7 @@ export async function applySnippetWorkspaceEdit(edit: vscode.WorkspaceEdit) { } for (const [uri, edits] of edit.entries()) { const editor = await editorFromUri(uri); - if (editor) + if (editor) { await editor.edit((builder) => { for (const indel of edits) { assert( @@ -21,6 +21,7 @@ export async function applySnippetWorkspaceEdit(edit: vscode.WorkspaceEdit) { builder.replace(indel.range, indel.newText); } }); + } } } -- cgit 1.4.1-3-g733a5