782896 - Reflect.parse shows wrong loc for multiline blocks

Status: RESOLVED WORKSFORME

(Core :: JavaScript Engine, defect)

Description

[Thaddee Tyl [:espadrine]]

The following input, entered in a chrome-enabled scratchpad:

> Components.utils.import("resource://gre/modules/reflect.jsm");
> JSON.stringify(Reflect.parse("{\nlet foo = 3;}"), null, 2);

… will output a parse tree where the BlockStatement node
has the following, invalid, "loc" property:

    {
      "start": {
        "line": 1,
        "column": 0
      },
      "end": {
        "line": 1,
        "column": 1
      },
      "source": null
    }

It should rather be like this:

    {
      "start": {
        "line": 1,
        "column": 0
      },
      "end": {
        "line": 2,
        "column": 13
      },
      "source": null
    }

Currently, only the span of the first line is reflected, for blocks.

Comment 1

[André Bargull [:anba]]

No longer reproducible, therefore closing as WFM.

---
      "loc": {
        "start": {
          "line": 1,
          "column": 0
        },
        "end": {
          "line": 2,
          "column": 13
        },
        "source": null
      },
      "type": "BlockStatement",
---