rate: use time.Time.Equal instead of == According to the Time Type documentation, in most cases, it is better to use Equal instead of == Change-Id: I201de68ee163a06cf9ed49f7b77a2de031dbdd95 Reviewed-on: https://go-review.googlesource.com/c/time/+/703175 Reviewed-by: Michael Knyszek <mknyszek@google.com> Reviewed-by: Florian Lehner <lehner.florian86@gmail.com> Reviewed-by: Sean Liao <sean@liao.dev> Reviewed-by: Mark Freeman <markfreeman@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
diff --git a/rate/rate.go b/rate/rate.go index 794b2e3..563270c 100644 --- a/rate/rate.go +++ b/rate/rate.go
@@ -195,7 +195,7 @@ // update state r.lim.last = t r.lim.tokens = tokens - if r.timeToAct == r.lim.lastEvent { + if r.timeToAct.Equal(r.lim.lastEvent) { prevEvent := r.timeToAct.Add(r.limit.durationFromTokens(float64(-r.tokens))) if !prevEvent.Before(t) { r.lim.lastEvent = prevEvent