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: Ib4c74df4f1304e9df3015885bf360bf0d7bc6ca2tags/v4.5.0.201609210915-r
@@ -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; | |||
} |