py3: stop using util.iterfile()

The Python 3 implementation is a no-op. So this is equivalent.

We still keep util.iterfile() around for backwards API compatibility to
help the Python 3 migration. It can be deleted in a future release.

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

hgext/convert/common.py

@@ -552,7 +552,7 b' class mapfile(dict):'
             if err.errno != errno.ENOENT:
                 raise
             return
-        for i, line in enumerate(util.iterfile(fp)):
+        for i, line in enumerate(fp):
             line = line.splitlines()[0].rstrip()
             if not line:
                 # Ignore blank lines

hgext/convert/convcmd.py

@@ -242,7 +242,7 b' class converter(object):'
         m = {}
         try:
             fp = open(path, b'rb')
-            for i, line in enumerate(util.iterfile(fp)):
+            for i, line in enumerate(fp):
                 line = line.splitlines()[0].rstrip()
                 if not line:
                     # Ignore blank lines

mercurial/match.py

@@ -1611,7 +1611,7 b' def readpatternfile(filepath, warn, sour'
     patterns = []
 
     fp = open(filepath, b'rb')
-    for lineno, line in enumerate(util.iterfile(fp), start=1):
+    for lineno, line in enumerate(fp, start=1):
         if b"#" in line:
             global _commentre
             if not _commentre:

mercurial/patch.py

@@ -1259,7 +1259,7 b' the hunk is left unchanged.'
                     # Remove comment lines
                     patchfp = open(patchfn, 'rb')
                     ncpatchfp = stringio()
-                    for line in util.iterfile(patchfp):
+                    for line in patchfp:
                         line = util.fromnativeeol(line)
                         if not line.startswith(b'#'):
                             ncpatchfp.write(line)
@@ -2343,7 +2343,7 b' def _externalpatch(ui, repo, patcher, pa'
     ui.debug(b'Using external patch tool: %s\n' % cmd)
     fp = procutil.popen(cmd, b'rb')
     try:
-        for line in util.iterfile(fp):
+        for line in fp:
             line = line.rstrip()
             ui.note(line + b'\n')
             if line.startswith(b'patching file '):

mercurial/store.py

@@ -661,7 +661,7 b' class fncache(object):'
         """make sure there is no empty string in entries"""
         if b'' in self.entries:
             fp.seek(0)
-            for n, line in enumerate(util.iterfile(fp)):
+            for n, line in enumerate(fp):
                 if not line.rstrip(b'\n'):
                     t = _(b'invalid entry in fncache, line %d') % (n + 1)
                     if warn: