Skip to content
Snippets Groups Projects
Commit bae6f141 authored by Boris Feld's avatar Boris Feld
Browse files

obsolete: explicitly pass relation items to effectflag computation

To track folds, we are about to change the content of `rel`. To simplify this
change, we update the `geteffectflag` function beforehand.
parent 0428feb1
Branches
Tags
No related merge requests found
......@@ -1016,7 +1016,7 @@
if saveeffectflag:
# The effect flag is saved in a versioned field name for future
# evolution
effectflag = obsutil.geteffectflag(rel)
effectflag = obsutil.geteffectflag(prec, sucs)
localmetadata[obsutil.EFFECTFLAGFIELD] = "%d" % effectflag
# Creating the marker causes the hidden cache to become invalid,
......
......@@ -413,9 +413,9 @@
return False
return True
def geteffectflag(relation):
def geteffectflag(source, successors):
""" From an obs-marker relation, compute what changed between the
predecessor and the successor.
"""
effects = 0
......@@ -417,11 +417,9 @@
""" From an obs-marker relation, compute what changed between the
predecessor and the successor.
"""
effects = 0
source = relation[0]
for changectx in relation[1]:
for changectx in successors:
# Check if description has changed
if changectx.description() != source.description():
effects |= DESCCHANGED
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment