Apply suggestions from code review

ooni · Feb 2, 2024 · c943c78 · c943c78
1 parent b2329f4
commit c943c78
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/internal/reliabletransport/sender.go b/internal/reliabletransport/sender.go
@@ -72,6 +72,7 @@ func (ws *workersState) blockOnTryingToSend(sender *reliableSender, ticker *time
 	ticker.Reset(timeout.Sub(now))
 	scheduledNow := inflightSequence(sender.inFlight).readyToSend(now)
 
+	// if we have packets to send piggyback the ACKs
 	if len(scheduledNow) > 0 {
 		// we flush everything that is ready to be sent.
 		for _, p := range scheduledNow {
@@ -91,6 +92,8 @@ func (ws *workersState) blockOnTryingToSend(sender *reliableSender, ticker *time
 		}
 		return
 	}
+
+	// if there are no ACKs to send, our job here is done
 	if !sender.hasPendingACKs() {
 		return
 	}