Report for github.com/k0swe/wsjtx-go
A+ Excellent! Found 3 issues across 7 files
gofmt100%
Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command
No problems detected. Good job!
go_vet100%
go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.
No problems detected. Good job!
gocyclo85%
Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.
-
-
wsjtx-go/parser.go
- Line 26: warning: cyclomatic complexity 21 of function parseMessage() is high (> 15) (gocyclo)
golint57%
Golint is a linter for Go source code.
-
-
wsjtx-go/server.go
- Line 14: warning: exported type Server should have comment or be unexported (golint)
-
-
wsjtx-go/messages.go
- Line 8: warning: comment on exported type HeartbeatMessage should be of the form "HeartbeatMessage ..." (with optional leading article) (golint)
- Line 29: warning: comment on exported type StatusMessage should be of the form "StatusMessage ..." (with optional leading article) (golint)
- Line 65: warning: comment on exported type DecodeMessage should be of the form "DecodeMessage ..." (with optional leading article) (golint)
- Line 91: warning: comment on exported type ClearMessage should be of the form "ClearMessage ..." (with optional leading article) (golint)
- Line 113: warning: comment on exported type ReplyMessage should be of the form "ReplyMessage ..." (with optional leading article) (golint)
- Line 140: warning: comment on exported type QsoLoggedMessage should be of the form "QsoLoggedMessage ..." (with optional leading article) (golint)
- Line 170: warning: comment on exported type CloseMessage should be of the form "CloseMessage ..." (with optional leading article) (golint)
- Line 184: warning: comment on exported type ReplayMessage should be of the form "ReplayMessage ..." (with optional leading article) (golint)
- Line 201: warning: comment on exported type HaltTxMessage should be of the form "HaltTxMessage ..." (with optional leading article) (golint)
- Line 217: warning: comment on exported type FreeTextMessage should be of the form "FreeTextMessage ..." (with optional leading article) (golint)
- Line 239: warning: comment on exported type WSPRDecodeMessage should be of the form "WSPRDecodeMessage ..." (with optional leading article) (golint)
- Line 263: warning: comment on exported type LocationMessage should be of the form "LocationMessage ..." (with optional leading article) (golint)
- Line 281: warning: comment on exported type LoggedAdifMessage should be of the form "LoggedAdifMessage ..." (with optional leading article) (golint)
- Line 297: warning: comment on exported type HighlightCallsignMessage should be of the form "HighlightCallsignMessage ..." (with optional leading article) (golint)
- Line 325: warning: comment on exported type SwitchConfigurationMessage should be of the form "SwitchConfigurationMessage ..." (with optional leading article) (golint)
- Line 341: warning: comment on exported type ConfigureMessage should be of the form "ConfigureMessage ..." (with optional leading article) (golint)
-
-
wsjtx-go/parser.go
- Line 19: warning: error var ParseError should have name of the form ErrFoo (golint)
- Line 19: warning: exported var ParseError should have comment or be unexported (golint)
- Line 20: warning: error var notEnoughBytes should have name of the form errFoo (golint)
ineffassign85%
IneffAssign detects ineffectual assignments in Go code.
-
-
wsjtx-go/parser.go
- Line 117: warning: ineffectual assignment to err (ineffassign)
- Line 118: warning: ineffectual assignment to err (ineffassign)
- Line 119: warning: ineffectual assignment to err (ineffassign)
- Line 127: warning: ineffectual assignment to err (ineffassign)
- Line 128: warning: ineffectual assignment to err (ineffassign)
- Line 129: warning: ineffectual assignment to err (ineffassign)
- Line 130: warning: ineffectual assignment to err (ineffassign)
- Line 131: warning: ineffectual assignment to err (ineffassign)
- Line 132: warning: ineffectual assignment to err (ineffassign)
- Line 133: warning: ineffectual assignment to err (ineffassign)
- Line 134: warning: ineffectual assignment to err (ineffassign)
- Line 135: warning: ineffectual assignment to err (ineffassign)
- Line 136: warning: ineffectual assignment to err (ineffassign)
- Line 137: warning: ineffectual assignment to err (ineffassign)
- Line 138: warning: ineffectual assignment to err (ineffassign)
- Line 139: warning: ineffectual assignment to err (ineffassign)
- Line 140: warning: ineffectual assignment to err (ineffassign)
- Line 141: warning: ineffectual assignment to err (ineffassign)
- Line 142: warning: ineffectual assignment to err (ineffassign)
- Line 143: warning: ineffectual assignment to err (ineffassign)
- Line 144: warning: ineffectual assignment to err (ineffassign)
- Line 145: warning: ineffectual assignment to err (ineffassign)
- Line 146: warning: ineffectual assignment to err (ineffassign)
- Line 147: warning: ineffectual assignment to err (ineffassign)
- Line 155: warning: ineffectual assignment to err (ineffassign)
- Line 156: warning: ineffectual assignment to err (ineffassign)
- Line 157: warning: ineffectual assignment to err (ineffassign)
- Line 158: warning: ineffectual assignment to err (ineffassign)
- Line 159: warning: ineffectual assignment to err (ineffassign)
- Line 160: warning: ineffectual assignment to err (ineffassign)
- Line 161: warning: ineffectual assignment to err (ineffassign)
- Line 162: warning: ineffectual assignment to err (ineffassign)
- Line 163: warning: ineffectual assignment to err (ineffassign)
- Line 178: warning: ineffectual assignment to err (ineffassign)
- Line 179: warning: ineffectual assignment to err (ineffassign)
- Line 180: warning: ineffectual assignment to err (ineffassign)
- Line 181: warning: ineffectual assignment to err (ineffassign)
- Line 182: warning: ineffectual assignment to err (ineffassign)
- Line 183: warning: ineffectual assignment to err (ineffassign)
- Line 184: warning: ineffectual assignment to err (ineffassign)
- Line 185: warning: ineffectual assignment to err (ineffassign)
- Line 186: warning: ineffectual assignment to err (ineffassign)
- Line 187: warning: ineffectual assignment to err (ineffassign)
- Line 188: warning: ineffectual assignment to err (ineffassign)
- Line 189: warning: ineffectual assignment to err (ineffassign)
- Line 190: warning: ineffectual assignment to err (ineffassign)
- Line 191: warning: ineffectual assignment to err (ineffassign)
- Line 192: warning: ineffectual assignment to err (ineffassign)
- Line 193: warning: ineffectual assignment to err (ineffassign)
- Line 208: warning: ineffectual assignment to err (ineffassign)
- Line 209: warning: ineffectual assignment to err (ineffassign)
- Line 210: warning: ineffectual assignment to err (ineffassign)
- Line 211: warning: ineffectual assignment to err (ineffassign)
- Line 212: warning: ineffectual assignment to err (ineffassign)
- Line 213: warning: ineffectual assignment to err (ineffassign)
- Line 214: warning: ineffectual assignment to err (ineffassign)
- Line 215: warning: ineffectual assignment to err (ineffassign)
- Line 216: warning: ineffectual assignment to err (ineffassign)
- Line 217: warning: ineffectual assignment to err (ineffassign)
- Line 225: warning: ineffectual assignment to err (ineffassign)
- Line 305: warning: ineffectual assignment to err (ineffassign)
- Line 307: warning: ineffectual assignment to err (ineffassign)