Comments and readability

config.go

@@ -41,8 +41,7 @@ type ClientConfig struct {
 	// Disable uploading even when it isn't fair.
 	DisableAggressiveUpload bool `long:"disable-aggressive-upload"`
 	// Upload even after there's nothing in it for us. By default uploading is
-	// not altruistic, we'll upload slightly more than we download from each
-	// peer.
+	// not altruistic, we'll only upload to encourage the peer to reciprocate.
 	Seed bool `long:"seed"`
 	// Only applies to chunks uploaded to peers, to maintain responsiveness
 	// communicating local Client state to peers. Each limiter token

connection.go

@@ -398,7 +398,11 @@ func (cn *connection) nominalMaxRequests() (ret int) {
 			),
 		))
 	}
-	return int(clamp(1, int64(cn.PeerMaxRequests), max(64, cn.stats.ChunksReadUseful.Int64()-(cn.stats.ChunksRead.Int64()-cn.stats.ChunksReadUseful.Int64()))))
+	return int(clamp(
+		1,
+		int64(cn.PeerMaxRequests),
+		max(64,
+			cn.stats.ChunksReadUseful.Int64()-(cn.stats.ChunksRead.Int64()-cn.stats.ChunksReadUseful.Int64()))))
 }
 
 func (cn *connection) totalExpectingTime() (ret time.Duration) {

peer_protocol/decoder.go

@@ -35,7 +35,7 @@ func (d *Decoder) Decode(msg *Message) (err error) {
 		return
 	}
 	msg.Keepalive = false
-	r := &io.LimitedReader{d.R, int64(length)}
+	r := &io.LimitedReader{R:d.R, N:int64(length)}
 	// Check that all of r was utilized.
 	defer func() {
 		if err != nil {