diff options
author | Shawn Pearce <spearce@spearce.org> | 2014-04-26 10:40:30 -0700 |
---|---|---|
committer | Robin Rosenberg <robin.rosenberg@dewire.com> | 2014-05-19 15:45:43 -0400 |
commit | 6d724dcd3355f09e3450e417cf173fcafaee9e08 (patch) | |
tree | 3960b2d01016111e9d443f5e151f0cc0e61f6bc2 /org.eclipse.jgit.java7.test | |
parent | a5ee6fe904d6eb75a0edf6b3e93d7bfd733b78fe (diff) | |
download | jgit-6d724dcd3355f09e3450e417cf173fcafaee9e08.tar.gz jgit-6d724dcd3355f09e3450e417cf173fcafaee9e08.zip |
blame: Revert common subtree elimination "optimization"
This partially reverts 6de12836d72fe4cba9afc297c8988c6fff851fa9.
Performing a TreeWalk over 2 trees to identify and skip unmodified
subtrees to pass all blame onto an ancestor appears to be a micro
optimization that works for a very limited number of files. In the
general case the 2 tree walk is slowing down blame more than it helps
to speed it up.
I keep coming up with files in multiple repositories where 6de128 is
making things worse, not better, and only one example where it
actually improved performance, render_view_impl.cc in chromium
as described in the commit message.
Change-Id: Ic6d5fff22acb5ab6485614a07bdb388e8c336679
Diffstat (limited to 'org.eclipse.jgit.java7.test')
0 files changed, 0 insertions, 0 deletions