Issue 6741 - Use ES2015 classes in lib/events.js

lib/events.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 44 matching lines @@
     {
       let idx = listeners.indexOf(listener);
       if (idx != -1)
         listeners.splice(idx, 1);
     }
   }
 
   /**
    * Adds a one time listener and returns a promise that
    * is resolved the next time the specified event is emitted.
+   *
    * @param {string} name

[Manish Jethani, 2018/08/17 03:07:21]

Can we also insert a blank line here before @param?

[Jon Sonesen, 2018/08/17 17:26:24]

Acknowledged.

    * @returns {Promise}
    */
   once(name)
   {
     return new Promise(resolve =>
     {
       let listener = () =>
       {
         this.off(name, listener);
         resolve();
@@ skipping 12 matching lines @@
   listeners(name)
   {
     let listeners = this._listeners.get(name);
     return listeners ? listeners.slice() : [];
   }
 
   /**
    * Calls all previously added listeners for the given event name.
    *
    * @param {string} name
-   * @param {args}   [arg]
+   * @param {...*}   [args]
    */
   emit(name, ...args)
   {
     let listeners = this.listeners(name);
     for (let listener of listeners)
       listener(...args);
   }
 }
 
 exports.EventEmitter = EventEmitter;