Report for github.com/gballet/go-verkle
(v0.0.0-20220201142824-b425ce18c8e8)
A+ Excellent! Found 10 issues across 18 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!
gocyclo94%
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.
golint44%
Golint is a linter for Go source code.
-
-
hashednode.go
- Line 33: warning: exported type HashedNode should have comment or be unexported (golint)
- Line 38: warning: exported method HashedNode.Insert should have comment or be unexported (golint)
- Line 42: warning: exported method HashedNode.InsertOrdered should have comment or be unexported (golint)
- Line 46: warning: exported method HashedNode.Delete should have comment or be unexported (golint)
- Line 50: warning: exported method HashedNode.Get should have comment or be unexported (golint)
- Line 54: warning: exported method HashedNode.ComputeCommitment should have comment or be unexported (golint)
- Line 58: warning: exported method HashedNode.GetCommitmentsAlongPath should have comment or be unexported (golint)
- Line 62: warning: exported method HashedNode.Serialize should have comment or be unexported (golint)
- Line 66: warning: exported method HashedNode.Copy should have comment or be unexported (golint)
-
-
ipa.go
- Line 33: warning: exported type Fr should have comment or be unexported (golint)
- Line 34: warning: exported type Point should have comment or be unexported (golint)
- Line 36: warning: exported function CopyFr should have comment or be unexported (golint)
- Line 40: warning: exported function CopyPoint should have comment or be unexported (golint)
- Line 54: warning: exported function FromLEBytes should have comment or be unexported (golint)
- Line 61: warning: exported function FromBytes should have comment or be unexported (golint)
- Line 65: warning: exported function Equal should have comment or be unexported (golint)
- Line 69: warning: exported function Generator should have comment or be unexported (golint)
-
-
config_ipa.go
- Line 33: warning: exported type IPAConfig should have comment or be unexported (golint)
- Line 37: warning: exported type Config should have comment or be unexported (golint)
- Line 39: warning: exported method IPAConfig.CommitToPoly should have comment or be unexported (golint)
- Line 46: warning: exported function GetConfig should have comment or be unexported (golint)
- Line 53: warning: exported function GetConfigWithPrecomputed should have comment or be unexported (golint)
- Line 60: warning: exported var FrZero should have comment or be unexported (golint)
- Line 61: warning: exported var FrOne should have comment or be unexported (golint)
-
-
empty.go
- Line 30: warning: exported type Empty should have comment or be unexported (golint)
- Line 32: warning: exported method Empty.Insert should have comment or be unexported (golint)
- Line 36: warning: exported method Empty.InsertOrdered should have comment or be unexported (golint)
- Line 40: warning: exported method Empty.Delete should have comment or be unexported (golint)
- Line 44: warning: exported method Empty.Get should have comment or be unexported (golint)
- Line 48: warning: exported method Empty.ComputeCommitment should have comment or be unexported (golint)
- Line 52: warning: exported method Empty.GetCommitmentsAlongPath should have comment or be unexported (golint)
- Line 56: warning: exported method Empty.Serialize should have comment or be unexported (golint)
- Line 60: warning: exported method Empty.Copy should have comment or be unexported (golint)
-
-
encoding.go
- Line 33: warning: exported var ErrInvalidNodeEncoding should have comment or be unexported (golint)
- Line 44: warning: error var serializedPayloadTooShort should have name of the form errFoo (golint)
- Line 46: warning: exported function ParseNode should have comment or be unexported (golint)
- Line 81: warning: exported function CreateInternalNode should have comment or be unexported (golint)
-
-
proof_ipa.go
- Line 38: warning: exported type Proof should have comment or be unexported (golint)
- Line 47: warning: exported function MakeVerkleProofOneLeaf should have comment or be unexported (golint)
- Line 67: warning: exported function GetCommitmentsForMultiproof should have comment or be unexported (golint)
- Line 95: warning: exported function MakeVerkleMultiProof should have comment or be unexported (golint)
- Line 138: warning: exported function VerifyVerkleProof should have comment or be unexported (golint)
- Line 143: warning: comment on exported type KeyValuePair should be of the form "KeyValuePair ..." (with optional leading article) (golint)
- Line 194: warning: comment on exported function DeserializeProof should be of the form "DeserializeProof ..." (golint)
-
-
stateless.go
- Line 63: warning: exported function NewStateless should have comment or be unexported (golint)
- Line 72: warning: exported function NewStatelessWithCommitment should have comment or be unexported (golint)
- Line 85: warning: exported method StatelessNode.Children should have comment or be unexported (golint)
- Line 97: warning: exported method StatelessNode.SetChild should have comment or be unexported (golint)
- Line 109: warning: exported method StatelessNode.SetStem should have comment or be unexported (golint)
- Line 113: warning: exported method StatelessNode.Insert should have comment or be unexported (golint)
- Line 225: warning: exported method StatelessNode.InsertOrdered should have comment or be unexported (golint)
- Line 229: warning: exported method StatelessNode.Delete should have comment or be unexported (golint)
- Line 257: warning: exported method StatelessNode.Get should have comment or be unexported (golint)
- Line 271: warning: exported method StatelessNode.ComputeCommitment should have comment or be unexported (golint)
- Line 331: warning: exported method StatelessNode.GetCommitmentsAlongPath should have comment or be unexported (golint)
- Line 335: warning: exported method StatelessNode.Serialize should have comment or be unexported (golint)
- Line 339: warning: exported method StatelessNode.Copy should have comment or be unexported (golint)
-
-
tree.go
- Line 33: warning: exported type NodeFlushFn should have comment or be unexported (golint)
- Line 34: warning: exported type NodeResolverFn should have comment or be unexported (golint)
- Line 42: warning: exported type VerkleNode should have comment or be unexported (golint)
- Line 142: warning: comment on exported type InternalNode should be of the form "InternalNode ..." (with optional leading article) (golint)
- Line 164: warning: exported type LeafNode should have comment or be unexported (golint)
- Line 194: warning: exported method InternalNode.Children should have comment or be unexported (golint)
- Line 198: warning: exported method InternalNode.SetChild should have comment or be unexported (golint)
- Line 206: warning: exported method InternalNode.Insert should have comment or be unexported (golint)
- Line 288: warning: exported method InternalNode.InsertOrdered should have comment or be unexported (golint)
- Line 391: warning: exported method InternalNode.Delete should have comment or be unexported (golint)
- Line 428: warning: exported method InternalNode.Get should have comment or be unexported (golint)
- Line 464: warning: exported method InternalNode.ComputeCommitment should have comment or be unexported (golint)
- Line 505: warning: exported method InternalNode.GetCommitmentsAlongPath should have comment or be unexported (golint)
- Line 539: warning: exported method InternalNode.Serialize should have comment or be unexported (golint)
- Line 552: warning: exported method InternalNode.Copy should have comment or be unexported (golint)
- Line 608: warning: exported method LeafNode.Insert should have comment or be unexported (golint)
- Line 619: warning: exported method LeafNode.InsertOrdered should have comment or be unexported (golint)
- Line 626: warning: exported method LeafNode.Delete should have comment or be unexported (golint)
- Line 639: warning: exported method LeafNode.Get should have comment or be unexported (golint)
- Line 651: warning: exported method LeafNode.ComputeCommitment should have comment or be unexported (golint)
- Line 712: warning: exported method LeafNode.GetCommitmentsAlongPath should have comment or be unexported (golint)
- Line 812: warning: exported method LeafNode.Serialize should have comment or be unexported (golint)
- Line 828: warning: exported method LeafNode.Copy should have comment or be unexported (golint)
- Line 849: warning: exported method LeafNode.Key should have comment or be unexported (golint)
- Line 856: warning: exported method LeafNode.Value should have comment or be unexported (golint)
- Line 874: warning: exported function ToDot should have comment or be unexported (golint)
-
-
tree_ipa_test.go
- Line 131: warning: don't use underscores in Go names; var key_a should be keyA (golint)
- Line 135: warning: don't use underscores in Go names; var key_b should be keyB (golint)
- Line 181: warning: don't use underscores in Go names; var key_b should be keyB (golint)
ineffassign100%
IneffAssign detects ineffectual assignments in Go code.
No problems detected. Good job!