# HG changeset patch
# User Martin von Zweigbergk <martinvonz@google.com>
# Date 2018-12-05 05:58:46
# Node ID b34b1b865ef953933ac92e651e7c72789411b188
# Parent  e58cd7ede1c3d21d2cb8216cf948abb3be6f660c

remotefilelog: avoid temporarily using "count" variable as synonym for "total"

The "count" variable is generally used for updating progress, but
early in fileserverclient.request(), its used to mean the total
count. We already have another "total" variable for that, so it seems
much clearer to use that.

Differential Revision: https://phab.mercurial-scm.org/D5382

diff --git a/hgext/remotefilelog/fileserverclient.py b/hgext/remotefilelog/fileserverclient.py
--- a/hgext/remotefilelog/fileserverclient.py
+++ b/hgext/remotefilelog/fileserverclient.py
@@ -317,8 +317,8 @@ class fileserverclient(object):
         writedata = self.writedata
 
         repo = self.repo
-        count = len(fileids)
-        request = "get\n%d\n" % count
+        total = len(fileids)
+        request = "get\n%d\n" % total
         idmap = {}
         reponame = repo.name
         for file, id in fileids:
@@ -330,8 +330,7 @@ class fileserverclient(object):
 
         cache.request(request)
 
-        total = count
-        progress = self.ui.makeprogress(_('downloading'), total=count)
+        progress = self.ui.makeprogress(_('downloading'), total=total)
         progress.update(0)
 
         missed = []