checknlink: rename file object from 'fd' to 'fp'
authorJun Wu <quark@fb.com>
Wed, 06 Sep 2017 12:56:19 -0700
changeset 34101 6c5cdb02f2f9
parent 34100 08346a8fa65f
child 34102 e8a7c1a0565a
checknlink: rename file object from 'fd' to 'fp' Make it clear that `fp` (`file` object) is different from `fd` (low-level file descriptor number). Differential Revision: https://phab.mercurial-scm.org/D642
mercurial/util.py
--- a/mercurial/util.py	Tue Sep 05 15:06:45 2017 -0700
+++ b/mercurial/util.py	Wed Sep 06 12:56:19 2017 -0700
@@ -1457,24 +1457,23 @@
 
     # testfile may be open, so we need a separate file for checking to
     # work around issue2543 (or testfile may get lost on Samba shares)
-    f1, f2, fd = None, None, None
+    f1, f2, fp = None, None, None
     try:
         fd, f1 = tempfile.mkstemp(prefix='.%s-' % os.path.basename(testfile),
                                   suffix='1~', dir=os.path.dirname(testfile))
         os.close(fd)
-        fd = None
         f2 = '%s2~' % f1[:-2]
 
         oslink(f1, f2)
         # nlinks() may behave differently for files on Windows shares if
         # the file is open.
-        fd = posixfile(f2)
+        fp = posixfile(f2)
         return nlinks(f2) > 1
     except OSError:
         return False
     finally:
-        if fd is not None:
-            fd.close()
+        if fp is not None:
+            fp.close()
         for f in (f1, f2):
             try:
                 if f is not None: