Replace use of Array.map and Array.filter in order to avoid the need for polyfills. Release v1.3.3

Author: eriwen

component.json

@@ -2,7 +2,7 @@
     "name": "error-stack-parser",
     "repository": "stacktracejs/error-stack-parser",
     "description": "Extract meaning from JS Errors",
-    "version": "1.2.2",
+    "version": "1.3.3",
     "keywords": [
         "stacktrace",
         "error",

dist/error-stack-parser-with-polyfills.min.js

@@ -17,6 +17,32 @@
     var CHROME_IE_STACK_REGEXP = /^\s*at .*(\S+\:\d+|\(native\))/m;
     var SAFARI_NATIVE_CODE_REGEXP = /^(eval@)?(\[native code\])?$/;
 
+    function _map(array, fn, thisArg) {
+        if (typeof Array.prototype.map === 'function') {
+            return array.map(fn, thisArg);
+        } else {
+            var output = new Array(array.length);
+            for (var i = 0; i < array.length; i++) {
+                output[i] = fn.call(thisArg, array[i]);
+            }
+            return output;
+        }
+    }
+
+    function _filter(array, fn, thisArg) {
+        if (typeof Array.prototype.filter === 'function') {
+            return array.filter(fn, thisArg);
+        } else {
+            var output = [];
+            for (var i = 0; i < array.length; i++) {
+                if (fn.call(thisArg, array[i])) {
+                    output.push(array[i]);
+                }
+            }
+            return output;
+        }
+    }
+
     return {
         /**
          * Given an Error object, extract the most information from it.
@@ -28,7 +54,7 @@
                 return this.parseOpera(error);
             } else if (error.stack && error.stack.match(CHROME_IE_STACK_REGEXP)) {
                 return this.parseV8OrIE(error);
-            } else if (error.stack && error.stack.match(FIREFOX_SAFARI_STACK_REGEXP)) {
+            } else if (error.stack) {
                 return this.parseFFOrSafari(error);
             } else {
                 throw new Error('Cannot parse given Error object');
@@ -58,9 +84,11 @@
         },
 
         parseV8OrIE: function ErrorStackParser$$parseV8OrIE(error) {
-            return error.stack.split('\n').filter(function (line) {
+            var filtered = _filter(error.stack.split('\n'), function (line) {
                 return !!line.match(CHROME_IE_STACK_REGEXP);
-            }, this).map(function (line) {
+            }, this);
+
+            return _map(filtered, function (line) {
                 if (line.indexOf('(eval ') > -1) {
                     // Throw away eval information until we implement stacktrace.js/stackframe#8
                     line = line.replace(/eval code/g, 'eval').replace(/(\(eval at [^\()]*)|(\)\,.*$)/g, '');
@@ -75,9 +103,11 @@
         },
 
         parseFFOrSafari: function ErrorStackParser$$parseFFOrSafari(error) {
-            return error.stack.split('\n').filter(function (line) {
+            var filtered = _filter(error.stack.split('\n'), function (line) {
                 return !line.match(SAFARI_NATIVE_CODE_REGEXP);
-            }, this).map(function (line) {
+            }, this);
+
+            return _map(filtered, function (line) {
                 // Throw away eval information until we implement stacktrace.js/stackframe#8
                 if (line.indexOf(' > eval') > -1) {
                     line = line.replace(/ line (\d+)(?: > eval line \d+)* > eval\:\d+\:\d+/g, ':$1');
@@ -138,10 +168,12 @@
 
         // Opera 10.65+ Error.stack very similar to FF/Safari
         parseOpera11: function ErrorStackParser$$parseOpera11(error) {
-            return error.stack.split('\n').filter(function (line) {
+            var filtered = _filter(error.stack.split('\n'), function (line) {
                 return !!line.match(FIREFOX_SAFARI_STACK_REGEXP) &&
                     !line.match(/^Error created at/);
-            }, this).map(function (line) {
+            }, this);
+
+            return _map(filtered, function (line) {
                 var tokens = line.split('@');
                 var locationParts = this.extractLocation(tokens.pop());
                 var functionCall = (tokens.shift() || '');