Go Report Card

Preparing report...

Report for github.com/asap2Go/calibrationReader

(v0.9.3)

A+    Excellent!    Found 50 issues across 212 files

Tweet
Last refresh:

Refresh now

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!

golint88%

Golint is a linter for Go source code.

    • srec19/record.go
    • Line 37: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 115: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • ihex32/record.go
    • Line 49: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 148: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • a2l/enums.go
    • Line 11: warning: exported const UTF8 should have comment (or a comment on this block) or be unexported (golint)
    • Line 36: warning: exported const UBYTE should have comment (or a comment on this block) or be unexported (golint)
    • Line 82: warning: exported const BYTE should have comment (or a comment on this block) or be unexported (golint)
    • Line 108: warning: exported const PBYTE should have comment (or a comment on this block) or be unexported (golint)
    • Line 111: warning: comment on exported const DIRECT should be of the form "DIRECT ..." (golint)
    • Line 212: warning: exported const INTERN should have comment (or a comment on this block) or be unexported (golint)
    • Line 272: warning: exported const Ascii should have comment (or a comment on this block) or be unexported (golint)
    • Line 318: warning: exported const Identical should have comment (or a comment on this block) or be unexported (golint)
    • Line 358: warning: exported const ColumnDir should have comment (or a comment on this block) or be unexported (golint)
    • Line 386: warning: exported const Absolute should have comment (or a comment on this block) or be unexported (golint)
    • calib_data_test.go
    • Line 1: warning: don't use MixedCaps in package name; calibrationReader should be calibrationreader (golint)
    • a2l/if_data.go
    • Line 9: warning: exported type IfData should have comment or be unexported (golint)
    • a2l/number.go
    • Line 9: warning: exported type Number should have comment or be unexported (golint)
    • calib_data.go
    • Line 9: warning: don't use MixedCaps in package name; calibrationReader should be calibrationreader (golint)
    • a2l/token_generator.go
    • Line 24: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 33: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 186: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 279: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 317: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • a2l/formula.go
    • Line 9: warning: exported type Formula should have comment or be unexported (golint)
    • a2l/project.go
    • Line 9: warning: exported type Project should have comment or be unexported (golint)

gocyclo87%

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.

    • a2l/module.go
    • Line 45: warning: cyclomatic complexity 58 of function parseModule() is high (> 15) (gocyclo)
    • Line 704: warning: cyclomatic complexity 58 of function parseModuleMainLoop() is high (> 15) (gocyclo)
    • Line 451: warning: cyclomatic complexity 28 of function collectChannelsMultithreaded() is high (> 15) (gocyclo)
    • a2l/blob.go
    • Line 33: warning: cyclomatic complexity 28 of function parseBlob() is high (> 15) (gocyclo)
    • a2l/axis_pts.go
    • Line 72: warning: cyclomatic complexity 52 of function parseAxisPts() is high (> 15) (gocyclo)
    • a2l/mod_par.go
    • Line 30: warning: cyclomatic complexity 37 of function parseModPar() is high (> 15) (gocyclo)
    • a2l/group.go
    • Line 23: warning: cyclomatic complexity 20 of function parseGroup() is high (> 15) (gocyclo)
    • a2l/function.go
    • Line 26: warning: cyclomatic complexity 26 of function parseFunction() is high (> 15) (gocyclo)
    • ihex32/hex.go
    • Line 101: warning: cyclomatic complexity 18 of function parseHex() is high (> 15) (gocyclo)
    • a2l/instance.go
    • Line 32: warning: cyclomatic complexity 32 of function parseInstance() is high (> 15) (gocyclo)
    • a2l/overwrite.go
    • Line 24: warning: cyclomatic complexity 19 of function parseOverwrite() is high (> 15) (gocyclo)
    • srec19/hex.go
    • Line 101: warning: cyclomatic complexity 17 of function parseHex() is high (> 15) (gocyclo)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!

misspell97%

Misspell Finds commonly misspelled English words

    • a2l/axis_rescale_x.go
    • Line 50: warning: "adressing" is a misspelling of "addressing" (misspell)
    • Line 103: warning: "adressing" is a misspelling of "addressing" (misspell)
    • Line 107: warning: "adressing" is a misspelling of "addressing" (misspell)