Skip to content

Commit 848be46

Browse files
authored
Revert "[javascript] fix problem with continue/break/return statements #3772 (#3773)" (#3806)
This reverts commit a2c72ef.
1 parent d0347c2 commit 848be46

File tree

10 files changed

+3
-50
lines changed

10 files changed

+3
-50
lines changed

javascript/javascript/CSharp/JavaScriptParserBase.cs

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -138,10 +138,4 @@ protected bool popHtmlTagName(string tagName)
138138
{
139139
return string.Equals(_tagNames.Pop(),tagName, System.StringComparison.InvariantCulture);
140140
}
141-
142-
143-
protected bool notLineTerminatorAhead()
144-
{
145-
return !lineTerminatorAhead();
146-
}
147141
}

javascript/javascript/Cpp/JavaScriptParserBase.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -88,8 +88,3 @@ bool JavaScriptParserBase::lineTerminatorAhead()
8888
return (type == JavaScriptParser::MultiLineComment && (text.find("\r") != std::string::npos || text.find("\n") != std::string::npos)) ||
8989
(type == JavaScriptParser::LineTerminator);
9090
}
91-
92-
bool JavaScriptParserBase::notLineTerminatorAhead()
93-
{
94-
return !notLineTerminatorAhead();
95-
}

javascript/javascript/Cpp/JavaScriptParserBase.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,5 +14,4 @@ class JavaScriptParserBase : public antlr4::Parser {
1414
bool closeBrace();
1515
bool here(int type);
1616
bool lineTerminatorAhead();
17-
bool notLineTerminatorAhead();
1817
};

javascript/javascript/Go/javascript_parser_base.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,3 @@ func (p *JavaScriptParserBase) lineTerminatorAhead() bool {
103103
return (_type == JavaScriptParserMultiLineComment && (strings.Contains(text, "\r") || strings.Contains(text, "\n"))) ||
104104
(_type == JavaScriptParserLineTerminator)
105105
}
106-
107-
func (p *JavaScriptParserBase) notLineTerminatorAhead() bool {
108-
return !p.lineTerminatorAhead()
109-
}

javascript/javascript/Java/JavaScriptParserBase.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -124,8 +124,4 @@ protected boolean lineTerminatorAhead() {
124124
return (type == JavaScriptParser.MultiLineComment && (text.contains("\r") || text.contains("\n"))) ||
125125
(type == JavaScriptParser.LineTerminator);
126126
}
127-
128-
protected boolean notLineTerminatorAhead() {
129-
return !lineTerminatorAhead();
130-
}
131127
}

javascript/javascript/JavaScript/JavaScriptParserBase.js

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -82,8 +82,4 @@ export default class JavaScriptParserBase extends antlr4.Parser {
8282
type === JavaScriptParser.LineTerminator
8383
);
8484
}
85-
86-
notLineTerminatorAhead() {
87-
return !this.lineTerminatorAhead();
88-
}
8985
}

javascript/javascript/JavaScriptParser.g4

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -185,15 +185,15 @@ varModifier // let, const - ECMAScript 6
185185
;
186186

187187
continueStatement
188-
: Continue ({this.notLineTerminatorAhead()}? identifier)? eos
188+
: Continue ({this.notLineTerminator()}? identifier)? eos
189189
;
190190

191191
breakStatement
192-
: Break ({this.notLineTerminatorAhead()}? identifier)? eos
192+
: Break ({this.notLineTerminator()}? identifier)? eos
193193
;
194194

195195
returnStatement
196-
: Return ({this.notLineTerminatorAhead()}? expressionSequence)? eos
196+
: Return ({this.notLineTerminator()}? expressionSequence)? eos
197197
;
198198

199199
yieldStatement

javascript/javascript/Python3/JavaScriptParserBase.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -114,8 +114,3 @@ def lineTerminatorAhead(self) -> bool:
114114

115115
return (tokenType == JavaScriptParser.MultiLineComment and (("\r" in text) or ("\n" in text))) or \
116116
(tokenType == JavaScriptParser.LineTerminator)
117-
118-
def notLineTerminatorAhead(self) -> bool:
119-
JavaScriptParser = self.parser()
120-
121-
return not self.lineTerminatorAhead()

javascript/javascript/examples/issue3772ContinueStatement.js

Lines changed: 0 additions & 17 deletions
This file was deleted.

javascript/javascript/examples/issue3772ContinueStatement.js.tree

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)