diff options
| author | Daniel Martà <mvdan@mvdan.cc> | 2017-08-18 16:05:33 +0100 |
|---|---|---|
| committer | Daniel Martà <mvdan@mvdan.cc> | 2017-08-18 18:45:05 +0000 |
| commit | 943dd0fe332ebc72a4b02b2a9b6e4344e4cda340 (patch) | |
| tree | 60f2431f25a2de8ad28080cf1845467a7cc2d6cf /src/cmd/trace/trace.go | |
| parent | 776c33ee5ed67a00bd05912bd15a7c8062daab88 (diff) | |
| download | go-943dd0fe332ebc72a4b02b2a9b6e4344e4cda340.tar.xz | |
cmd/*: remove negative uint checks
All of these are uints of different sizes, so checking >= 0 or < 0 are
effectively no-ops.
Found with staticcheck.
Change-Id: I16ac900eb7007bc8f9018b302136d42e483a4180
Reviewed-on: https://go-review.googlesource.com/56950
Reviewed-by: Matt Layher <mdlayher@gmail.com>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Matt Layher <mdlayher@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Diffstat (limited to 'src/cmd/trace/trace.go')
| -rw-r--r-- | src/cmd/trace/trace.go | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/cmd/trace/trace.go b/src/cmd/trace/trace.go index 30dde5ae26..7475cec94a 100644 --- a/src/cmd/trace/trace.go +++ b/src/cmd/trace/trace.go @@ -449,9 +449,6 @@ func generateTrace(params *traceParams) (ViewerData, error) { if setGStateErr != nil { return ctx.data, setGStateErr } - if ctx.gstates[gRunnable] < 0 || ctx.gstates[gRunning] < 0 || ctx.threadStats.insyscall < 0 { - return ctx.data, fmt.Errorf("invalid state after processing %v: runnable=%d running=%d insyscall=%d", ev, ctx.gstates[gRunnable], ctx.gstates[gRunning], ctx.threadStats.insyscall) - } // Ignore events that are from uninteresting goroutines // or outside of the interesting timeframe. |