From be56fb7ab91fe021f2c73b75019610e4b22d96aa Mon Sep 17 00:00:00 2001
From: ZYSzys <17367077526@163.com>
Date: Sun, 11 Nov 2018 19:31:22 +0800
Subject: [PATCH] events: extract listener check as a function

PR-URL: https://github.com/nodejs/node/pull/24303
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 lib/events.js | 29 +++++++++++++----------------
 1 file changed, 13 insertions(+), 16 deletions(-)

diff --git a/lib/events.js b/lib/events.js
index e59ff3f04dd93b..5dfa34469b09af 100644
--- a/lib/events.js
+++ b/lib/events.js
@@ -48,6 +48,13 @@ function lazyErrors() {
   return errors;
 }
 
+function checkListener(listener) {
+  if (typeof listener !== 'function') {
+    const errors = lazyErrors();
+    throw new errors.ERR_INVALID_ARG_TYPE('listener', 'Function', listener);
+  }
+}
+
 Object.defineProperty(EventEmitter, 'defaultMaxListeners', {
   enumerable: true,
   get: function() {
@@ -195,10 +202,7 @@ function _addListener(target, type, listener, prepend) {
   var events;
   var existing;
 
-  if (typeof listener !== 'function') {
-    const errors = lazyErrors();
-    throw new errors.ERR_INVALID_ARG_TYPE('listener', 'Function', listener);
-  }
+  checkListener(listener);
 
   events = target._events;
   if (events === undefined) {
@@ -283,20 +287,16 @@ function _onceWrap(target, type, listener) {
 }
 
 EventEmitter.prototype.once = function once(type, listener) {
-  if (typeof listener !== 'function') {
-    const errors = lazyErrors();
-    throw new errors.ERR_INVALID_ARG_TYPE('listener', 'Function', listener);
-  }
+  checkListener(listener);
+
   this.on(type, _onceWrap(this, type, listener));
   return this;
 };
 
 EventEmitter.prototype.prependOnceListener =
     function prependOnceListener(type, listener) {
-      if (typeof listener !== 'function') {
-        const errors = lazyErrors();
-        throw new errors.ERR_INVALID_ARG_TYPE('listener', 'Function', listener);
-      }
+      checkListener(listener);
+
       this.prependListener(type, _onceWrap(this, type, listener));
       return this;
     };
@@ -306,10 +306,7 @@ EventEmitter.prototype.removeListener =
     function removeListener(type, listener) {
       var list, events, position, i, originalListener;
 
-      if (typeof listener !== 'function') {
-        const errors = lazyErrors();
-        throw new errors.ERR_INVALID_ARG_TYPE('listener', 'Function', listener);
-      }
+      checkListener(listener);
 
       events = this._events;
       if (events === undefined)