Replace removeListener() with off()

Author: ehmicky

src/main.js

@@ -54,9 +54,7 @@ const stopLogging = function(listeners) {
 }
 
 const removeListener = function({ eventListener, name }) {
-  // TODO: use `process.off()` instead of `process.removeListener()`
-  // after dropping Node.js <10 support
-  process.removeListener(name, eventListener)
+  process.off(name, eventListener)
 }
 
 // We do not use `export default` because Babel transpiles it in a way that

src/warnings.js

@@ -12,7 +12,7 @@ export const removeWarningListener = function() {
   }
 
   // This will be a noop if `init()` is called several times
-  process.removeListener('warning', warningListener)
+  process.off('warning', warningListener)
 }
 
 // When this module is undone, Node.js default `warning` listener is restored

test/main/init.js

@@ -55,9 +55,7 @@ each(EVENTS, ({ title }, { eventName, emit, defaultLevel }) => {
 
       stopLogging()
 
-      // TODO: use `process.off()` instead of `process.removeListener()`
-      // after dropping Node.js <10 support
-      process.removeListener(eventName, processHandler)
+      process.off(eventName, processHandler)
     },
   )
 
@@ -77,8 +75,6 @@ each(EVENTS, ({ title }, { eventName, emit, defaultLevel }) => {
 
     stopLogging()
 
-    // TODO: use `process.off()` instead of `process.removeListener()`
-    // after dropping Node.js <10 support
-    process.removeListener(eventName, processHandler)
+    process.off(eventName, processHandler)
   })
 })