diff options
author | Russ Cox <rsc@golang.org> | 2013-01-31 14:37:47 -0800 |
---|---|---|
committer | Russ Cox <rsc@golang.org> | 2013-01-31 14:37:47 -0800 |
commit | ff1ff9b1224f583c13d7cc1019a1b9e090b496a0 (patch) | |
tree | 66fca694777153cf051325d3e917bedc9a139e8e /src | |
parent | a12c3574e46c1abcb718c348619baa602fcba22b (diff) | |
download | go-ff1ff9b1224f583c13d7cc1019a1b9e090b496a0.tar.gz |
cmd/vet: add missing -all logic
R=golang-dev, cookieo9, bradfitz
CC=golang-dev
https://codereview.appspot.com/7260043
Diffstat (limited to 'src')
-rw-r--r-- | src/cmd/vet/main.go | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/cmd/vet/main.go b/src/cmd/vet/main.go index 2f254f3e0..22e307386 100644 --- a/src/cmd/vet/main.go +++ b/src/cmd/vet/main.go @@ -25,7 +25,8 @@ import ( var verbose = flag.Bool("v", false, "verbose") var exitCode = 0 -// Flags to control which checks to perform +// Flags to control which checks to perform. +// NOTE: Add new flags to the if statement at the top of func main too. var ( vetAll = flag.Bool("all", true, "check everything; disabled if any explicit check is requested") vetAtomic = flag.Bool("atomic", false, "check for common mistaken usages of the sync/atomic package") @@ -65,7 +66,7 @@ func main() { flag.Parse() // If a check is named explicitly, turn off the 'all' flag. - if *vetMethods || *vetPrintf || *vetStructTags || *vetUntaggedLiteral || *vetRangeLoops { + if *vetAtomic || *vetBuildTags || *vetMethods || *vetPrintf || *vetStructTags || *vetRangeLoops || *vetUntaggedLiteral { *vetAll = false } |