Merge pull request #240 from gyandeeps/master

Fix: Remove `inherit` and use Object.create for inheritance (fixes #239)
nodeca · Jan 11, 2016 · 05799a3 · 05799a3
2 parents df57eb5 + 6bf7c65
commit 05799a3
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 7 deletions.
diff --git a/.gitignore b/.gitignore
@@ -4,3 +4,4 @@ node_modules
 doc
 benchmark/implementations/*
 !/benchmark/implementations/current/
+.idea
diff --git a/lib/js-yaml/exception.js b/lib/js-yaml/exception.js
@@ -2,10 +2,6 @@
 //
 'use strict';
 
-
-var inherits = require('inherit');
-
-
 function YAMLException(reason, mark) {
   // Super constructor
   Error.call(this);
@@ -27,7 +23,8 @@ function YAMLException(reason, mark) {
 
 
 // Inherit from Error
-inherits(YAMLException, Error);
+YAMLException.prototype = Object.create(Error.prototype);
+YAMLException.prototype.constructor = YAMLException;
 
 
 YAMLException.prototype.toString = function toString(compact) {

diff --git a/package.json b/package.json
@@ -28,8 +28,7 @@
   },
   "dependencies": {
     "argparse": "^1.0.2",
-    "esprima": "^2.6.0",
-    "inherit": "^2.2.2"
+    "esprima": "^2.6.0"
   },
   "devDependencies": {
     "ansi": "*",