Go Report Card

Preparing report...

Report for github.com/deadsy/rvdbg

A+    Excellent!    Found 21 issues across 98 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!

gocyclo93%

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.

golint87%

Golint is a linter for Go source code.

    • rvdbg/chip/sifive/fe310/soc.go
    • Line 75: warning: exported type Variant should have comment or be unexported (golint)
    • Line 78: warning: exported const G000 should have comment (or a comment on this block) or be unexported (golint)
    • rvdbg/cpu/arm/cm/debug.go
    • Line 33: warning: exported type CmDebug should have comment or be unexported (golint)
    • Line 48: warning: exported method CmDebug.GetPrompt should have comment or be unexported (golint)
    • Line 55: warning: exported method CmDebug.RdReg should have comment or be unexported (golint)
    • Line 60: warning: exported method CmDebug.WrReg should have comment or be unexported (golint)
    • Line 65: warning: exported method CmDebug.GetAddressSize should have comment or be unexported (golint)
    • Line 69: warning: exported method CmDebug.RdMem should have comment or be unexported (golint)
    • Line 73: warning: exported method CmDebug.WrMem should have comment or be unexported (golint)
    • rvdbg/itf/itf.go
    • Line 29: warning: exported const TypeNone should have comment (or a comment on this block) or be unexported (golint)
    • Line 46: warning: exported type Info should have comment or be unexported (golint)
    • Line 86: warning: exported const ModeJtag should have comment (or a comment on this block) or be unexported (golint)
    • Line 111: warning: exported function NewJtagDriver should have comment or be unexported (golint)
    • Line 165: warning: exported function NewSwdDriver should have comment or be unexported (golint)
    • rvdbg/soc/device.go
    • Line 50: warning: comment on exported method Device.AddPeripheral should be of the form "AddPeripheral ..." (golint)
    • rvdbg/soc/register.go
    • Line 128: warning: exported method Register.Wr should have comment or be unexported (golint)
    • Line 132: warning: exported method Register.Rd should have comment or be unexported (golint)
    • rvdbg/cpu/riscv/cli.go
    • Line 256: warning: exported var CmdHart should have comment or be unexported (golint)
    • Line 271: warning: exported var DisassembleHelp should have comment or be unexported (golint)
    • rvdbg/cpu/arm/jtag_dp.go
    • Line 32: warning: don't use underscores in Go names; const dr_ABORT_LEN should be drABORTLEN (golint)
    • Line 33: warning: don't use underscores in Go names; const dr_DPACC_LEN should be drDPACCLEN (golint)
    • Line 34: warning: don't use underscores in Go names; const dr_APACC_LEN should be drAPACCLEN (golint)
    • Line 35: warning: don't use underscores in Go names; const dr_IDCODE_LEN should be drIDCODELEN (golint)
    • Line 36: warning: don't use underscores in Go names; const dr_BYPASS_LEN should be drBYPASSLEN (golint)
    • Line 49: warning: don't use underscores in Go names; const ack_OK_FAULT should be ackOKFAULT (golint)
    • Line 50: warning: don't use underscores in Go names; const ack_WAIT should be ackWAIT (golint)
    • Line 53: warning: don't use underscores in Go names; const dp_WR should be dpWR (golint)
    • Line 54: warning: don't use underscores in Go names; const dp_RD should be dpRD (golint)
    • Line 59: warning: don't use underscores in Go names; const dpacc_CTRL_STAT should be dpaccCTRLSTAT (golint)
    • Line 60: warning: don't use underscores in Go names; const dpacc_SELECT should be dpaccSELECT (golint)
    • Line 61: warning: don't use underscores in Go names; const dpacc_RDBUFF should be dpaccRDBUFF (golint)
    • Line 72: warning: don't use underscores in Go names; const abort_ORUNERRCLR should be abortORUNERRCLR (golint)
    • Line 73: warning: don't use underscores in Go names; const abort_WDERRCLR should be abortWDERRCLR (golint)
    • Line 74: warning: don't use underscores in Go names; const abort_STKERRCLR should be abortSTKERRCLR (golint)
    • Line 75: warning: don't use underscores in Go names; const abort_STKCMPCLR should be abortSTKCMPCLR (golint)
    • Line 76: warning: don't use underscores in Go names; const abort_DAPABORT should be abortDAPABORT (golint)
    • Line 81: warning: don't use underscores in Go names; const cs_ORUNDETECT should be csORUNDETECT (golint)
    • Line 82: warning: don't use underscores in Go names; const cs_STICKYORUN should be csSTICKYORUN (golint)
    • Line 85: warning: don't use underscores in Go names; const cs_STICKYCMP should be csSTICKYCMP (golint)
    • Line 86: warning: don't use underscores in Go names; const cs_STICKYERR should be csSTICKYERR (golint)
    • Line 87: warning: don't use underscores in Go names; const cs_READOK should be csREADOK (golint)
    • Line 88: warning: don't use underscores in Go names; const cs_WDATAERR should be csWDATAERR (golint)
    • Line 91: warning: don't use underscores in Go names; const cs_DBGRSTREQ should be csDBGRSTREQ (golint)
    • Line 92: warning: don't use underscores in Go names; const cs_DBGRSTACK should be csDBGRSTACK (golint)
    • Line 93: warning: don't use underscores in Go names; const cs_DBGPWRUPREQ should be csDBGPWRUPREQ (golint)
    • Line 94: warning: don't use underscores in Go names; const cs_DBGPWRUPACK should be csDBGPWRUPACK (golint)
    • Line 95: warning: don't use underscores in Go names; const cs_SYSPWRUPREQ should be csSYSPWRUPREQ (golint)
    • Line 96: warning: don't use underscores in Go names; const cs_SYSPWRUPACK should be csSYSPWRUPACK (golint)
    • Line 98: warning: don't use underscores in Go names; const cs_PWR_REQ should be csPWRREQ (golint)
    • Line 99: warning: don't use underscores in Go names; const cs_PWR_ACK should be csPWRACK (golint)
    • Line 100: warning: don't use underscores in Go names; const cs_ERR should be csERR (golint)
    • Line 229: warning: don't use underscores in Go names; method WrDPACC_Select should be WrDPACCSelect (golint)
    • rvdbg/cpu/riscv/rv/ins.go
    • Line 33: warning: don't use underscores in Go names; const opcodeFMV_X_W should be opcodeFMVXW (golint)
    • Line 34: warning: don't use underscores in Go names; const opcodeFMV_W_X should be opcodeFMVWX (golint)
    • Line 35: warning: don't use underscores in Go names; const opcodeFMV_D_X should be opcodeFMVDX (golint)
    • Line 36: warning: don't use underscores in Go names; const opcodeFMV_X_D should be opcodeFMVXD (golint)

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

misspell95%

Misspell Finds commonly misspelled English words