From: Rob Browning <rlb@defaultvalue.org>
Date: Fri, 23 May 2014 22:02:11 +0000 (-0500)
Subject: Don't call _mksubs() in VFS Node nlink()
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=0c4ba8efc5da8ee9ce03595ef3b7426158166aa5;p=packages%2Fb%2Fbup.git

Don't call _mksubs() in VFS Node nlink()

The VFS Node nlink() code calls _mksubs(), but then just returns 1.
It looks like that may be unnecessary work, so stop doing it.

Thanks to William Stein <wstein@gmail.com> for pointing it out.

Signed-off-by: Rob Browning <rlb@defaultvalue.org>
---

diff --git a/lib/bup/vfs.py b/lib/bup/vfs.py
index 9e2d560..d76f26f 100644
--- a/lib/bup/vfs.py
+++ b/lib/bup/vfs.py
@@ -281,8 +281,6 @@ class Node(object):
 
     def nlinks(self):
         """Get the number of hard links to the current node."""
-        if self._subs == None:
-            self._mksubs()
         return 1
 
     def size(self):