Skip to content

Go linter to detect erroneous multiplication of duration values

License

Notifications You must be signed in to change notification settings

sylvia7788/durationcheck

 
 

Repository files navigation

CircleCI

Duration Check

A Go linter to detect cases where two time.Duration values are being multiplied in possibly erroneous ways.

For example, consider the following (highly contrived) function:

func waitFor(someDuration time.Duration) {
    timeToWait := someDuration * time.Second
    time.Sleep(timeToWait)
}

Although the above code would compile without any errors, its runtime behaviour would almost certainly be incorrect. A caller would reasonably expect waitFor(5 * time.Seconds) to wait for ~5 seconds but they would actually end up waiting for ~1,388,889 hours.

The above example is just for illustration purposes only. The problem is glaringly obvious in such a simple function and even the greenest Gopher would discover the issue immediately. However, imagine a much more complicated function with many more lines and it is not inconceivable that such logic errors could go unnoticed.

See the test cases for more examples of the types of errors detected by the linter.

Installation

Requires Go 1.11 or above.

go get -u github.com/charithe/durationcheck/cmd/durationcheck

Usage

Invoke durationcheck with your package name

durationcheck ./...
# or
durationcheck github.com/you/yourproject/...

About

Go linter to detect erroneous multiplication of duration values

Resources

License

Stars

Watchers

Forks

Packages

No packages published

Languages

  • Go 96.7%
  • Makefile 3.3%