PartialRange: decompose #compute => #set{Relays,Workers}

partial_range.js

@@ -67,20 +67,15 @@ function isValid() {
     return this.relayIndex >= 0;
 };
 
-PartialRange.prototype.compute =
-function compute(relays, workers, now) {
-    if (relays) {
-        this.relays = relays;
-    }
-
-    if (workers) {
-        this.workers = workers;
-    }
-
-    if (!relays && !workers) {
-        return;
-    }
+PartialRange.prototype.setRelays =
+function setRelays(relays, now) {
+    this.relays = relays;
+    this.pendingCompute = now;
+};
 
+PartialRange.prototype.setWorkers =
+function setWorkers(workers, now) {
+    this.workers = workers;
     this.pendingCompute = now;
 };
 

service-proxy.js

@@ -773,7 +773,8 @@ function getPartialRange(serviceName, reason, now) {
             self.minPeersPerRelay
         );
         self.partialRanges[serviceName] = partialRange;
-        partialRange.compute(relays, workers, now);
+        partialRange.setRelays(relays, now);
+        partialRange.setWorkers(workers, now);
     }
 
     if (!partialRange.isValid()) {
@@ -1095,7 +1096,7 @@ function updateServiceChannel(serviceChannel, now) {
         if (self.partialAffinityEnabled) {
             var partialRange = self.partialRanges[serviceChannel.serviceName];
             if (partialRange) {
-                partialRange.compute(self.relaysFor[serviceChannel.serviceName], null, now);
+                partialRange.setRelays(self.relaysFor[serviceChannel.serviceName], now);
             }
         }
 

test/partial-range.js

@@ -71,7 +71,8 @@ function buildPartialRanges(state) {
 function computePartialRanges(state, assert) {
     for (var i = 0; i < state.partialRanges.length; i++) {
         var partialRange = state.partialRanges[i];
-        partialRange.compute(state.relays, state.workers, 1);
+        partialRange.setRelays(state.relays, 1);
+        partialRange.setWorkers(state.workers, 1);
         partialRange.computeIfNeeded();
         assert.ok(
             partialRange.isValid(),