gofmt98%

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

• go-imap/backend/backendutil/search_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• go-imap/server/cmd_selected.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

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!

gocyclo86%

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.

• go-imap/backend/backendutil/body.go
  • Line 32: warning: cyclomatic complexity 22 of function FetchBodySection() is high (> 15) (gocyclo)

• go-imap/backend/backendutil/search_test.go
  • Line 364: warning: cyclomatic complexity 16 of function TestMatchIssue298Regression() is high (> 15) (gocyclo)

• go-imap/client/client.go
  • Line 361: warning: cyclomatic complexity 26 of function (*Client).handleUnilateral() is high (> 15) (gocyclo)
  • Line 180: warning: cyclomatic complexity 24 of function (*Client).execute() is high (> 15) (gocyclo)

• go-imap/message.go
  • Line 891: warning: cyclomatic complexity 38 of function (*BodyStructure).Parse() is high (> 15) (gocyclo)
  • Line 1049: warning: cyclomatic complexity 21 of function (*BodyStructure).Format() is high (> 15) (gocyclo)
  • Line 204: warning: cyclomatic complexity 20 of function (*Message).Parse() is high (> 15) (gocyclo)
  • Line 535: warning: cyclomatic complexity 16 of function (*BodyPartName).parse() is high (> 15) (gocyclo)
  • Line 369: warning: cyclomatic complexity 16 of function (*BodySectionName).parse() is high (> 15) (gocyclo)

• go-imap/server/conn.go
  • Line 267: warning: cyclomatic complexity 16 of function (*conn).serve() is high (> 15) (gocyclo)

• go-imap/read_test.go
  • Line 383: warning: cyclomatic complexity 20 of function TestReader_ReadList() is high (> 15) (gocyclo)
  • Line 137: warning: cyclomatic complexity 18 of function TestReader_ReadAtom() is high (> 15) (gocyclo)
  • Line 333: warning: cyclomatic complexity 16 of function TestReader_ReadFields() is high (> 15) (gocyclo)

• go-imap/backend/backendutil/bodystructure.go
  • Line 39: warning: cyclomatic complexity 17 of function FetchBodyStructure() is high (> 15) (gocyclo)

• go-imap/responses/select.go
  • Line 64: warning: cyclomatic complexity 18 of function (*Select).WriteTo() is high (> 15) (gocyclo)

• go-imap/server/server.go
  • Line 292: warning: cyclomatic complexity 20 of function (*Server).listenUpdates() is high (> 15) (gocyclo)

• go-imap/response.go
  • Line 14: warning: cyclomatic complexity 16 of function ReadResp() is high (> 15) (gocyclo)

• go-imap/backend/backendutil/search.go
  • Line 71: warning: cyclomatic complexity 50 of function Match() is high (> 15) (gocyclo)

• go-imap/backend/backendutil/flags.go
  • Line 8: warning: cyclomatic complexity 18 of function UpdateFlags() is high (> 15) (gocyclo)

• go-imap/utf7/decoder.go
  • Line 18: warning: cyclomatic complexity 18 of function (*decoder).Transform() is high (> 15) (gocyclo)

• go-imap/search.go
  • Line 90: warning: cyclomatic complexity 70 of function (*SearchCriteria).parseField() is high (> 15) (gocyclo)
  • Line 275: warning: cyclomatic complexity 31 of function (*SearchCriteria).Format() is high (> 15) (gocyclo)

• go-imap/read.go
  • Line 309: warning: cyclomatic complexity 21 of function (*Reader).ReadFields() is high (> 15) (gocyclo)
  • Line 181: warning: cyclomatic complexity 17 of function (*Reader).ReadAtom() is high (> 15) (gocyclo)

golint49%

Golint is a linter for Go source code.

• go-imap/commands/expunge.go
  • Line 10: warning: exported method Expunge.Command should have comment or be unexported (golint)
  • Line 14: warning: exported method Expunge.Parse should have comment or be unexported (golint)

• go-imap/server/cmd_noauth.go
  • Line 19: warning: exported type StartTLS should have comment or be unexported (golint)
  • Line 23: warning: exported method StartTLS.Handle should have comment or be unexported (golint)
  • Line 43: warning: exported method StartTLS.Upgrade should have comment or be unexported (golint)
  • Line 85: warning: exported type Login should have comment or be unexported (golint)
  • Line 89: warning: exported method Login.Handle should have comment or be unexported (golint)
  • Line 108: warning: exported type Authenticate should have comment or be unexported (golint)
  • Line 112: warning: exported method Authenticate.Handle should have comment or be unexported (golint)

• go-imap/message.go
  • Line 73: warning: exported function ParseParamList should have comment or be unexported (golint)
  • Line 97: warning: exported function FormatParamList should have comment or be unexported (golint)
  • Line 157: warning: comment on exported type Message should be of the form "Message ..." (with optional leading article) (golint)
  • Line 187: warning: comment on exported function NewMessage should be of the form "NewMessage ..." (golint)
  • Line 316: warning: exported method Message.Format should have comment or be unexported (golint)
  • Line 354: warning: comment on exported type BodySectionName should be of the form "BodySectionName ..." (with optional leading article) (golint)
  • Line 438: warning: exported method BodySectionName.FetchItem should have comment or be unexported (golint)
  • Line 522: warning: comment on exported type BodyPartName should be of the form "BodyPartName ..." (with optional leading article) (golint)
  • Line 652: warning: comment on exported type Address should be of the form "Address ..." (with optional leading article) (golint)
  • Line 717: warning: comment on exported function ParseAddressList should be of the form "ParseAddressList ..." (golint)
  • Line 731: warning: comment on exported function FormatAddressList should be of the form "FormatAddressList ..." (golint)
  • Line 746: warning: comment on exported type Envelope should be of the form "Envelope ..." (with optional leading article) (golint)
  • Line 841: warning: comment on exported type BodyStructure should be of the form "BodyStructure ..." (with optional leading article) (golint)
  • Line 891: warning: exported method BodyStructure.Parse should have comment or be unexported (golint)
  • Line 1049: warning: exported method BodyStructure.Format should have comment or be unexported (golint)

• go-imap/commands/delete.go
  • Line 15: warning: exported method Delete.Command should have comment or be unexported (golint)
  • Line 24: warning: exported method Delete.Parse should have comment or be unexported (golint)

• go-imap/responses/search.go
  • Line 9: warning: comment on exported type Search should be of the form "Search ..." (with optional leading article) (golint)
  • Line 15: warning: exported method Search.Handle should have comment or be unexported (golint)
  • Line 25: 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)
  • Line 33: warning: exported method Search.WriteTo should have comment or be unexported (golint)

• go-imap/backend/memory/user.go
  • Line 9: warning: exported type User should have comment or be unexported (golint)
  • Line 15: warning: exported method User.Username should have comment or be unexported (golint)
  • Line 19: warning: exported method User.ListMailboxes should have comment or be unexported (golint)
  • Line 30: warning: exported method User.GetMailbox should have comment or be unexported (golint)
  • Line 38: warning: exported method User.CreateMailbox should have comment or be unexported (golint)
  • Line 47: warning: exported method User.DeleteMailbox should have comment or be unexported (golint)
  • Line 59: warning: exported method User.RenameMailbox should have comment or be unexported (golint)
  • Line 80: warning: exported method User.Logout should have comment or be unexported (golint)

• go-imap/commands/authenticate.go
  • Line 29: warning: exported method Authenticate.Command should have comment or be unexported (golint)
  • Line 49: warning: exported method Authenticate.Parse should have comment or be unexported (golint)
  • Line 82: warning: exported method Authenticate.Handle should have comment or be unexported (golint)

• go-imap/commands/fetch.go
  • Line 16: warning: exported method Fetch.Command should have comment or be unexported (golint)
  • Line 28: warning: exported method Fetch.Parse should have comment or be unexported (golint)

• go-imap/commands/uid.go
  • Line 16: warning: exported method Uid.Command should have comment or be unexported (golint)
  • Line 28: warning: exported method Uid.Parse should have comment or be unexported (golint)

• go-imap/server/cmd_any.go
  • Line 10: warning: exported type Capability should have comment or be unexported (golint)
  • Line 14: warning: exported method Capability.Handle should have comment or be unexported (golint)
  • Line 19: warning: exported type Noop should have comment or be unexported (golint)
  • Line 23: warning: exported method Noop.Handle should have comment or be unexported (golint)
  • Line 35: warning: exported type Logout should have comment or be unexported (golint)
  • Line 39: warning: exported method Logout.Handle should have comment or be unexported (golint)

• go-imap/responses/fetch.go
  • Line 9: warning: comment on exported type Fetch should be of the form "Fetch ..." (with optional leading article) (golint)
  • Line 17: warning: exported method Fetch.Handle should have comment or be unexported (golint)
  • Line 61: warning: exported method Fetch.WriteTo should have comment or be unexported (golint)

• go-imap/commands/list.go
  • Line 19: warning: exported method List.Command should have comment or be unexported (golint)
  • Line 35: warning: exported method List.Parse should have comment or be unexported (golint)

• go-imap/commands/status.go
  • Line 17: warning: exported method Status.Command should have comment or be unexported (golint)
  • Line 31: warning: exported method Status.Parse should have comment or be unexported (golint)
  • Line 52: 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)

• go-imap/responses/capability.go
  • Line 7: warning: comment on exported type Capability should be of the form "Capability ..." (with optional leading article) (golint)
  • Line 13: warning: exported method Capability.WriteTo should have comment or be unexported (golint)

• go-imap/commands/append.go
  • Line 19: warning: exported method Append.Command should have comment or be unexported (golint)
  • Line 45: warning: exported method Append.Parse should have comment or be unexported (golint)

• go-imap/commands/capability.go
  • Line 10: warning: exported method Capability.Command should have comment or be unexported (golint)
  • Line 16: warning: exported method Capability.Parse should have comment or be unexported (golint)

• go-imap/server/cmd_auth.go
  • Line 17: warning: exported type Select should have comment or be unexported (golint)
  • Line 21: warning: exported method Select.Handle should have comment or be unexported (golint)
  • Line 70: warning: exported type Create should have comment or be unexported (golint)
  • Line 74: warning: exported method Create.Handle should have comment or be unexported (golint)
  • Line 83: warning: exported type Delete should have comment or be unexported (golint)
  • Line 87: warning: exported method Delete.Handle should have comment or be unexported (golint)
  • Line 96: warning: exported type Rename should have comment or be unexported (golint)
  • Line 100: warning: exported method Rename.Handle should have comment or be unexported (golint)
  • Line 109: warning: exported type Subscribe should have comment or be unexported (golint)
  • Line 113: warning: exported method Subscribe.Handle should have comment or be unexported (golint)
  • Line 127: warning: exported type Unsubscribe should have comment or be unexported (golint)
  • Line 131: warning: exported method Unsubscribe.Handle should have comment or be unexported (golint)
  • Line 145: warning: exported type List should have comment or be unexported (golint)
  • Line 149: warning: exported method List.Handle should have comment or be unexported (golint)
  • Line 203: warning: exported type Status should have comment or be unexported (golint)
  • Line 207: warning: exported method Status.Handle should have comment or be unexported (golint)
  • Line 234: warning: exported type Append should have comment or be unexported (golint)
  • Line 238: warning: exported method Append.Handle should have comment or be unexported (golint)

• go-imap/backend/memory/backend.go
  • Line 1: warning: package comment should be of the form "Package memory ..." (golint)
  • Line 12: warning: exported type Backend should have comment or be unexported (golint)
  • Line 16: warning: exported method Backend.Login should have comment or be unexported (golint)
  • Line 25: warning: exported function New should have comment or be unexported (golint)

• go-imap/commands/search.go
  • Line 17: warning: exported method Search.Command should have comment or be unexported (golint)
  • Line 30: warning: exported method Search.Parse should have comment or be unexported (golint)

• go-imap/search.go
  • Line 133: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 151: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 234: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 249: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• go-imap/write.go
  • Line 18: warning: comment on exported type RawString should be of the form "RawString ..." (with optional leading article) (golint)
  • Line 22: warning: exported type WriterTo should have comment or be unexported (golint)
  • Line 30: warning: comment on exported function FormatStringList should be of the form "FormatStringList ..." (golint)
  • Line 49: warning: comment on exported type Writer should be of the form "Writer ..." (with optional leading article) (golint)
  • Line 242: warning: exported method Writer.Flush should have comment or be unexported (golint)
  • Line 249: warning: exported function NewWriter should have comment or be unexported (golint)
  • Line 253: warning: exported function NewClientWriter should have comment or be unexported (golint)

• go-imap/commands/noop.go
  • Line 10: warning: exported method Noop.Command should have comment or be unexported (golint)
  • Line 16: warning: exported method Noop.Parse should have comment or be unexported (golint)

• go-imap/commands/logout.go
  • Line 10: warning: exported method Logout.Command should have comment or be unexported (golint)
  • Line 16: warning: exported method Logout.Parse should have comment or be unexported (golint)

• go-imap/server/server.go
  • Line 20: warning: comment on exported const MinAutoLogout should be of the form "MinAutoLogout ..." (golint)
  • Line 23: warning: comment on exported type Handler should be of the form "Handler ..." (with optional leading article) (golint)
  • Line 34: warning: comment on exported type Upgrader should be of the form "Upgrader ..." (with optional leading article) (golint)
  • Line 44: warning: comment on exported type HandlerFactory should be of the form "HandlerFactory ..." (with optional leading article) (golint)
  • Line 47: warning: comment on exported type SASLServerFactory should be of the form "SASLServerFactory ..." (with optional leading article) (golint)
  • Line 50: warning: comment on exported type Extension should be of the form "Extension ..." (with optional leading article) (golint)
  • Line 58: warning: comment on exported type ConnExtension should be of the form "ConnExtension ..." (with optional leading article) (golint)
  • Line 86: warning: comment on exported type Server should be of the form "Server ..." (with optional leading article) (golint)
  • Line 122: warning: comment on exported function New should be of the form "New ..." (golint)
  • Line 384: warning: comment on exported method Server.Close should be of the form "Close ..." (golint)
  • Line 406: warning: comment on exported method Server.EnableAuth should be of the form "EnableAuth ..." (golint)

• go-imap/internal/testutil.go
  • Line 3: warning: exported type MapListSorter should have comment or be unexported (golint)

• go-imap/read.go
  • Line 51: warning: comment on exported type StringReader should be of the form "StringReader ..." (with optional leading article) (golint)
  • Line 113: warning: comment on exported function ParseStringList should be of the form "ParseStringList ..." (golint)
  • Line 134: warning: comment on exported type Reader should be of the form "Reader ..." (with optional leading article) (golint)
  • Line 146: warning: exported method Reader.ReadSp should have comment or be unexported (golint)
  • Line 157: warning: exported method Reader.ReadCrlf should have comment or be unexported (golint)
  • Line 181: warning: exported method Reader.ReadAtom should have comment or be unexported (golint)
  • Line 227: warning: exported method Reader.ReadLiteral should have comment or be unexported (golint)
  • Line 271: warning: exported method Reader.ReadQuotedString should have comment or be unexported (golint)
  • Line 309: warning: exported method Reader.ReadFields should have comment or be unexported (golint)
  • Line 363: warning: exported method Reader.ReadList should have comment or be unexported (golint)
  • Line 388: warning: exported method Reader.ReadLine should have comment or be unexported (golint)
  • Line 399: warning: exported method Reader.ReadRespCode should have comment or be unexported (golint)
  • Line 445: warning: exported method Reader.ReadInfo should have comment or be unexported (golint)
  • Line 457: warning: exported function NewReader should have comment or be unexported (golint)
  • Line 461: warning: exported function NewServerReader should have comment or be unexported (golint)
  • Line 465: warning: exported type Parser should have comment or be unexported (golint)

• go-imap/command.go
  • Line 8: warning: comment on exported type Commander should be of the form "Commander ..." (with optional leading article) (golint)
  • Line 13: warning: comment on exported type Command should be of the form "Command ..." (with optional leading article) (golint)
  • Line 24: warning: comment on exported method Command.Command should be of the form "Command ..." (golint)
  • Line 29: warning: exported method Command.WriteTo should have comment or be unexported (golint)

• go-imap/mailbox.go
  • Line 12: warning: comment on exported const InboxName should be of the form "InboxName ..." (golint)
  • Line 71: warning: comment on exported const ImportantAttr should be of the form "ImportantAttr ..." (golint)
  • Line 76: warning: comment on exported type MailboxInfo should be of the form "MailboxInfo ..." (with optional leading article) (golint)
  • Line 191: warning: comment on exported type MailboxStatus should be of the form "MailboxStatus ..." (with optional leading article) (golint)
  • Line 226: warning: comment on exported function NewMailboxStatus should be of the form "NewMailboxStatus ..." (golint)
  • Line 240: warning: exported method MailboxStatus.Parse should have comment or be unexported (golint)
  • Line 248: 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)
  • Line 279: warning: exported method MailboxStatus.Format should have comment or be unexported (golint)
  • Line 300: warning: exported function FormatMailboxName should have comment or be unexported (golint)

• go-imap/responses/select.go
  • Line 9: warning: comment on exported type Select should be of the form "Select ..." (with optional leading article) (golint)
  • Line 14: warning: exported method Select.Handle should have comment or be unexported (golint)
  • Line 64: warning: exported method Select.WriteTo should have comment or be unexported (golint)

• go-imap/backend/memory/mailbox.go
  • Line 12: warning: exported var Delimiter should have comment or be unexported (golint)
  • Line 14: warning: exported type Mailbox should have comment or be unexported (golint)
  • Line 22: warning: exported method Mailbox.Name should have comment or be unexported (golint)
  • Line 26: warning: exported method Mailbox.Info should have comment or be unexported (golint)
  • Line 81: warning: exported method Mailbox.Status should have comment or be unexported (golint)
  • Line 105: warning: exported method Mailbox.SetSubscribed should have comment or be unexported (golint)
  • Line 110: warning: exported method Mailbox.Check should have comment or be unexported (golint)
  • Line 114: warning: exported method Mailbox.ListMessages should have comment or be unexported (golint)
  • Line 141: warning: exported method Mailbox.SearchMessages should have comment or be unexported (golint)
  • Line 162: warning: exported method Mailbox.CreateMessage should have comment or be unexported (golint)
  • Line 182: warning: exported method Mailbox.UpdateMessagesFlags should have comment or be unexported (golint)
  • Line 200: warning: exported method Mailbox.CopyMessages should have comment or be unexported (golint)
  • Line 225: warning: exported method Mailbox.Expunge should have comment or be unexported (golint)

• go-imap/commands/close.go
  • Line 10: warning: exported method Close.Command should have comment or be unexported (golint)
  • Line 16: warning: exported method Close.Parse should have comment or be unexported (golint)

• go-imap/literal.go
  • Line 7: warning: comment on exported type Literal should be of the form "Literal ..." (with optional leading article) (golint)

• go-imap/responses/expunge.go
  • Line 9: warning: comment on exported type Expunge should be of the form "Expunge ..." (with optional leading article) (golint)
  • Line 15: warning: exported method Expunge.Handle should have comment or be unexported (golint)
  • Line 34: warning: exported method Expunge.WriteTo should have comment or be unexported (golint)

• go-imap/conn.go
  • Line 11: warning: comment on exported type ConnState should be of the form "ConnState ..." (with optional leading article) (golint)
  • Line 16: warning: comment on exported const ConnectingState should be of the form "ConnectingState ..." (golint)
  • Line 20: warning: comment on exported const NotAuthenticatedState should be of the form "NotAuthenticatedState ..." (golint)
  • Line 26: warning: comment on exported const AuthenticatedState should be of the form "AuthenticatedState ..." (golint)
  • Line 34: warning: comment on exported const SelectedState should be of the form "SelectedState ..." (golint)
  • Line 39: warning: comment on exported const LogoutState should be of the form "LogoutState ..." (golint)
  • Line 50: warning: comment on exported type ConnUpgrader should be of the form "ConnUpgrader ..." (with optional leading article) (golint)
  • Line 56: warning: exported type Waiter should have comment or be unexported (golint)
  • Line 62: warning: exported function NewWaiter should have comment or be unexported (golint)
  • Line 69: warning: exported method Waiter.Wait should have comment or be unexported (golint)
  • Line 79: warning: exported method Waiter.WaitReady should have comment or be unexported (golint)
  • Line 86: warning: exported method Waiter.Close should have comment or be unexported (golint)
  • Line 93: warning: exported type LockedWriter should have comment or be unexported (golint)
  • Line 139: warning: comment on exported type ConnInfo should be of the form "ConnInfo ..." (with optional leading article) (golint)
  • Line 148: warning: comment on exported type Conn should be of the form "Conn ..." (with optional leading article) (golint)
  • Line 226: warning: exported method Conn.Info should have comment or be unexported (golint)
  • Line 268: warning: comment on exported method Conn.Wait should be of the form "Wait ..." (golint)
  • Line 273: warning: comment on exported method Conn.WaitReady should be of the form "WaitReady ..." (golint)

• go-imap/server/cmd_selected.go
  • Line 17: warning: comment on exported type UidHandler should be of the form "UidHandler ..." (with optional leading article) (golint)
  • Line 25: warning: exported type Check should have comment or be unexported (golint)
  • Line 29: warning: exported method Check.Handle should have comment or be unexported (golint)
  • Line 41: warning: exported type Close should have comment or be unexported (golint)
  • Line 45: warning: exported method Close.Handle should have comment or be unexported (golint)
  • Line 59: warning: exported type Expunge should have comment or be unexported (golint)
  • Line 63: warning: exported method Expunge.Handle should have comment or be unexported (golint)
  • Line 124: warning: exported type Search should have comment or be unexported (golint)
  • Line 143: warning: exported method Search.Handle should have comment or be unexported (golint)
  • Line 147: warning: exported method Search.UidHandle should have comment or be unexported (golint)
  • Line 151: warning: exported type Fetch should have comment or be unexported (golint)
  • Line 168: warning: should omit values from range; this loop is equivalent to `for range ...` (golint)
  • Line 180: warning: exported method Fetch.Handle should have comment or be unexported (golint)
  • Line 184: warning: exported method Fetch.UidHandle should have comment or be unexported (golint)
  • Line 200: warning: exported type Store should have comment or be unexported (golint)
  • Line 268: warning: exported method Store.Handle should have comment or be unexported (golint)
  • Line 272: warning: exported method Store.UidHandle should have comment or be unexported (golint)
  • Line 276: warning: exported type Copy should have comment or be unexported (golint)
  • Line 289: warning: exported method Copy.Handle should have comment or be unexported (golint)
  • Line 293: warning: exported method Copy.UidHandle should have comment or be unexported (golint)
  • Line 297: warning: exported type Uid should have comment or be unexported (golint)
  • Line 301: warning: exported method Uid.Handle should have comment or be unexported (golint)

• go-imap/commands/store.go
  • Line 17: warning: exported method Store.Command should have comment or be unexported (golint)
  • Line 24: warning: exported method Store.Parse should have comment or be unexported (golint)
  • Line 40: 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)

• go-imap/responses/status.go
  • Line 12: warning: comment on exported type Status should be of the form "Status ..." (with optional leading article) (golint)
  • Line 18: warning: exported method Status.Handle should have comment or be unexported (golint)
  • Line 48: warning: exported method Status.WriteTo should have comment or be unexported (golint)

• go-imap/commands/check.go
  • Line 10: warning: exported method Check.Command should have comment or be unexported (golint)
  • Line 16: warning: exported method Check.Parse should have comment or be unexported (golint)

• go-imap/commands/copy.go
  • Line 16: warning: exported method Copy.Command should have comment or be unexported (golint)
  • Line 25: warning: exported method Copy.Parse should have comment or be unexported (golint)

• go-imap/commands/select.go
  • Line 17: warning: exported method Select.Command should have comment or be unexported (golint)
  • Line 31: warning: exported method Select.Parse should have comment or be unexported (golint)

• go-imap/seqset_test.go
  • Line 353: warning: don't use underscores in Go names; var test_ok should be testOk (golint)
  • Line 542: warning: don't use underscores in Go names; var test_empty should be testEmpty (golint)
  • Line 546: warning: don't use underscores in Go names; var test_dynamic should be testDynamic (golint)

• go-imap/responses/authenticate.go
  • Line 10: warning: comment on exported type Authenticate should be of the form "Authenticate ..." (with optional leading article) (golint)
  • Line 17: warning: comment on exported method Authenticate.Replies should be of the form "Replies ..." (golint)
  • Line 31: warning: exported method Authenticate.Handle should have comment or be unexported (golint)

• go-imap/commands/login.go
  • Line 15: warning: exported method Login.Command should have comment or be unexported (golint)
  • Line 22: warning: exported method Login.Parse should have comment or be unexported (golint)

• go-imap/responses/responses.go
  • Line 1: warning: package comment should be of the form "Package responses ..." (golint)

• go-imap/response.go
  • Line 119: warning: exported method DataResp.WriteTo should have comment or be unexported (golint)
  • Line 138: warning: exported method ContinuationReq.WriteTo should have comment or be unexported (golint)

• go-imap/status.go
  • Line 7: warning: comment on exported type StatusRespType should be of the form "StatusRespType ..." (with optional leading article) (golint)
  • Line 42: warning: exported type StatusRespCode should have comment or be unexported (golint)
  • Line 59: warning: comment on exported type StatusResp should be of the form "StatusResp ..." (with optional leading article) (golint)
  • Line 77: warning: comment on exported method StatusResp.Err should be of the form "Err ..." (golint)
  • Line 91: warning: exported method StatusResp.WriteTo should have comment or be unexported (golint)

• go-imap/responses/list.go
  • Line 12: warning: comment on exported type List should be of the form "List ..." (with optional leading article) (golint)
  • Line 20: warning: exported method List.Name should have comment or be unexported (golint)
  • Line 23: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 28: warning: exported method List.Handle should have comment or be unexported (golint)
  • Line 43: warning: exported method List.WriteTo should have comment or be unexported (golint)

• go-imap/commands/rename.go
  • Line 16: warning: exported method Rename.Command should have comment or be unexported (golint)
  • Line 27: warning: exported method Rename.Parse should have comment or be unexported (golint)

• go-imap/commands/starttls.go
  • Line 10: warning: exported method StartTLS.Command should have comment or be unexported (golint)
  • Line 16: warning: exported method StartTLS.Parse should have comment or be unexported (golint)

• go-imap/backend/memory/message.go
  • Line 15: warning: exported type Message should have comment or be unexported (golint)
  • Line 33: warning: exported method Message.Fetch should have comment or be unexported (golint)
  • Line 71: warning: exported method Message.Match should have comment or be unexported (golint)

• go-imap/commands/subscribe.go
  • Line 15: warning: exported method Subscribe.Command should have comment or be unexported (golint)
  • Line 24: warning: exported method Subscribe.Parse should have comment or be unexported (golint)
  • Line 37: warning: comment on exported type Unsubscribe should be of the form "Unsubscribe ..." (with optional leading article) (golint)
  • Line 43: warning: exported method Unsubscribe.Command should have comment or be unexported (golint)
  • Line 52: warning: exported method Unsubscribe.Parse should have comment or be unexported (golint)

• go-imap/client/client.go
  • Line 156: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 241: 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)
  • Line 610: warning: exported type Dialer should have comment or be unexported (golint)

• go-imap/date.go
  • Line 26: warning: comment on exported type Date should be of the form "Date ..." (with optional leading article) (golint)
  • Line 26: warning: comment on exported type DateTime should be of the form "DateTime ..." (with optional leading article) (golint)

• go-imap/imap.go
  • Line 15: warning: exported const StatusMessages should have comment (or a comment on this block) or be unexported (golint)
  • Line 102: warning: comment on exported var CharsetReader should be of the form "CharsetReader ..." (golint)

• go-imap/commands/create.go
  • Line 15: warning: exported method Create.Command should have comment or be unexported (golint)
  • Line 24: warning: exported method Create.Parse should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign99%

IneffAssign detects ineffectual assignments in Go code.

• go-imap/message.go
  • Line 959: warning: ineffectual assignment to end (ineffassign)
  • Line 1042: warning: ineffectual assignment to end (ineffassign)

misspell98%

Misspell Finds commonly misspelled English words

• go-imap/client/example_test.go
  • Line 61: warning: "substract" is a misspelling of "subtract" (misspell)

• go-imap/client/client.go
  • Line 213: warning: "occured" is a misspelling of "occurred" (misspell)