Closed Bug 748201 Opened 12 years ago Closed 11 years ago

Bogus |end| for ContinueStatement

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED DUPLICATE of bug 747831

People

(Reporter: n.nethercote, Assigned: n.nethercote)

Details

This code looks wrong:

4135       case TOK_CONTINUE:
4136       {
4137         TokenPtr begin = tokenStream.currentToken().pos.begin;
4138         PropertyName *label;
4139         if (!MatchLabel(context, &tokenStream, &label))
4140             return NULL;
4141         TokenPtr end = tokenStream.currentToken().pos.begin;
4142         pn = new_<ContinueStatement>(label, begin, end);

In line 4141 it probably should be |.end|, not |.begin|.  Reflect.parse could be used to make a test.
My patch in bug 747831 fixes this.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.