Replace expensive arr.splice() with unordered-array-remove in hot code

For webtorrent/webtorrent-desktop#256

lib/peer.js

@@ -1,3 +1,4 @@
+var arrayRemove = require('unordered-array-remove')
 var debug = require('debug')('webtorrent:peer')
 var Wire = require('bittorrent-protocol')
 
@@ -240,8 +241,7 @@ Peer.prototype.destroy = function (err) {
   self.wire = null
 
   if (swarm && wire) {
-    var index = swarm.wires.indexOf(wire)
-    if (index >= 0) swarm.wires.splice(index, 1)
+    arrayRemove(swarm.wires, swarm.wires.indexOf(wire))
   }
   if (conn) {
     conn.on('error', noop)

lib/server.js

@@ -1,5 +1,6 @@
 module.exports = Server
 
+var arrayRemove = require('unordered-array-remove')
 var debug = require('debug')('webtorrent:server')
 var http = require('http')
 var mime = require('mime')
@@ -17,8 +18,7 @@ function Server (torrent, opts) {
     socket.setTimeout(36000000)
     sockets.push(socket)
     socket.on('close', function () {
-      var index = sockets.indexOf(socket)
-      if (index >= 0) sockets.splice(index, 1)
+      arrayRemove(sockets, sockets.indexOf(socket))
     })
   })
 

lib/tcp-pool.js

@@ -1,5 +1,6 @@
 module.exports = TCPPool
 
+var arrayRemove = require('unordered-array-remove')
 var debug = require('debug')('webtorrent:tcp-pool')
 var net = require('net') // browser exclude
 
@@ -198,7 +199,7 @@ TCPPool.prototype._onConnection = function (conn) {
   conn.once('close', removePendingConn)
 
   function removePendingConn () {
-    self.pendingConns.splice(self.pendingConns.indexOf(conn))
+    arrayRemove(self.pendingConns, self.pendingConns.indexOf(conn))
   }
 
   var peer = Peer.createTCPIncomingPeer(conn)

package.json

@@ -64,6 +64,7 @@
     "torrent-discovery": "^7.0.0",
     "torrent-piece": "^1.0.0",
     "uniq": "^1.0.1",
+    "unordered-array-remove": "feross/unordered-array-remove",
     "ut_metadata": "^3.0.2",
     "ut_pex": "^1.0.1",
     "xtend": "^4.0.0",