golint or go vet; flagging case statements that are suspect

[Joshua Boelter]

I wrote a test for golint to detect switch case statements that may be suspect. I realize that this is perfectly normal in Go, it's also completely opposite C which has caught me a few times now.  As a style thing, I prefer to make it explicit w/ a break.  The pull request was closed as wontfix.

I ran this across a wide range of packages and didn't find any obvious failures, although a good number of false positives appear. They generally appear to be an intentional use of the lack of implicit fallthrough (at least when commented). Others are suspect.

(Note: I did not commit golint.go w/ a new package reference if you're testing this)

https://github.com/golang/lint/pull/72 

Any interest in keeping this alive or making a patch for go vet?

[Rob Pike]

No interest in adding support for this to the tools. It's just a
difference from C one must learn.

-rob

> --
> You received this message because you are subscribed to the Google Groups
> "golang-nuts" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to golang-nuts...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.