diff --git a/src/vs/editor/contrib/snippet/snippet.md b/src/vs/editor/contrib/snippet/snippet.md index ab479c9c08e3e..ce571c614af2f 100644 --- a/src/vs/editor/contrib/snippet/snippet.md +++ b/src/vs/editor/contrib/snippet/snippet.md @@ -86,7 +86,6 @@ tabstop ::= '$' int | '${' int transform '}' placeholder ::= '${' int ':' any '}' choice ::= '${' int '|' text (',' text)* '|}' - | '${' int '|' text (',' text)* '|' transform '}' variable ::= '$' var | '${' var }' | '${' var ':' any '}' | '${' var transform '}' @@ -102,5 +101,3 @@ var ::= [_a-zA-Z] [_a-zA-Z0-9]* int ::= [0-9]+ text ::= .* ``` - -Transformations for placeholders and choices are an extension to the TextMate snippet grammar and only support by Visual Studio Code. diff --git a/src/vs/editor/contrib/snippet/snippetParser.ts b/src/vs/editor/contrib/snippet/snippetParser.ts index a51273c773e0b..bf034b00bb3fa 100644 --- a/src/vs/editor/contrib/snippet/snippetParser.ts +++ b/src/vs/editor/contrib/snippet/snippetParser.ts @@ -746,13 +746,6 @@ export class SnippetParser { parent.appendChild(placeholder); return true; } - if (this._accept(TokenType.Forwardslash)) { - // ...|///} -> transform - if (this._parseTransform(placeholder)) { - parent.appendChild(placeholder); - return true; - } - } } } diff --git a/src/vs/editor/contrib/snippet/test/snippetParser.test.ts b/src/vs/editor/contrib/snippet/test/snippetParser.test.ts index 649ad400d7905..301c9dd1b81ca 100644 --- a/src/vs/editor/contrib/snippet/test/snippetParser.test.ts +++ b/src/vs/editor/contrib/snippet/test/snippetParser.test.ts @@ -254,13 +254,6 @@ suite('SnippetParser', () => { assertTextAndMarker('${1/regex/format/options', '${1/regex/format/options', Text); }); - test('Parser, placeholder with choice and transformation', () => { - assertTextAndMarker('${1|one,two,three|/foo/bar/}', 'one', Placeholder); - assertTextAndMarker('${1|one|/foo/bar/}', 'one', Placeholder); - assertTextAndMarker('${1|one,two,three,|/foo/bar/}', '${1|one,two,three,|/foo/bar/}', Text); - assertTextAndMarker('${1|one,/foo/bar/', '${1|one,/foo/bar/', Text); - }); - test('No way to escape forward slash in snippet regex #36715', function () { assertMarker('${TM_DIRECTORY/src\\//$1/}', Variable); });