fix: Do not use the native parser in Safari and Node.js 4

prantlf · prantlf · commit a4a606c333e4 · 2019-06-02T10:26:02.000+02:00
It does not return the offset of the error occurrence, which would not allow adding the context information to the thrown error.
diff --git a/src/configurable-parser.js b/src/configurable-parser.js
@@ -39,9 +39,12 @@ function restoreContext (changed) {
   }
 }
 
+var isSafari = typeof navigator !== 'undefined' && /Safari/.test(navigator.userAgent) && /Apple Computer/.test(navigator.vendor)
+var oldNode = typeof process !== 'undefined' && process.version.startsWith('v4.')
+
 function needsCustomParser () {
   return this.ignoreComments || this.allowSingleQuotedStrings ||
-    this.mode === 'cjson' || this.mode === 'json5'
+    this.mode === 'cjson' || this.mode === 'json5' || isSafari || oldNode
 }
 
 function getReviver (options) {

Original file line number	Diff line number	Diff line change
`@@ -39,9 +39,12 @@ function restoreContext (changed) {`
`39`	`39`	`}`
`40`	`40`	`}`
`41`	`41`
	`42`	`+var isSafari = typeof navigator !== 'undefined' && /Safari/.test(navigator.userAgent) && /Apple Computer/.test(navigator.vendor)`
	`43`	`+var oldNode = typeof process !== 'undefined' && process.version.startsWith('v4.')`
	`44`	`+`
`42`	`45`	`function needsCustomParser () {`
`43`	`46`	`return this.ignoreComments \|\| this.allowSingleQuotedStrings \|\|`
`44`		`- this.mode === 'cjson' \|\| this.mode === 'json5'`
	`47`	`+ this.mode === 'cjson' \|\| this.mode === 'json5' \|\| isSafari \|\| oldNode`
`45`	`48`	`}`
`46`	`49`
`47`	`50`	`function getReviver (options) {`