browser: reuse index of found services between responses

watson · Dec 29, 2015 · af0c2fb · af0c2fb
1 parent a2038a1
commit af0c2fb
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/lib/browser.js b/lib/browser.js
@@ -33,6 +33,7 @@ function Browser (mdns, opts, onup) {
  this._name = serviceName.stringify(opts.type, opts.protocol || 'tcp') + TLD
  this._mdns = mdns
  this._onresponse = null
+ this._serviceMap = {}
  this.services = []
  this.start()
 }
@@ -46,13 +47,8 @@ Browser.prototype.start = function () {
  var matches = buildServicesFor(self._name, packet)
  if (matches.length === 0) return
 
- var exists = {}
- self.services.forEach(function (service) {
- exists[service.fqdn] = true
- })
-
  matches.forEach(function (service) {
- if (exists[service.fqdn]) return
+ if (self._serviceMap[service.fqdn]) return
  self._addService(service)
  })
  }
@@ -70,6 +66,7 @@ Browser.prototype.stop = function () {
 
 Browser.prototype._addService = function (service) {
  this.services.push(service)
+ this._serviceMap[service.fqdn] = true
  this.emit('up', service)
 }