Remove unnecessary indirection for DiskObjectStore.
authorJelmer Vernooij <jelmer@samba.org>
Sun, 10 May 2009 13:13:50 +0000 (15:13 +0200)
committerJelmer Vernooij <jelmer@samba.org>
Sun, 10 May 2009 13:13:50 +0000 (15:13 +0200)
dulwich/repo.py
dulwich/tests/test_repository.py

index b20fb667bb1221c57188bc5d317023c26320063a..059e88eaa207830001f7f663ebf1ff40015bdec6 100644 (file)
@@ -108,7 +108,8 @@ class Repo(object):
             raise NotGitRepository(root)
         self.path = root
         self.tags = Tags(self.tagdir(), self.get_tags())
-        self._object_store = None
+        self.object_store = DiskObjectStore(
+            os.path.join(self.controldir(), OBJECTDIR))
 
     def controldir(self):
         """Return the path of the control directory."""
@@ -149,16 +150,6 @@ class Repo(object):
             heads = self.heads().values()
         return self.object_store.get_graph_walker(heads)
 
-    def object_dir(self):
-        """Return path of the object directory."""
-        return os.path.join(self.controldir(), OBJECTDIR)
-
-    @property
-    def object_store(self):
-        if self._object_store is None:
-            self._object_store = DiskObjectStore(self.object_dir())
-        return self._object_store
-
     def _get_ref(self, file):
         f = open(file, 'rb')
         try:
index 43b24967758b23b8e896c2e9e4954a0916ccbdce..f84f01e42ea322866b69efc91ad542ec772a2d76 100644 (file)
@@ -35,7 +35,6 @@ class RepositoryTests(unittest.TestCase):
         r = self.open_repo('a')
         basedir = os.path.join(os.path.dirname(__file__), 'data/repos/a/.git')
         self.assertEqual(r.controldir(), basedir)
-        self.assertEqual(r.object_dir(), os.path.join(basedir, 'objects'))
   
     def test_ref(self):
         r = self.open_repo('a')