comparison tests/test-eol-hook.t @ 28364:f1460af18c50

branchmap: check node against changelog instead of repo Testing 'node in repo' requires constructing a changectx, which is a little expensive. Testing 'repo.changelog.hasnode(node)' is notably faster. This saves 10-20ms off of every command, when testing a few thousand nodes from the branch cache. I considered changing the implementation of localrepository.__contains__ so every place would benefit from the change, but since localrepository.__contains__ uses changectx to check if the commit exists, it means it supports a wider range of possible inputs (like revs, hashes, '.', etc), so it seemed unnecessarily risky.
author Durham Goode <durham@fb.com>
date Mon, 07 Mar 2016 17:26:47 -0800
parents f5b6b4e574c1
children d7304434390f
comparison
equal deleted inserted replaced
28363:1f94ef2bd88d 28364:f1460af18c50