# HG changeset patch
# User Yuya Nishihara <yuya@tcha.org>
# Date 2015-09-13 08:47:18
# Node ID f2788794183aee9abbadb9da4c689700772c6670
# Parent  428a8747f4eeabe36efc411d216bbe0e39faae12

obsstore: initialize _all markers without using _addmarkers()

The next patch will make _all variable propertycached to avoid costly parsing
of obsstore. This means we can't call _addmarkers() to initialize _all.

Because all cached markers depend on _all, it isn't necessary to update these
caches when _all is initially loaded.

diff --git a/mercurial/obsolete.py b/mercurial/obsolete.py
--- a/mercurial/obsolete.py
+++ b/mercurial/obsolete.py
@@ -527,7 +527,9 @@ class obsstore(object):
         self._readonly = readonly
         if data:
             self._version, markers = _readmarkers(data)
-            self._addmarkers(markers)
+            markers = list(markers)
+            _checkinvalidmarkers(markers)
+            self._all = markers
 
     def __iter__(self):
         return iter(self._all)