From de4bcf87f8e23ba1f64886849b060c46ad4940aa Mon Sep 17 00:00:00 2001 From: Shawn Pearce Date: Mon, 4 Jul 2016 17:19:06 -0700 Subject: [PATCH] UploadPack: Remove duplicate sentReady assignment This field was being set twice within the block. Setting it just once is sufficient. writeString() does not examine the field so it is fine to set it after the call. Change-Id: Ib4c74df4f1304e9df3015885bf360bf0d7bc6ca2 --- org.eclipse.jgit/src/org/eclipse/jgit/transport/UploadPack.java | 1 - 1 file changed, 1 deletion(-) diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/transport/UploadPack.java b/org.eclipse.jgit/src/org/eclipse/jgit/transport/UploadPack.java index ac8f5f61f..3f68bfeb7 100644 --- a/org.eclipse.jgit/src/org/eclipse/jgit/transport/UploadPack.java +++ b/org.eclipse.jgit/src/org/eclipse/jgit/transport/UploadPack.java @@ -1142,7 +1142,6 @@ public class UploadPack { if (multiAck == MultiAck.DETAILED && !didOkToGiveUp && okToGiveUp()) { ObjectId id = peerHas.get(peerHas.size() - 1); - sentReady = true; pckOut.writeString("ACK " + id.name() + " ready\n"); //$NON-NLS-1$ //$NON-NLS-2$ sentReady = true; }