obsolete: stop doing membership test on list

According to the Surgeon General, computer should not use list for membership
testing because of the risk of being slow.
This commit is contained in:
Pierre-Yves David 2013-11-16 20:34:58 -05:00
parent d5a4643492
commit f1f6dad830

View File

@ -267,7 +267,8 @@ class obsstore(object):
Return the number of new marker."""
if not _enabled:
raise util.Abort('obsolete feature is not enabled on this repo')
new = [m for m in markers if m not in self._all]
known = set(self._all)
new = [m for m in markers if m not in known]
if new:
f = self.sopener('obsstore', 'ab')
try: