##// END OF EJS Templates
with: use context manager for transaction in pushphase
Bryan O'Sullivan -
r27874:4ff0e234 default
parent child Browse files
Show More
@@ -409,12 +409,8 b' def pushphase(repo, nhex, oldphasestr, n'
409 newphase = abs(int(newphasestr)) # let's avoid negative index surprise
409 newphase = abs(int(newphasestr)) # let's avoid negative index surprise
410 oldphase = abs(int(oldphasestr)) # let's avoid negative index surprise
410 oldphase = abs(int(oldphasestr)) # let's avoid negative index surprise
411 if currentphase == oldphase and newphase < oldphase:
411 if currentphase == oldphase and newphase < oldphase:
412 try:
412 with repo.transaction('pushkey-phase') as tr:
413 tr = repo.transaction('pushkey-phase')
414 advanceboundary(repo, tr, newphase, [bin(nhex)])
413 advanceboundary(repo, tr, newphase, [bin(nhex)])
415 tr.close()
416 finally:
417 tr.release()
418 return 1
414 return 1
419 elif currentphase == newphase:
415 elif currentphase == newphase:
420 # raced, but got correct result
416 # raced, but got correct result
General Comments 0
You need to be logged in to leave comments. Login now