treemanifest: fix writepending to return False

Summary:
writepending needs to return False so the transaction mechanics know there is no
further work to be done here. This was just something caught during development.

Reviewed By: ryanmce

Differential Revision: D6867879

fbshipit-source-id: 73d49b06c5379f9a3388d42fbb454b5eba534a20
This commit is contained in:
Saurabh Singh 2018-02-27 11:09:55 -08:00 committed by Saurabh Singh
parent 10f72bc375
commit 120f1433c3

View File

@ -733,6 +733,9 @@ def _writeclientmanifest(newtree, tr, mfl, p1node, p2node, linknode,
# happens during amend.
tr.addpending('treepack', writepending)
# returning False indicates there are no pending changes.
return False
tr.addfinalize('treepack', finalize)
tr.addabort('treepack', abort)
tr.addpending('treepack', writepending)