Based on Adam Simpkin's ideas, faster code for the template functions.
Details
Details
- Reviewers
quark - Group Reviewers
Restricted Project - Commits
- rFBHGX2ed26f4a0c71: undo: preview perf improvements
Diff Detail
Diff Detail
- Repository
- rFBHGX Facebook Mercurial Extensions
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
Comment Actions
It's now binary nodes, so maybe hex should be removed. But I'm fine with doing that in a later patch.
hgext3rd/undo.py | ||
---|---|---|
549 | It seems _oldmarks is just a hex node. In this case we can avoid creating a ctx object: ctxhex = ctx.hex() for ... if kv[1] == ctxhex: |
hgext3rd/undo.py | ||
---|---|---|
529 | This could also be optimized to avoid create a ctx (repo[...]) object. |
This could also be optimized to avoid create a ctx (repo[...]) object.