notify-mailer: fix off-by-one in printStatus args

Daniel · Daniel · commit 7d8f55d64ca8 · 2019-01-15T16:25:37.000-05:00
diff --git a/cmd/notify-mailer/main.go b/cmd/notify-mailer/main.go
@@ -110,7 +110,7 @@ func (m *mailer) run() error {
 	startTime := m.clk.Now()
 
 	for i, dest := range destinations {
-		m.printStatus(dest, i, len(destinations), startTime)
+		m.printStatus(dest, i, len(destinations)-1, startTime)
 		if strings.TrimSpace(dest) == "" {
 			continue
 		}

Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,7 @@ func (m *mailer) run() error {`
`110`	`110`	`startTime := m.clk.Now()`
`111`	`111`
`112`	`112`	`for i, dest := range destinations {`
`113`		`- m.printStatus(dest, i, len(destinations), startTime)`
	`113`	`+ m.printStatus(dest, i, len(destinations)-1, startTime)`
`114`	`114`	`if strings.TrimSpace(dest) == "" {`
`115`	`115`	`continue`
`116`	`116`	`}`