Preparing report...

Report for github.com/goki/pi

A    Great!    Found 48 issues across 73 files

Tweet

gofmt98%

Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command


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!


gocyclo78%

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.

    • pi/lex/passtwo.go
    • Line 188: warning: cyclomatic complexity 21 of function (*PassTwo).EosDetectPos() is high (> 15) (gocyclo)
    • pi/langs/golang/typeinfer.go
    • Line 28: warning: cyclomatic complexity 43 of function (*GoLang).InferSymbolType() is high (> 15) (gocyclo)
    • Line 192: warning: cyclomatic complexity 22 of function (*GoLang).InferForRangeSymbolType() is high (> 15) (gocyclo)
    • pi/langs/bibtex/yaccpar
    • Line 156: warning: cyclomatic complexity 59 of function (*bibtexParserImpl).Parse() is high (> 15) (gocyclo)
    • Line 53: warning: cyclomatic complexity 20 of function bibtexErrorMessage() is high (> 15) (gocyclo)
    • pi/langs/golang/types.go
    • Line 298: warning: cyclomatic complexity 35 of function (*GoLang).TypeFromAstComp() is high (> 15) (gocyclo)
    • Line 225: warning: cyclomatic complexity 20 of function (*GoLang).TypeFromAstPrim() is high (> 15) (gocyclo)
    • pi/langs/golang/complete.go
    • Line 372: warning: cyclomatic complexity 28 of function (*GoLang).CompleteAstStart() is high (> 15) (gocyclo)
    • Line 131: warning: cyclomatic complexity 24 of function (*GoLang).CompleteLine() is high (> 15) (gocyclo)
    • Line 26: warning: cyclomatic complexity 23 of function (*GoLang).Lookup() is high (> 15) (gocyclo)
    • Line 310: warning: cyclomatic complexity 16 of function (*GoLang).LookupString() is high (> 15) (gocyclo)
    • pi/lex/brace.go
    • Line 40: warning: cyclomatic complexity 23 of function BraceMatch() is high (> 15) (gocyclo)
    • pi/lex/rule.go
    • Line 137: warning: cyclomatic complexity 21 of function (*Rule).Validate() is high (> 15) (gocyclo)
    • Line 295: warning: cyclomatic complexity 20 of function (*Rule).IsMatch() is high (> 15) (gocyclo)
    • Line 360: warning: cyclomatic complexity 17 of function (*Rule).IsMatchPos() is high (> 15) (gocyclo)
    • Line 477: warning: cyclomatic complexity 16 of function (*Rule).WriteGrammar() is high (> 15) (gocyclo)
    • pi/langs/golang/expr.go
    • Line 31: warning: cyclomatic complexity 47 of function (*GoLang).TypeFromAstExpr() is high (> 15) (gocyclo)
    • Line 219: warning: cyclomatic complexity 35 of function (*GoLang).TypeFromAstType() is high (> 15) (gocyclo)
    • pi/spell/model.go
    • Line 590: warning: cyclomatic complexity 19 of function (*Model).suggestPotential() is high (> 15) (gocyclo)
    • pi/parse/rule.go
    • Line 1469: warning: cyclomatic complexity 69 of function (*Rule).DoAct() is high (> 15) (gocyclo)
    • Line 1234: warning: cyclomatic complexity 40 of function (*Rule).DoRules() is high (> 15) (gocyclo)
    • Line 1122: warning: cyclomatic complexity 30 of function (*Rule).MatchExclude() is high (> 15) (gocyclo)
    • Line 246: warning: cyclomatic complexity 29 of function (*Rule).Compile() is high (> 15) (gocyclo)
    • Line 938: warning: cyclomatic complexity 27 of function (*Rule).MatchMixed() is high (> 15) (gocyclo)
    • Line 529: warning: cyclomatic complexity 22 of function (*Rule).Validate() is high (> 15) (gocyclo)
    • Line 881: warning: cyclomatic complexity 18 of function (*Rule).MatchToken() is high (> 15) (gocyclo)
    • Line 757: warning: cyclomatic complexity 17 of function (*Rule).Match() is high (> 15) (gocyclo)
    • Line 674: warning: cyclomatic complexity 16 of function (*Rule).ParseRules() is high (> 15) (gocyclo)
    • pi/lex/state.go
    • Line 216: warning: cyclomatic complexity 22 of function (*State).ReadUntil() is high (> 15) (gocyclo)
    • Line 276: warning: cyclomatic complexity 19 of function (*State).ReadNumber() is high (> 15) (gocyclo)

golint42%

Golint is a linter for Go source code.

    • pi/filecat/filecat.go
    • Line 31: warning: comment on exported type Cat should be of the form "Cat ..." (with optional leading article) (golint)
    • Line 92: warning: exported const CatN should have comment (or a comment on this block) or be unexported (golint)
    • Line 97: warning: don't use underscores in Go names; var KiT_Cat should be KiTCat (golint)
    • Line 97: warning: exported var KiT_Cat should have comment or be unexported (golint)
    • Line 99: warning: exported method Cat.MarshalJSON should have comment or be unexported (golint)
    • Line 100: warning: exported method Cat.UnmarshalJSON should have comment or be unexported (golint)
    • Line 102: warning: comment on exported method Cat.MarshalText should be of the form "MarshalText ..." (golint)
    • Line 103: warning: receiver name ev should be consistent with previous receiver name kf for Cat (golint)
    • Line 104: warning: exported method Cat.UnmarshalText should have comment or be unexported (golint)
    • Line 104: warning: receiver name ev should be consistent with previous receiver name kf for Cat (golint)
    • Line 106: warning: comment on exported function CatFromMime should be of the form "CatFromMime ..." (golint)
    • pi/spell/model.go
    • Line 33: warning: exported const SpellDepthDefault should have comment (or a comment on this block) or be unexported (golint)
    • Line 38: warning: exported type Pair should have comment or be unexported (golint)
    • Line 43: warning: exported type Method should have comment or be unexported (golint)
    • Line 46: warning: exported const MethodIsWord should have comment (or a comment on this block) or be unexported (golint)
    • Line 81: warning: comment on exported type Autos should be of the form "Autos ..." (with optional leading article) (golint)
    • Line 123: warning: comment on exported function NewModel should be of the form "NewModel ..." (golint)
    • Line 129: warning: exported method Model.Init should have comment or be unexported (golint)
    • Line 246: warning: comment on exported method Model.SaveLight should be of the form "SaveLight ..." (golint)
    • Line 287: warning: comment on exported method Model.SetDepth should be of the form "SetDepth ..." (golint)
    • Line 295: warning: comment on exported method Model.SetThreshold should be of the form "SetThreshold ..." (golint)
    • Line 303: warning: comment on exported method Model.SetUseAutocomplete should be of the form "SetUseAutocomplete ..." (golint)
    • Line 314: warning: comment on exported method Model.SetDivergenceThreshold should be of the form "SetDivergenceThreshold ..." (golint)
    • Line 324: warning: comment on exported function Levenshtein should be of the form "Levenshtein ..." (golint)
    • Line 358: warning: comment on exported method Model.Train should be of the form "Train ..." (golint)
    • Line 366: warning: comment on exported method Model.SetCount should be of the form "SetCount ..." (golint)
    • Line 379: warning: comment on exported method Model.TrainWord should be of the form "TrainWord ..." (golint)
    • Line 460: warning: comment on exported method Model.EditsMulti should be of the form "EditsMulti ..." (golint)
    • Line 486: warning: don't use underscores in Go names; var total_set should be totalSet (golint)
    • Line 558: warning: comment on exported method Model.CheckKnown should be of the form "CheckKnown ..." (golint)
    • Line 655: warning: comment on exported method Model.Potentials should be of the form "Potentials ..." (golint)
    • Line 663: warning: comment on exported method Model.Suggestions should be of the form "Suggestions ..." (golint)
    • Line 675: warning: comment on exported method Model.SpellCheck should be of the form "SpellCheck ..." (golint)
    • Line 683: warning: comment on exported method Model.SpellCheckSuggestions should be of the form "SpellCheckSuggestions ..." (golint)
    • Line 691: warning: exported function SampleEnglish should have comment or be unexported (golint)
    • Line 739: warning: comment on exported method Model.Autocomplete should be of the form "Autocomplete ..." (golint)
    • pi/lex/indent.go
    • Line 44: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 103: warning: comment on exported function LastLexIgnoreComment should be of the form "LastLexIgnoreComment ..." (golint)
    • pi/lex/matches.go
    • Line 19: warning: don't use underscores in Go names; var KiT_Matches should be KiTMatches (golint)
    • Line 19: warning: exported var KiT_Matches should have comment or be unexported (golint)
    • Line 21: warning: exported method Matches.MarshalJSON should have comment or be unexported (golint)
    • Line 22: warning: exported method Matches.UnmarshalJSON should have comment or be unexported (golint)
    • Line 63: warning: don't use underscores in Go names; var KiT_MatchPos should be KiTMatchPos (golint)
    • Line 63: warning: exported var KiT_MatchPos should have comment or be unexported (golint)
    • Line 65: warning: exported method MatchPos.MarshalJSON should have comment or be unexported (golint)
    • Line 66: warning: exported method MatchPos.UnmarshalJSON should have comment or be unexported (golint)
    • Line 97: warning: exported function IsLetter should have comment or be unexported (golint)
    • Line 101: warning: exported function IsDigit should have comment or be unexported (golint)
    • Line 105: warning: exported function IsLetterOrDigit should have comment or be unexported (golint)
    • Line 109: warning: exported function IsWhiteSpace should have comment or be unexported (golint)
    • pi/langs/golang/expr.go
    • Line 73: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 312: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 345: warning: comment on exported method GoLang.TypeFromFuncCall should be of the form "TypeFromFuncCall ..." (golint)
    • pi/parse/rule.go
    • Line 28: warning: comment on exported var GuiActive should be of the form "GuiActive ..." (golint)
    • Line 54: warning: comment on exported type Rule should be of the form "Rule ..." (with optional leading article) (golint)
    • Line 82: warning: don't use underscores in Go names; var KiT_Rule should be KiTRule (golint)
    • Line 82: warning: exported var KiT_Rule should have comment or be unexported (golint)
    • Line 153: warning: exported method RuleEl.IsRule should have comment or be unexported (golint)
    • Line 157: warning: exported method RuleEl.IsToken should have comment or be unexported (golint)
    • Line 200: warning: exported method Rule.BaseIface should have comment or be unexported (golint)
    • Line 204: warning: exported method Rule.AsParseRule should have comment or be unexported (golint)
    • Line 1767: warning: exported var RuleProps should have comment or be unexported (golint)
    • pi/lex/errors.go
    • Line 24: warning: comment on exported type Error should be of the form "Error ..." (with optional leading article) (golint)
    • pi/langs/tex/complete.go
    • Line 16: warning: exported method TexLang.CompleteLine should have comment or be unexported (golint)
    • Line 50: warning: exported method TexLang.CompleteEdit should have comment or be unexported (golint)
    • pi/token/token.go
    • Line 31: warning: don't use underscores in Go names; var KiT_Tokens should be KiTTokens (golint)
    • Line 31: warning: exported var KiT_Tokens should have comment or be unexported (golint)
    • Line 33: warning: exported method Tokens.MarshalJSON should have comment or be unexported (golint)
    • Line 34: warning: exported method Tokens.UnmarshalJSON should have comment or be unexported (golint)
    • Line 36: warning: comment on exported method Tokens.MarshalText should be of the form "MarshalText ..." (golint)
    • Line 38: warning: exported method Tokens.UnmarshalText should have comment or be unexported (golint)
    • Line 52: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 57: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 62: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 67: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 72: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 76: warning: comment on exported method Tokens.InSubCat should be of the form "InSubCat ..." (golint)
    • Line 77: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 84: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 89: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 94: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 104: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 118: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 123: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 128: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 149: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 155: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 162: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 168: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 174: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 190: warning: exported var KeyTokenZero should have comment or be unexported (golint)
    • Line 237: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 256: warning: receiver name tk should be consistent with previous receiver name ev for Tokens (golint)
    • Line 525: warning: comment on exported var Cats should be of the form "Cats ..." (golint)
    • Line 538: warning: comment on exported var SubCats should be of the form "SubCats ..." (golint)
    • pi/pi/langsup.go
    • Line 24: warning: don't use underscores in Go names; var KiT_LangFlags should be KiTLangFlags (golint)
    • Line 24: warning: exported var KiT_LangFlags should have comment or be unexported (golint)
    • Line 26: warning: exported method LangFlags.MarshalJSON should have comment or be unexported (golint)
    • Line 27: warning: exported method LangFlags.UnmarshalJSON should have comment or be unexported (golint)
    • pi/syms/kinds.go
    • Line 23: warning: don't use underscores in Go names; var KiT_Kinds should be KiTKinds (golint)
    • Line 23: warning: exported var KiT_Kinds should have comment or be unexported (golint)
    • Line 25: warning: exported method Kinds.MarshalJSON should have comment or be unexported (golint)
    • Line 26: warning: exported method Kinds.UnmarshalJSON should have comment or be unexported (golint)
    • Line 28: warning: comment on exported method Kinds.MarshalText should be of the form "MarshalText ..." (golint)
    • Line 30: warning: exported method Kinds.UnmarshalText should have comment or be unexported (golint)
    • Line 48: warning: receiver name tk should be consistent with previous receiver name ev for Kinds (golint)
    • Line 53: warning: receiver name tk should be consistent with previous receiver name ev for Kinds (golint)
    • Line 58: warning: receiver name tk should be consistent with previous receiver name ev for Kinds (golint)
    • Line 63: warning: receiver name tk should be consistent with previous receiver name ev for Kinds (golint)
    • Line 68: warning: receiver name tk should be consistent with previous receiver name ev for Kinds (golint)
    • Line 73: warning: receiver name tk should be consistent with previous receiver name ev for Kinds (golint)
    • Line 77: warning: exported method Kinds.InCat should have comment or be unexported (golint)
    • Line 77: warning: receiver name tk should be consistent with previous receiver name ev for Kinds (golint)
    • Line 81: warning: exported method Kinds.InSubCat should have comment or be unexported (golint)
    • Line 81: warning: receiver name tk should be consistent with previous receiver name ev for Kinds (golint)
    • Line 85: warning: exported method Kinds.InSub2Cat should have comment or be unexported (golint)
    • Line 85: warning: receiver name tk should be consistent with previous receiver name ev for Kinds (golint)
    • Line 89: warning: exported method Kinds.IsPtr should have comment or be unexported (golint)
    • Line 89: warning: receiver name tk should be consistent with previous receiver name ev for Kinds (golint)
    • Line 93: warning: exported method Kinds.IsPrimitiveNonPtr should have comment or be unexported (golint)
    • Line 93: warning: receiver name tk should be consistent with previous receiver name ev for Kinds (golint)
    • Line 218: warning: comment on exported var Cats should be of the form "Cats ..." (golint)
    • Line 227: warning: comment on exported var SubCats should be of the form "SubCats ..." (golint)
    • Line 250: warning: comment on exported var Sub2Cats should be of the form "Sub2Cats ..." (golint)
    • Line 329: warning: exported var ReflectKindMap should have comment or be unexported (golint)
    • pi/syms/type.go
    • Line 114: warning: comment on exported method Type.ArgString should be of the form "ArgString ..." (golint)
    • Line 122: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 127: warning: comment on exported method Type.ReturnString should be of the form "ReturnString ..." (golint)
    • Line 208: warning: comment on exported method TypeEl.Clone should be of the form "Clone ..." (golint)
    • Line 256: warning: comment on exported method TypeEls.StringRange should be of the form "StringRange ..." (golint)
    • pi/lex/passtwo.go
    • Line 119: warning: comment on exported method PassTwo.NestDepth should be of the form "NestDepth ..." (golint)
    • Line 158: warning: comment on exported method PassTwo.NestDepthLine should be of the form "NestDepthLine ..." (golint)
    • Line 181: warning: comment on exported method PassTwo.EosDetect should be of the form "EosDetect ..." (golint)
    • Line 187: warning: comment on exported method PassTwo.EosDetectPos should be of the form "EosDetectPos ..." (golint)
    • pi/langs/golang/builtin.go
    • Line 16: warning: exported var BuiltinTypes should have comment or be unexported (golint)
    • Line 30: warning: exported method GoLang.CompleteBuiltins should have comment or be unexported (golint)
    • pi/parse/actions.go
    • Line 20: warning: don't use underscores in Go names; var KiT_Actions should be KiTActions (golint)
    • Line 20: warning: exported var KiT_Actions should have comment or be unexported (golint)
    • Line 22: warning: exported method Actions.MarshalJSON should have comment or be unexported (golint)
    • Line 23: warning: exported method Actions.UnmarshalJSON should have comment or be unexported (golint)
    • Line 85: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 123: warning: don't use underscores in Go names; var KiT_AstActs should be KiTAstActs (golint)
    • Line 123: warning: exported var KiT_AstActs should have comment or be unexported (golint)
    • Line 125: warning: exported method AstActs.MarshalJSON should have comment or be unexported (golint)
    • Line 126: warning: exported method AstActs.UnmarshalJSON should have comment or be unexported (golint)
    • pi/parse/state.go
    • Line 15: warning: comment on exported type State should be of the form "State ..." (with optional leading article) (golint)
    • pi/lsp/completions.go
    • Line 17: warning: don't use underscores in Go names; var KiT_CompletionKind should be KiTCompletionKind (golint)
    • Line 17: warning: exported var KiT_CompletionKind should have comment or be unexported (golint)
    • Line 19: warning: exported method CompletionKind.MarshalJSON should have comment or be unexported (golint)
    • Line 20: warning: exported method CompletionKind.UnmarshalJSON should have comment or be unexported (golint)
    • Line 22: warning: comment on exported method CompletionKind.MarshalText should be of the form "MarshalText ..." (golint)
    • Line 24: warning: exported method CompletionKind.UnmarshalText should have comment or be unexported (golint)
    • pi/complete/complete.go
    • Line 5: warning: package comment should be of the form "Package complete ..." (golint)
    • Line 5: warning: package comment should not have leading space (golint)
    • Line 76: warning: comment on exported function MatchSeedString should be of the form "MatchSeedString ..." (golint)
    • Line 187: warning: don't use underscores in Go names; var keep_looking should be keepLooking (golint)
    • Line 188: warning: don't use underscores in Go names; var new_seed should be newSeed (golint)
    • Line 189: warning: don't use underscores in Go names; var potential_seed should be potentialSeed (golint)
    • Line 190: warning: don't use underscores in Go names; var first_match should be firstMatch (golint)
    • pi/cmd/pi/pi.go
    • Line 21: warning: exported var Excludes should have comment or be unexported (golint)
    • Line 58: warning: exported function DoGoPath should have comment or be unexported (golint)
    • Line 69: warning: exported function DoGoRecursive should have comment or be unexported (golint)
    • pi/spell/spell.go
    • Line 31: warning: exported var Ignore should have comment or be unexported (golint)
    • Line 48: warning: exported function ResetLearnTime should have comment or be unexported (golint)
    • pi/lex/actions.go
    • Line 16: warning: don't use underscores in Go names; var KiT_Actions should be KiTActions (golint)
    • Line 16: warning: exported var KiT_Actions should have comment or be unexported (golint)
    • Line 18: warning: exported method Actions.MarshalJSON should have comment or be unexported (golint)
    • Line 19: warning: exported method Actions.UnmarshalJSON should have comment or be unexported (golint)
    • pi/langs/golang/golang.go
    • Line 36: warning: exported method GoLang.Parser should have comment or be unexported (golint)
    • Line 82: warning: exported method GoLang.LexLine should have comment or be unexported (golint)
    • Line 90: warning: exported method GoLang.ParseLine should have comment or be unexported (golint)
    • Line 99: warning: exported method GoLang.HiLine should have comment or be unexported (golint)
    • Line 118: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • pi/lsp/symbols.go
    • Line 22: warning: don't use underscores in Go names; var KiT_SymbolKind should be KiTSymbolKind (golint)
    • Line 22: warning: exported var KiT_SymbolKind should have comment or be unexported (golint)
    • Line 24: warning: exported method SymbolKind.MarshalJSON should have comment or be unexported (golint)
    • Line 25: warning: exported method SymbolKind.UnmarshalJSON should have comment or be unexported (golint)
    • Line 27: warning: comment on exported method SymbolKind.MarshalText should be of the form "MarshalText ..." (golint)
    • Line 29: warning: exported method SymbolKind.UnmarshalText should have comment or be unexported (golint)
    • pi/langs/markdown/markdown.go
    • Line 34: warning: exported method MarkdownLang.Parser should have comment or be unexported (golint)
    • Line 49: warning: exported method MarkdownLang.ParseFile should have comment or be unexported (golint)
    • Line 61: warning: exported method MarkdownLang.LexLine should have comment or be unexported (golint)
    • Line 69: warning: exported method MarkdownLang.ParseLine should have comment or be unexported (golint)
    • Line 74: warning: exported method MarkdownLang.HiLine should have comment or be unexported (golint)
    • Line 79: warning: exported method MarkdownLang.CompleteLine should have comment or be unexported (golint)
    • Line 105: warning: exported method MarkdownLang.CompleteEdit should have comment or be unexported (golint)
    • Line 139: warning: exported method MarkdownLang.ParseDir should have comment or be unexported (golint)
    • Line 144: warning: comment on exported var ListKeys should be of the form "ListKeys ..." (golint)
    • pi/langs/tex/tex.go
    • Line 32: warning: exported method TexLang.Parser should have comment or be unexported (golint)
    • Line 47: warning: exported method TexLang.ParseFile should have comment or be unexported (golint)
    • Line 59: warning: exported method TexLang.LexLine should have comment or be unexported (golint)
    • Line 67: warning: exported method TexLang.ParseLine should have comment or be unexported (golint)
    • Line 72: warning: exported method TexLang.HiLine should have comment or be unexported (golint)
    • Line 77: warning: exported method TexLang.ParseDir should have comment or be unexported (golint)
    • pi/parse/ast.go
    • Line 32: warning: don't use underscores in Go names; var KiT_Ast should be KiTAst (golint)
    • Line 32: warning: exported var KiT_Ast should have comment or be unexported (golint)
    • Line 114: warning: exported var AstProps should have comment or be unexported (golint)
    • pi/lex/file.go
    • Line 426: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • pi/lex/line.go
    • Line 23: warning: comment on exported method Line.AddLex should be of the form "AddLex ..." (golint)
    • pi/lex/state.go
    • Line 27: warning: comment on exported type State should be of the form "State ..." (with optional leading article) (golint)
    • Line 344: warning: exported function DigitVal should have comment or be unexported (golint)
    • Line 356: warning: exported method State.ScanMantissa should have comment or be unexported (golint)
    • Line 364: warning: exported method State.ReadQuoted should have comment or be unexported (golint)
    • pi/filecat/supported.go
    • Line 14: warning: comment on exported type Supported should be of the form "Supported ..." (with optional leading article) (golint)
    • Line 308: warning: don't use underscores in Go names; var KiT_Supported should be KiTSupported (golint)
    • Line 308: warning: exported var KiT_Supported should have comment or be unexported (golint)
    • Line 310: warning: exported method Supported.MarshalJSON should have comment or be unexported (golint)
    • Line 310: warning: receiver name kf should be consistent with previous receiver name sup for Supported (golint)
    • Line 311: warning: exported method Supported.UnmarshalJSON should have comment or be unexported (golint)
    • Line 311: warning: receiver name kf should be consistent with previous receiver name sup for Supported (golint)
    • Line 313: warning: comment on exported method Supported.MarshalText should be of the form "MarshalText ..." (golint)
    • Line 314: warning: receiver name ev should be consistent with previous receiver name sup for Supported (golint)
    • Line 315: warning: exported method Supported.UnmarshalText should have comment or be unexported (golint)
    • Line 315: warning: receiver name ev should be consistent with previous receiver name sup for Supported (golint)
    • pi/pi/filestate.go
    • Line 108: warning: comment on exported method FileState.PassTwoErrReport should be of the form "PassTwoErrReport ..." (golint)
    • Line 147: warning: comment on exported method FileState.ParseRuleString should be of the form "ParseRuleString ..." (golint)
    • pi/syms/symbol.go
    • Line 64: warning: don't use underscores in Go names; var KiT_Symbol should be KiTSymbol (golint)
    • Line 64: warning: exported var KiT_Symbol should have comment or be unexported (golint)
    • pi/lex/rule.go
    • Line 40: warning: comment on exported type Rule should be of the form "Rule ..." (with optional leading article) (golint)
    • Line 68: warning: don't use underscores in Go names; var KiT_Rule should be KiTRule (golint)
    • Line 68: warning: exported var KiT_Rule should have comment or be unexported (golint)
    • Line 70: warning: exported method Rule.BaseIface should have comment or be unexported (golint)
    • Line 74: warning: exported method Rule.AsLexRule should have comment or be unexported (golint)
    • Line 529: warning: exported var RuleProps should have comment or be unexported (golint)
    • pi/langs/golang/types.go
    • Line 17: warning: exported var TraceTypes should have comment or be unexported (golint)
    • Line 259: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 272: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 290: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • pi/pi/version.go
    • Line 6: warning: exported const Version should have comment (or a comment on this block) or be unexported (golint)
    • pi/parse/trace.go
    • Line 145: warning: don't use underscores in Go names; var KiT_Steps should be KiTSteps (golint)
    • Line 145: warning: exported var KiT_Steps should have comment or be unexported (golint)
    • Line 147: warning: exported method Steps.MarshalJSON should have comment or be unexported (golint)
    • Line 148: warning: exported method Steps.UnmarshalJSON should have comment or be unexported (golint)
    • pi/lex/lex.go
    • Line 24: warning: exported function NewLex should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign87%

IneffAssign detects ineffectual assignments in Go code.

    • pi/parse/rule.go
    • Line 290: warning: ineffectual assignment to nr (ineffassign)
    • Line 598: warning: ineffectual assignment to ok (ineffassign)
    • Line 675: warning: ineffectual assignment to ok (ineffassign)
    • Line 797: warning: ineffectual assignment to match (ineffassign)
    • Line 883: warning: ineffectual assignment to ok (ineffassign)
    • Line 1599: warning: ineffectual assignment to added (ineffassign)

misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!