Fix the format of bigint property on bigint literal

FIX: Fix an issue where the `bigint` property of literal nodes for non-decimal
bigints had the wrong format.

Closes #1371

Author: marijnh

acorn-loose/src/expression.js

@@ -271,7 +271,8 @@ lp.parseExprAtom = function() {
     node = this.startNode()
     node.value = this.tok.value
     node.raw = this.input.slice(this.tok.start, this.tok.end)
-    if (this.tok.type === tt.num && node.raw.charCodeAt(node.raw.length - 1) === 110) node.bigint = node.raw.slice(0, -1).replace(/_/g, "")
+    if (this.tok.type === tt.num && node.raw.charCodeAt(node.raw.length - 1) === 110)
+      node.bigint = node.value != null ? node.value.toString() : node.raw.slice(0, -1).replace(/_/g, "")
     this.next()
     return this.finishNode(node, "Literal")
 

acorn/src/expression.js

@@ -598,7 +598,8 @@ pp.parseLiteral = function(value) {
   let node = this.startNode()
   node.value = value
   node.raw = this.input.slice(this.start, this.end)
-  if (node.raw.charCodeAt(node.raw.length - 1) === 110) node.bigint = node.raw.slice(0, -1).replace(/_/g, "")
+  if (node.raw.charCodeAt(node.raw.length - 1) === 110)
+    node.bigint = node.value != null ? node.value.toString() : node.raw.slice(0, -1).replace(/_/g, "")
   this.next()
   return this.finishNode(node, "Literal")
 }

test/tests-bigint.js

@@ -3,28 +3,28 @@ if (typeof exports !== "undefined") {
   var testFail = require("./driver.js").testFail;
 }
 
-const newBigIntLiteral = (start, stringValue) => ({
+const newBigIntLiteral = (start, stringValue, bigint = stringValue) => ({
   start: start,
   type: "Literal",
   end: start + stringValue.length + 1,
   value: typeof BigInt !== "undefined" ? BigInt(stringValue) : null,
   raw: `${stringValue}n`,
-  bigint: stringValue
+  bigint
 })
 
 const digits = [
   {d: "0", ast: start => newBigIntLiteral(start, "0")},
   {d: "2", ast: start => newBigIntLiteral(start, "2")},
-  {d: "0x2", ast: start => newBigIntLiteral(start, "0x2")},
-  {d: "0o2", ast: start => newBigIntLiteral(start, "0o2")},
-  {d: "0b10", ast: start => newBigIntLiteral(start, "0b10")},
+  {d: "0x2", ast: start => newBigIntLiteral(start, "0x2", "2")},
+  {d: "0o2", ast: start => newBigIntLiteral(start, "0o2", "2")},
+  {d: "0b10", ast: start => newBigIntLiteral(start, "0b10", "2")},
   {d: "-0xbf2ed51ff75d380fd3be813ec6185780", ast: start => ({
     start: start,
     type: "UnaryExpression",
     end: start + 36,
     operator: "-",
     prefix: true,
-    argument: newBigIntLiteral(start + 1, "0xbf2ed51ff75d380fd3be813ec6185780")
+    argument: newBigIntLiteral(start + 1, "0xbf2ed51ff75d380fd3be813ec6185780", "254125715536285641815112686497309415296")
   })},
   {d: "02", error: start => `Identifier directly after number (1:${start + 2})`},
   {d: "2e2", error: start => `Identifier directly after number (1:${start + 3})`},