From 47f47ffc07e00109dc069e111274252e4053a599 Mon Sep 17 00:00:00 2001 From: Roberto Tyley Date: Wed, 29 Jan 2014 00:01:53 +0000 Subject: Initialise ReceiveCommand status to NOT_ATTEMPTED for all constructors Formerly the 4-arg constructor did not do this, which was unfortunate as that constructor's the only way for an external user of JGit to set the /type/ of the ref-update - which you might want to do to indicate that the update is expected to be a UPDATE_NONFASTFORWARD, and thus does not require expensive isMergedInto() calculations: http://dev.eclipse.org/mhonarc/lists/jgit-dev/msg02258.html Change-Id: I84c5e4927131e105bed93e31a62da6367c78de32 Signed-off-by: Roberto Tyley --- .../tst/org/eclipse/jgit/internal/storage/file/RefDirectoryTest.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'org.eclipse.jgit.test') diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/internal/storage/file/RefDirectoryTest.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/internal/storage/file/RefDirectoryTest.java index a821e948e7..87b9176368 100644 --- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/internal/storage/file/RefDirectoryTest.java +++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/internal/storage/file/RefDirectoryTest.java @@ -1278,10 +1278,8 @@ public class RefDirectoryTest extends LocalDiskRepositoryTestCase { private static ReceiveCommand newCommand(RevCommit a, RevCommit b, String string, Type update) { - ReceiveCommand ret = new ReceiveCommand(a != null ? a.getId() : null, + return new ReceiveCommand(a != null ? a.getId() : null, b != null ? b.getId() : null, string, update); - ret.setResult(ReceiveCommand.Result.NOT_ATTEMPTED); - return ret; } private void writeLooseRef(String name, AnyObjectId id) throws IOException { -- cgit v1.2.3