diff options
| author | Jesse Greenwald <jesse.greenwald@garmin.com> | 2011-01-14 16:34:35 -0600 |
|---|---|---|
| committer | Chris Aniszczyk <caniszczyk@gmail.com> | 2011-01-25 16:39:00 -0600 |
| commit | 51dedfdc319d51aa729f3e2801c4a06491d6ddac (patch) | |
| tree | b95e6d25b319962fea7527d5d4c7eb5e0cdfba7d /org.eclipse.jgit/src/org/eclipse/jgit/revwalk/PendingGenerator.java | |
| parent | 920ac08777d4048c06c8d7a3671d37c5ead5917f (diff) | |
| download | jgit-51dedfdc319d51aa729f3e2801c4a06491d6ddac.tar.gz jgit-51dedfdc319d51aa729f3e2801c4a06491d6ddac.zip | |
Parse RevCommit bodies before calling RevFilter.include()
RevFilter.include()'s documentation promises the RevCommit's
body is parsed before include is invoked. This wasn't always
true if the commit was parsed once, had its body discarded,
the RevWalk was reset() and started a new traversal.
Change-Id: Ie5cafde09ae870712b165d8a97a2c9daf90b1dbd
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>
Diffstat (limited to 'org.eclipse.jgit/src/org/eclipse/jgit/revwalk/PendingGenerator.java')
| -rw-r--r-- | org.eclipse.jgit/src/org/eclipse/jgit/revwalk/PendingGenerator.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/revwalk/PendingGenerator.java b/org.eclipse.jgit/src/org/eclipse/jgit/revwalk/PendingGenerator.java index ec7069b4d2..d0a1a832fe 100644 --- a/org.eclipse.jgit/src/org/eclipse/jgit/revwalk/PendingGenerator.java +++ b/org.eclipse.jgit/src/org/eclipse/jgit/revwalk/PendingGenerator.java @@ -137,8 +137,10 @@ class PendingGenerator extends Generator { final boolean produce; if ((c.flags & UNINTERESTING) != 0) produce = false; - else + else { + c.parseBody(walker); produce = filter.include(walker, c); + } for (final RevCommit p : c.parents) { if ((p.flags & SEEN) != 0) |