revid: sendRetry check is now in if rather than loop

revid/senders.go

@@ -75,16 +75,18 @@ func (s *multiSender) Write(d []byte) (int, error) {
 		if err != nil {
 			s.owner.config.Logger.Log(logger.Warning, "send failed", "output", i, "error", err)
 			s.handleFail(sender, err)
-			for s.owner.config.SendRetry {
-				if !s.owner.IsRunning() {
-					sender.release()
-					return 0, err
+			if s.owner.config.SendRetry {
+				for {
+					if !s.owner.IsRunning() {
+						sender.release()
+						return 0, err
+					}
+					err = sender.send()
+					if err == nil {
+						break
+					}
+					s.handleFail(sender, err)
 				}
-				err = sender.send()
-				if err == nil {
-					break
-				}
-				s.handleFail(sender, err)
 			}
 		}
 		sender.release()