motiejus
/
jgit
1
Fork 0
Code Releases Activity

Building bitmaps: Simplify the logic to sort by chains 

Change-Id: I3da98e85107154c159093c138893f54dfa7ef435
Signed-off-by: Terry Parker <tparker@google.com>
This commit is contained in:
Terry Parker 2015-10-29 11:02:01 -07:00
parent f6b9cd38ca
commit 1eee0466ca
1 changed files with 10 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/pack/PackWriterBitmapPreparer.java
View File

@ -269,30 +269,23 @@ Collection<BitmapCommit> selectCommits(int expectedCommitCount)
// selected commit to fullBitmap.
}
// Sort the commits by independent chains in its history,
// yielding better compression when building bitmaps.
List<List<BitmapCommit>> candidateChain = new ArrayList<
List<BitmapCommit>>();
// Sort the commits by independent chains in this branch's
// history, yielding better compression when building bitmaps.
List<BitmapCommit> longestAncestorChain = null;
for (List<BitmapCommit> chain : chains) {
BitmapCommit mostRecentCommit = chain.get(chain.size() - 1);
if (fullBitmap.contains(mostRecentCommit)) {
candidateChain.add(chain);
}
}
List<BitmapCommit> longestAncestorChain;
if (candidateChain.isEmpty()) {
longestAncestorChain = new ArrayList<BitmapCommit>();
chains.add(longestAncestorChain);
} else {
longestAncestorChain = candidateChain.get(0);
// Append to longest
for (List<BitmapCommit> chain : candidateChain) {
if (chain.size() > longestAncestorChain.size()) {
if (longestAncestorChain == null
|| longestAncestorChain.size() < chain.size()) {
longestAncestorChain = chain;
}
}
}
if (longestAncestorChain == null) {
longestAncestorChain = new ArrayList<BitmapCommit>();
chains.add(longestAncestorChain);
}
longestAncestorChain.add(new BitmapCommit(
c, !longestAncestorChain.isEmpty(), flags));
writeBitmaps.addBitmap(c, fullBitmap, 0);