gofmt78%

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

• goofys/internal/backend.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• goofys/internal/backend_adlv2.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• goofys/internal/backend_azblob.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• goofys/internal/backend_s3.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• goofys/internal/goofys_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• goofys/internal/handles.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• goofys/example/test_api.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!

gocyclo68%

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.

• goofys/internal/flags.go
  • Line 320: warning: cyclomatic complexity 23 of function PopulateFlags() is high (> 15) (gocyclo)

• goofys/internal/file.go
  • Line 496: warning: cyclomatic complexity 16 of function (*FileHandle).readFile() is high (> 15) (gocyclo)
  • Line 708: warning: cyclomatic complexity 16 of function (*FileHandle).FlushFile() is high (> 15) (gocyclo)

• goofys/main.go
  • Line 136: warning: cyclomatic complexity 16 of function main() is high (> 15) (gocyclo)

• goofys/internal/backend_adlv2.go
  • Line 925: warning: cyclomatic complexity 31 of function (adl2PathClient).CreatePreparer() is high (> 15) (gocyclo)
  • Line 1623: warning: cyclomatic complexity 27 of function (adl2PathClient).UpdatePreparer() is high (> 15) (gocyclo)
  • Line 250: warning: cyclomatic complexity 20 of function mapADLv2Error() is high (> 15) (gocyclo)
  • Line 413: warning: cyclomatic complexity 16 of function (*ADLv2).ListBlobs() is high (> 15) (gocyclo)

• goofys/internal/dir.go
  • Line 1325: warning: cyclomatic complexity 29 of function (*Inode).LookUpInodeMaybeDir() is high (> 15) (gocyclo)
  • Line 432: warning: cyclomatic complexity 26 of function (*DirHandle).ReadDir() is high (> 15) (gocyclo)
  • Line 111: warning: cyclomatic complexity 21 of function (*Inode).OpenDir() is high (> 15) (gocyclo)
  • Line 206: warning: cyclomatic complexity 18 of function (*DirHandle).listObjectsSlurp() is high (> 15) (gocyclo)

• goofys/internal/goofys_test.go
  • Line 462: warning: cyclomatic complexity 19 of function (*GoofysTest).SetUpTest() is high (> 15) (gocyclo)

• goofys/internal/goofys.go
  • Line 595: warning: cyclomatic complexity 19 of function (*Goofys).LookUpInode() is high (> 15) (gocyclo)

• goofys/api/common/conf_azure.go
  • Line 286: warning: cyclomatic complexity 33 of function AzureBlobConfig() is high (> 15) (gocyclo)
  • Line 163: warning: cyclomatic complexity 16 of function (AzureAuthorizerConfig).Authorizer() is high (> 15) (gocyclo)

• goofys/internal/backend_azblob.go
  • Line 346: warning: cyclomatic complexity 26 of function mapAZBError() is high (> 15) (gocyclo)
  • Line 481: warning: cyclomatic complexity 26 of function (*AZBlob).ListBlobs() is high (> 15) (gocyclo)

• goofys/internal/backend_s3.go
  • Line 129: warning: cyclomatic complexity 22 of function (*S3Backend).detectBucketLocationByHEAD() is high (> 15) (gocyclo)
  • Line 625: warning: cyclomatic complexity 21 of function (*S3Backend).CopyBlob() is high (> 15) (gocyclo)

golint18%

Golint is a linter for Go source code.

• goofys/api/common/logger.go
  • Line 37: warning: exported function InitLoggers should have comment or be unexported (golint)
  • Line 54: warning: exported function SetCloudLogLevel should have comment or be unexported (golint)
  • Line 64: warning: exported type LogHandle should have comment or be unexported (golint)
  • Line 71: warning: exported method LogHandle.Format should have comment or be unexported (golint)
  • Line 99: warning: comment on exported method LogHandle.Log should be of the form "Log ..." (golint)
  • Line 104: warning: exported function NewLogger should have comment or be unexported (golint)
  • Line 116: warning: exported function GetLogger should have comment or be unexported (golint)
  • Line 125: 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 135: warning: exported function GetStdLogger should have comment or be unexported (golint)
  • Line 143: warning: exported type RetryHTTPLogger should have comment or be unexported (golint)
  • Line 147: warning: exported const DEBUG_TAG should have comment or be unexported (golint)
  • Line 148: warning: exported const ERR_TAG should have comment or be unexported (golint)
  • Line 150: warning: exported method RetryHTTPLogger.Printf should have comment or be unexported (golint)

• goofys/internal/backend_adlv2.go
  • Line 19: warning: should not use dot imports (golint)
  • Line 43: warning: exported type ADLv2 should have comment or be unexported (golint)
  • Line 53: warning: exported const ADL2_CLIENT_REQUEST_ID should have comment or be unexported (golint)
  • Line 54: warning: exported const ADL2_REQUEST_ID should have comment or be unexported (golint)
  • Line 58: warning: exported type ADLv2MultipartBlobCommitInput should have comment or be unexported (golint)
  • Line 64: warning: exported function IsADLv2Endpoint should have comment or be unexported (golint)
  • Line 87: warning: exported function NewADLv2 should have comment or be unexported (golint)
  • Line 189: warning: exported method ADLv2.Bucket should have comment or be unexported (golint)
  • Line 193: warning: exported method ADLv2.Delegate should have comment or be unexported (golint)
  • Line 197: warning: exported method ADLv2.Init should have comment or be unexported (golint)
  • Line 205: warning: exported method ADLv2.Capabilities should have comment or be unexported (golint)
  • Line 209: warning: exported type ADL2Error should have comment or be unexported (golint)
  • Line 228: warning: should omit 2nd value from range; this loop is equivalent to `for k := range ...` (golint)
  • Line 264: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 275: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 296: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 332: 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 341: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 355: warning: exported method ADLv2.HeadBlob should have comment or be unexported (golint)
  • Line 413: warning: exported method ADLv2.ListBlobs should have comment or be unexported (golint)
  • Line 429: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 498: warning: exported method ADLv2.DeleteBlob should have comment or be unexported (golint)
  • Line 512: warning: exported method ADLv2.DeleteBlobs should have comment or be unexported (golint)
  • Line 516: warning: exported method ADLv2.RenameBlob should have comment or be unexported (golint)
  • Line 546: warning: exported method ADLv2.CopyBlob should have comment or be unexported (golint)
  • Line 563: warning: exported method ADLv2.GetBlob should have comment or be unexported (golint)
  • Line 669: warning: exported method ADLv2.PutBlob should have comment or be unexported (golint)
  • Line 680: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 722: warning: comment on exported method ADLv2.MultipartBlobBegin should be of the form "MultipartBlobBegin ..." (golint)
  • Line 809: warning: exported method ADLv2.MultipartBlobAdd should have comment or be unexported (golint)
  • Line 828: warning: exported method ADLv2.MultipartBlobAbort should have comment or be unexported (golint)
  • Line 838: warning: exported method ADLv2.MultipartBlobCommit should have comment or be unexported (golint)
  • Line 871: warning: exported method ADLv2.MultipartExpire should have comment or be unexported (golint)
  • Line 875: warning: exported method ADLv2.RemoveBucket should have comment or be unexported (golint)
  • Line 884: warning: exported method ADLv2.MakeBucket should have comment or be unexported (golint)
  • Line 1812: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 1851: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• goofys/internal/backend_s3.go
  • Line 18: warning: should not use dot imports (golint)
  • Line 39: warning: exported type S3Backend should have comment or be unexported (golint)
  • Line 54: warning: exported function NewS3 should have comment or be unexported (golint)
  • Line 86: warning: exported method S3Backend.Bucket should have comment or be unexported (golint)
  • Line 90: warning: exported method S3Backend.Capabilities should have comment or be unexported (golint)
  • Line 129: warning: error should be the last type when returning multiple items (golint)
  • Line 237: warning: exported method S3Backend.Init should have comment or be unexported (golint)
  • Line 283: warning: exported method S3Backend.ListObjectsV2 should have comment or be unexported (golint)
  • Line 291: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 353: warning: exported method S3Backend.HeadBlob should have comment or be unexported (golint)
  • Line 383: warning: exported method S3Backend.ListBlobs should have comment or be unexported (golint)
  • Line 427: warning: exported method S3Backend.DeleteBlob should have comment or be unexported (golint)
  • Line 439: warning: exported method S3Backend.DeleteBlobs should have comment or be unexported (golint)
  • Line 440: warning: don't use underscores in Go names; var num_objs should be numObjs (golint)
  • Line 445: warning: should omit 2nd value from range; this loop is equivalent to `for i := range ...` (golint)
  • Line 464: warning: exported method S3Backend.RenameBlob should have comment or be unexported (golint)
  • Line 592: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 625: warning: exported method S3Backend.CopyBlob should have comment or be unexported (golint)
  • Line 714: warning: exported method S3Backend.GetBlob should have comment or be unexported (golint)
  • Line 773: warning: exported method S3Backend.PutBlob should have comment or be unexported (golint)
  • Line 817: warning: exported method S3Backend.MultipartBlobBegin should have comment or be unexported (golint)
  • Line 854: warning: exported method S3Backend.MultipartBlobAdd should have comment or be unexported (golint)
  • Line 886: warning: exported method S3Backend.MultipartBlobCommit should have comment or be unexported (golint)
  • Line 921: warning: exported method S3Backend.MultipartBlobAbort should have comment or be unexported (golint)
  • Line 935: warning: exported method S3Backend.MultipartExpire should have comment or be unexported (golint)
  • Line 968: warning: exported method S3Backend.RemoveBucket should have comment or be unexported (golint)
  • Line 976: warning: exported method S3Backend.MakeBucket should have comment or be unexported (golint)
  • Line 1016: warning: exported method S3Backend.Delegate should have comment or be unexported (golint)

• goofys/internal/file.go
  • Line 30: warning: exported type FileHandle should have comment or be unexported (golint)
  • Line 69: warning: exported const MAX_READAHEAD should have comment or be unexported (golint)
  • Line 70: warning: exported const READAHEAD_CHUNK should have comment or be unexported (golint)
  • Line 125: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
  • Line 228: warning: exported method FileHandle.WriteFile should have comment or be unexported (golint)
  • Line 293: warning: exported type S3ReadBuffer should have comment or be unexported (golint)
  • Line 304: warning: exported method S3ReadBuffer.Init should have comment or be unexported (golint)
  • Line 368: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 370: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
  • Line 453: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 468: warning: exported method FileHandle.ReadFile should have comment or be unexported (golint)
  • Line 557: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 573: warning: exported method FileHandle.Release should have comment or be unexported (golint)
  • Line 708: warning: exported method FileHandle.FlushFile should have comment or be unexported (golint)

• goofys/internal/flags.go
  • Line 19: warning: should not use dot imports (golint)
  • Line 80: warning: exported var VersionHash should have comment or be unexported (golint)
  • Line 82: warning: exported function NewApp should have comment or be unexported (golint)
  • Line 447: warning: exported function MassageMountFlags should have comment or be unexported (golint)

• goofys/internal/goofys_test.go
  • Line 135: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 211: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 255: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 264: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 267: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 314: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 325: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 333: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 403: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 424: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 441: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 462: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 654: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 660: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 665: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 670: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 675: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 712: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 715: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 735: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 745: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 754: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 787: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 794: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 809: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 827: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 860: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 883: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 927: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 955: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 992: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1028: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1028: warning: don't use underscores in Go names; method parameter write_size should be writeSize (golint)
  • Line 1032: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1032: warning: don't use underscores in Go names; method parameter write_size should be writeSize (golint)
  • Line 1104: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1110: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1117: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1122: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1132: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1147: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1151: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1177: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1199: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1213: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1240: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1259: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1285: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1322: warning: should omit 2nd value from range; this loop is equivalent to `for b := range ...` (golint)
  • Line 1349: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1453: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1485: warning: don't use underscores in Go names; var new_dir2 should be newDir2 (golint)
  • Line 1514: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1560: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1610: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1667: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1686: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1726: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1732: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1739: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1745: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1750: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1756: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1778: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1786: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1793: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1800: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1807: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1815: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1823: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1842: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1856: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1884: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1905: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1939: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1947: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1955: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 1993: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2025: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2034: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2068: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2109: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2143: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2150: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2173: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2296: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2353: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2373: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2390: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2408: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2488: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2544: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2552: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2598: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2640: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2685: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2718: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2743: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2772: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2796: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2837: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2859: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2873: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2896: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2914: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 2949: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3023: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3117: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3142: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3154: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3204: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3286: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3319: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3370: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3455: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3503: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3525: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3560: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3625: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3642: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3652: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3661: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3677: warning: don't use underscores in Go names; var dir_in should be dirIn (golint)
  • Line 3683: warning: don't use underscores in Go names; var dir_a should be dirA (golint)
  • Line 3689: warning: don't use underscores in Go names; var dir_dir should be dirDir (golint)
  • Line 3729: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3773: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3820: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3849: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3909: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3949: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 3991: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 4000: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 4024: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 4060: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 4100: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 4104: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 4108: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)
  • Line 4192: warning: receiver name s should be consistent with previous receiver name t for GoofysTest (golint)

• goofys/main.go
  • Line 20: warning: should not use dot imports (golint)
  • Line 21: warning: should not use dot imports (golint)
  • Line 134: warning: exported var Version should have comment or be unexported (golint)
  • Line 190: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• goofys/api/common/conf_azure.go
  • Line 39: warning: exported type SASTokenProvider should have comment or be unexported (golint)
  • Line 41: warning: exported type AZBlobConfig should have comment or be unexported (golint)
  • Line 52: warning: exported method AZBlobConfig.Init should have comment or be unexported (golint)
  • Line 65: warning: comment on exported method AZBlobConfig.WithAuthorization should be of the form "WithAuthorization ..." (golint)
  • Line 85: warning: exported type ADLv1Config should have comment or be unexported (golint)
  • Line 90: warning: exported method ADLv1Config.Init should have comment or be unexported (golint)
  • Line 93: warning: exported type ADLv2Config should have comment or be unexported (golint)
  • Line 98: warning: exported type AzureAuthorizerConfig should have comment or be unexported (golint)
  • Line 163: warning: exported method AzureAuthorizerConfig.Authorizer should have comment or be unexported (golint)
  • Line 286: warning: exported function AzureBlobConfig should have comment or be unexported (golint)
  • Line 393: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• goofys/internal/backend_azblob.go
  • Line 18: warning: should not use dot imports (golint)
  • Line 41: warning: exported const AzuriteEndpoint should have comment or be unexported (golint)
  • Line 42: warning: exported const AzureDirBlobMetadataKey should have comment or be unexported (golint)
  • Line 43: warning: exported const AzureBlobMetaDataHeaderPrefix should have comment or be unexported (golint)
  • Line 100: warning: exported type AZBlob should have comment or be unexported (golint)
  • Line 120: warning: exported function NewAZBlob should have comment or be unexported (golint)
  • Line 199: warning: exported method AZBlob.Delegate should have comment or be unexported (golint)
  • Line 203: warning: exported method AZBlob.Capabilities should have comment or be unexported (golint)
  • Line 207: warning: exported method AZBlob.Bucket should have comment or be unexported (golint)
  • Line 336: warning: exported method AZBlob.Init should have comment or be unexported (golint)
  • Line 399: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 411: warning: should omit 2nd value from range; this loop is equivalent to `for k := range ...` (golint)
  • Line 428: warning: exported method AZBlob.HeadBlob should have comment or be unexported (golint)
  • Line 476: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 481: warning: exported method AZBlob.ListBlobs should have comment or be unexported (golint)
  • Line 525: warning: should omit 2nd value from range; this loop is equivalent to `for i := range ...` (golint)
  • Line 568: warning: should omit 2nd value from range; this loop is equivalent to `for idx := range ...` (golint)
  • Line 640: warning: exported method AZBlob.DeleteBlob should have comment or be unexported (golint)
  • Line 658: warning: exported method AZBlob.DeleteBlobs should have comment or be unexported (golint)
  • Line 696: warning: exported method AZBlob.RenameBlob should have comment or be unexported (golint)
  • Line 700: warning: exported method AZBlob.CopyBlob should have comment or be unexported (golint)
  • Line 738: warning: exported method AZBlob.GetBlob should have comment or be unexported (golint)
  • Line 779: warning: exported method AZBlob.PutBlob should have comment or be unexported (golint)
  • Line 820: warning: exported method AZBlob.MultipartBlobBegin should have comment or be unexported (golint)
  • Line 833: warning: exported method AZBlob.MultipartBlobAdd should have comment or be unexported (golint)
  • Line 856: warning: exported method AZBlob.MultipartBlobAbort should have comment or be unexported (golint)
  • Line 861: warning: exported method AZBlob.MultipartBlobCommit should have comment or be unexported (golint)
  • Line 887: warning: exported method AZBlob.MultipartExpire should have comment or be unexported (golint)
  • Line 891: warning: exported method AZBlob.RemoveBucket should have comment or be unexported (golint)
  • Line 904: warning: exported method AZBlob.MakeBucket should have comment or be unexported (golint)

• goofys/internal/handles.go
  • Line 37: warning: exported type InodeAttributes should have comment or be unexported (golint)
  • Line 42: warning: exported method InodeAttributes.Equal should have comment or be unexported (golint)
  • Line 46: warning: exported type Inode should have comment or be unexported (golint)
  • Line 91: warning: exported function NewInode should have comment or be unexported (golint)
  • Line 128: warning: exported method Inode.SetFromBlobItem should have comment or be unexported (golint)
  • Line 215: warning: exported method Inode.FullName should have comment or be unexported (golint)
  • Line 218: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 228: warning: exported method Inode.InflateAttributes should have comment or be unexported (golint)
  • Line 264: warning: exported method Inode.ToDir should have comment or be unexported (golint)
  • Line 275: warning: comment on exported method Inode.Ref should be of the form "Ref ..." (golint)
  • Line 286: warning: exported method Inode.DeRef should have comment or be unexported (golint)
  • Line 299: warning: exported method Inode.GetAttributes should have comment or be unexported (golint)
  • Line 357: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 390: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 424: warning: exported method Inode.SetXattr should have comment or be unexported (golint)
  • Line 453: warning: exported method Inode.RemoveXattr should have comment or be unexported (golint)
  • Line 468: 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 473: warning: exported method Inode.GetXattr should have comment or be unexported (golint)
  • Line 487: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 492: warning: exported method Inode.ListXattr should have comment or be unexported (golint)
  • Line 508: warning: should omit 2nd value from range; this loop is equivalent to `for k := range ...` (golint)
  • Line 512: warning: should omit 2nd value from range; this loop is equivalent to `for k := range ...` (golint)
  • Line 521: warning: exported method Inode.OpenFile should have comment or be unexported (golint)

• goofys/api/api.go
  • Line 4: warning: should not use dot imports (golint)
  • Line 20: warning: exported function Mount should have comment or be unexported (golint)
  • Line 190: warning: exported type Goofys should have comment or be unexported (golint)

• goofys/api/common/panic_logger.go
  • Line 26: warning: exported type FusePanicLogger should have comment or be unexported (golint)
  • Line 30: warning: exported function LogPanic should have comment or be unexported (golint)
  • Line 39: warning: exported method FusePanicLogger.StatFS should have comment or be unexported (golint)
  • Line 43: warning: exported method FusePanicLogger.LookUpInode should have comment or be unexported (golint)
  • Line 47: warning: exported method FusePanicLogger.GetInodeAttributes should have comment or be unexported (golint)
  • Line 51: warning: exported method FusePanicLogger.SetInodeAttributes should have comment or be unexported (golint)
  • Line 55: warning: exported method FusePanicLogger.Fallocate should have comment or be unexported (golint)
  • Line 59: warning: exported method FusePanicLogger.ForgetInode should have comment or be unexported (golint)
  • Line 63: warning: exported method FusePanicLogger.MkDir should have comment or be unexported (golint)
  • Line 67: warning: exported method FusePanicLogger.MkNode should have comment or be unexported (golint)
  • Line 71: warning: exported method FusePanicLogger.CreateFile should have comment or be unexported (golint)
  • Line 75: warning: exported method FusePanicLogger.CreateLink should have comment or be unexported (golint)
  • Line 79: warning: exported method FusePanicLogger.CreateSymlink should have comment or be unexported (golint)
  • Line 83: warning: exported method FusePanicLogger.Rename should have comment or be unexported (golint)
  • Line 87: warning: exported method FusePanicLogger.RmDir should have comment or be unexported (golint)
  • Line 91: warning: exported method FusePanicLogger.Unlink should have comment or be unexported (golint)
  • Line 95: warning: exported method FusePanicLogger.OpenDir should have comment or be unexported (golint)
  • Line 99: warning: exported method FusePanicLogger.ReadDir should have comment or be unexported (golint)
  • Line 103: warning: exported method FusePanicLogger.ReleaseDirHandle should have comment or be unexported (golint)
  • Line 107: warning: exported method FusePanicLogger.OpenFile should have comment or be unexported (golint)
  • Line 111: warning: exported method FusePanicLogger.ReadFile should have comment or be unexported (golint)
  • Line 115: warning: exported method FusePanicLogger.WriteFile should have comment or be unexported (golint)
  • Line 119: warning: exported method FusePanicLogger.SyncFile should have comment or be unexported (golint)
  • Line 123: warning: exported method FusePanicLogger.FlushFile should have comment or be unexported (golint)
  • Line 127: warning: exported method FusePanicLogger.ReleaseFileHandle should have comment or be unexported (golint)
  • Line 131: warning: exported method FusePanicLogger.ReadSymlink should have comment or be unexported (golint)
  • Line 135: warning: exported method FusePanicLogger.RemoveXattr should have comment or be unexported (golint)
  • Line 139: warning: exported method FusePanicLogger.GetXattr should have comment or be unexported (golint)
  • Line 143: warning: exported method FusePanicLogger.ListXattr should have comment or be unexported (golint)
  • Line 147: warning: exported method FusePanicLogger.SetXattr should have comment or be unexported (golint)
  • Line 152: warning: exported method FusePanicLogger.Destroy should have comment or be unexported (golint)

• goofys/internal/backend_adlv1.go
  • Line 18: warning: should not use dot imports (golint)
  • Line 38: warning: exported type ADLv1 should have comment or be unexported (golint)
  • Line 54: warning: exported type ADLv1Err should have comment or be unexported (golint)
  • Line 67: warning: exported const ADL1_REQUEST_ID should have comment or be unexported (golint)
  • Line 71: warning: exported type ADLv1MultipartBlobCommitInput should have comment or be unexported (golint)
  • Line 75: warning: exported function IsADLv1Endpoint should have comment or be unexported (golint)
  • Line 90: warning: exported function NewADLv1 should have comment or be unexported (golint)
  • Line 167: warning: exported method ADLv1.Bucket should have comment or be unexported (golint)
  • Line 171: warning: exported method ADLv1.Delegate should have comment or be unexported (golint)
  • Line 183: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 198: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 206: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 227: warning: exported method ADLv1.Init should have comment or be unexported (golint)
  • Line 238: warning: exported method ADLv1.Capabilities should have comment or be unexported (golint)
  • Line 254: warning: exported method ADLv1.HeadBlob should have comment or be unexported (golint)
  • Line 337: warning: exported method ADLv1.ListBlobs should have comment or be unexported (golint)
  • Line 370: warning: exported method ADLv1.DeleteBlob should have comment or be unexported (golint)
  • Line 382: warning: exported method ADLv1.DeleteBlobs should have comment or be unexported (golint)
  • Line 386: warning: exported method ADLv1.RenameBlob should have comment or be unexported (golint)
  • Line 425: warning: exported method ADLv1.CopyBlob should have comment or be unexported (golint)
  • Line 429: warning: exported method ADLv1.GetBlob should have comment or be unexported (golint)
  • Line 489: warning: exported method ADLv1.PutBlob should have comment or be unexported (golint)
  • Line 508: warning: exported method ADLv1.MultipartBlobBegin should have comment or be unexported (golint)
  • Line 584: warning: exported method ADLv1.MultipartBlobAdd should have comment or be unexported (golint)
  • Line 604: warning: exported method ADLv1.MultipartBlobAbort should have comment or be unexported (golint)
  • Line 621: warning: exported method ADLv1.MultipartBlobCommit should have comment or be unexported (golint)
  • Line 650: warning: exported method ADLv1.MultipartExpire should have comment or be unexported (golint)
  • Line 654: warning: exported method ADLv1.RemoveBucket should have comment or be unexported (golint)
  • Line 671: warning: exported method ADLv1.MakeBucket should have comment or be unexported (golint)

• goofys/internal/backend_gcs3.go
  • Line 18: warning: should not use dot imports (golint)
  • Line 32: warning: comment on exported type GCS3 should be of the form "GCS3 ..." (with optional leading article) (golint)
  • Line 37: warning: exported type GCSMultipartBlobCommitInput should have comment or be unexported (golint)
  • Line 43: warning: exported function NewGCS3 should have comment or be unexported (golint)
  • Line 55: warning: exported method GCS3.Delegate should have comment or be unexported (golint)
  • Line 59: warning: exported method GCS3.DeleteBlobs should have comment or be unexported (golint)
  • Line 84: warning: exported method GCS3.MultipartBlobBegin should have comment or be unexported (golint)
  • Line 185: warning: exported method GCS3.MultipartBlobAdd should have comment or be unexported (golint)
  • Line 212: warning: exported method GCS3.MultipartBlobCommit should have comment or be unexported (golint)

• goofys/internal/buffer_pool.go
  • Line 18: warning: should not use dot imports (golint)
  • Line 29: warning: exported type BufferPool should have comment or be unexported (golint)
  • Line 42: warning: exported const BUF_SIZE should have comment or be unexported (golint)
  • Line 80: warning: exported method BufferPool.Init should have comment or be unexported (golint)
  • Line 91: warning: comment on exported function NewBufferPool should be of the form "NewBufferPool ..." (golint)
  • Line 97: warning: exported method BufferPool.RequestBuffer should have comment or be unexported (golint)
  • Line 110: warning: exported method BufferPool.RequestMultiple should have comment or be unexported (golint)
  • Line 151: warning: exported method BufferPool.MaybeGC should have comment or be unexported (golint)
  • Line 157: warning: exported method BufferPool.Free should have comment or be unexported (golint)
  • Line 170: warning: exported type MBuf should have comment or be unexported (golint)
  • Line 179: warning: exported method MBuf.Init should have comment or be unexported (golint)
  • Line 192: warning: exported method MBuf.Len should have comment or be unexported (golint)
  • Line 214: warning: comment on exported method MBuf.Seek should be of the form "Seek ..." (golint)
  • Line 273: warning: exported method MBuf.Full should have comment or be unexported (golint)
  • Line 299: warning: exported method MBuf.WriteFrom should have comment or be unexported (golint)
  • Line 321: warning: exported method MBuf.Reset should have comment or be unexported (golint)
  • Line 328: warning: exported method MBuf.Close should have comment or be unexported (golint)
  • Line 333: warning: exported method MBuf.Free should have comment or be unexported (golint)
  • Line 343: warning: exported type Buffer should have comment or be unexported (golint)
  • Line 352: warning: exported type ReaderProvider should have comment or be unexported (golint)
  • Line 354: warning: exported method Buffer.Init should have comment or be unexported (golint)
  • Line 458: warning: exported method Buffer.ReInit should have comment or be unexported (golint)
  • Line 477: warning: exported method Buffer.Close should have comment or be unexported (golint)

• goofys/internal/buffer_pool_test.go
  • Line 119: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 308: warning: don't use underscores in Go names; var test_input should be testInput (golint)
  • Line 319: warning: don't use underscores in Go names; var mem_path should be memPath (golint)

• goofys/internal/cgroup.go
  • Line 26: warning: exported const CGROUP_PATH should have comment or be unexported (golint)
  • Line 27: warning: exported const CGROUP_FOLDER_PREFIX should have comment or be unexported (golint)
  • Line 28: warning: exported const MEM_LIMIT_FILE_SUFFIX should have comment or be unexported (golint)
  • Line 29: warning: exported const MEM_USAGE_FILE_SUFFIX should have comment or be unexported (golint)
  • Line 56: warning: don't use underscores in Go names; var mem_limit should be memLimit (golint)
  • Line 62: warning: don't use underscores in Go names; var mem_usage should be memUsage (golint)

• goofys/internal/perms.go
  • Line 16: warning: package comment should be of the form "Package internal ..." (golint)

• goofys/internal/ticket.go
  • Line 21: warning: exported type Ticket should have comment or be unexported (golint)
  • Line 31: warning: exported method Ticket.Init should have comment or be unexported (golint)
  • Line 37: warning: exported method Ticket.Take should have comment or be unexported (golint)
  • Line 54: warning: exported method Ticket.Return should have comment or be unexported (golint)

• goofys/api/common/conf_s3.go
  • Line 30: warning: exported type S3Config should have comment or be unexported (golint)
  • Line 62: warning: exported method S3Config.Init should have comment or be unexported (golint)
  • Line 72: warning: exported method S3Config.ToAwsConfig should have comment or be unexported (golint)

• goofys/api/common/config.go
  • Line 28: warning: exported type FlagStorage should have comment or be unexported (golint)
  • Line 60: warning: exported method FlagStorage.GetMimeType should have comment or be unexported (golint)
  • Line 81: warning: exported method FlagStorage.Cleanup should have comment or be unexported (golint)
  • Line 104: warning: exported function GetHTTPTransport should have comment or be unexported (golint)

• goofys/internal/backend.go
  • Line 28: warning: exported type Capabilities should have comment or be unexported (golint)
  • Line 36: warning: exported type HeadBlobInput should have comment or be unexported (golint)
  • Line 40: warning: exported type BlobItemOutput should have comment or be unexported (golint)
  • Line 48: warning: exported type HeadBlobOutput should have comment or be unexported (golint)
  • Line 58: warning: exported type ListBlobsInput should have comment or be unexported (golint)
  • Line 66: warning: exported type BlobPrefixOutput should have comment or be unexported (golint)
  • Line 70: warning: exported type ListBlobsOutput should have comment or be unexported (golint)
  • Line 79: warning: exported type DeleteBlobInput should have comment or be unexported (golint)
  • Line 83: warning: exported type DeleteBlobOutput should have comment or be unexported (golint)
  • Line 87: warning: exported type DeleteBlobsInput should have comment or be unexported (golint)
  • Line 91: warning: exported type DeleteBlobsOutput should have comment or be unexported (golint)
  • Line 95: warning: exported type RenameBlobInput should have comment or be unexported (golint)
  • Line 100: warning: exported type RenameBlobOutput should have comment or be unexported (golint)
  • Line 104: warning: exported type CopyBlobInput should have comment or be unexported (golint)
  • Line 114: warning: exported type CopyBlobOutput should have comment or be unexported (golint)
  • Line 118: warning: exported type GetBlobInput should have comment or be unexported (golint)
  • Line 125: warning: exported type GetBlobOutput should have comment or be unexported (golint)
  • Line 133: warning: exported type PutBlobInput should have comment or be unexported (golint)
  • Line 143: warning: exported type PutBlobOutput should have comment or be unexported (golint)
  • Line 151: warning: exported type MultipartBlobBeginInput should have comment or be unexported (golint)
  • Line 157: warning: exported type MultipartBlobCommitInput should have comment or be unexported (golint)
  • Line 169: warning: exported type MultipartBlobAddInput should have comment or be unexported (golint)
  • Line 180: warning: exported type MultipartBlobAddOutput should have comment or be unexported (golint)
  • Line 184: warning: exported type MultipartBlobCommitOutput should have comment or be unexported (golint)
  • Line 192: warning: exported type MultipartBlobAbortOutput should have comment or be unexported (golint)
  • Line 196: warning: exported type MultipartExpireInput should have comment or be unexported (golint)
  • Line 199: warning: exported type MultipartExpireOutput should have comment or be unexported (golint)
  • Line 203: warning: exported type RemoveBucketInput should have comment or be unexported (golint)
  • Line 206: warning: exported type RemoveBucketOutput should have comment or be unexported (golint)
  • Line 210: warning: exported type MakeBucketInput should have comment or be unexported (golint)
  • Line 213: warning: exported type MakeBucketOutput should have comment or be unexported (golint)
  • Line 217: warning: comment on exported type StorageBackend should be of the form "StorageBackend ..." (with optional leading article) (golint)
  • Line 247: warning: exported type Delegator should have comment or be unexported (golint)
  • Line 251: warning: exported var SmallActionsGate should have comment or be unexported (golint)
  • Line 289: warning: exported type ReadSeekerCloser should have comment or be unexported (golint)
  • Line 293: warning: exported method ReadSeekerCloser.Close should have comment or be unexported (golint)
  • Line 296: 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 301: warning: exported type StorageBackendInitWrapper should have comment or be unexported (golint)
  • Line 308: warning: exported const INIT_ERR_BLOB should have comment or be unexported (golint)
  • Line 310: warning: exported method StorageBackendInitWrapper.Init should have comment or be unexported (golint)
  • Line 324: warning: exported method StorageBackendInitWrapper.Capabilities should have comment or be unexported (golint)
  • Line 328: warning: exported method StorageBackendInitWrapper.Bucket should have comment or be unexported (golint)
  • Line 332: warning: exported method StorageBackendInitWrapper.HeadBlob should have comment or be unexported (golint)
  • Line 337: warning: exported method StorageBackendInitWrapper.ListBlobs should have comment or be unexported (golint)
  • Line 342: warning: exported method StorageBackendInitWrapper.DeleteBlob should have comment or be unexported (golint)
  • Line 347: warning: exported method StorageBackendInitWrapper.DeleteBlobs should have comment or be unexported (golint)
  • Line 352: warning: exported method StorageBackendInitWrapper.RenameBlob should have comment or be unexported (golint)
  • Line 357: warning: exported method StorageBackendInitWrapper.CopyBlob should have comment or be unexported (golint)
  • Line 362: warning: exported method StorageBackendInitWrapper.GetBlob should have comment or be unexported (golint)
  • Line 367: warning: exported method StorageBackendInitWrapper.PutBlob should have comment or be unexported (golint)
  • Line 372: warning: exported method StorageBackendInitWrapper.MultipartBlobBegin should have comment or be unexported (golint)
  • Line 377: warning: exported method StorageBackendInitWrapper.MultipartBlobAdd should have comment or be unexported (golint)
  • Line 382: warning: exported method StorageBackendInitWrapper.MultipartBlobAbort should have comment or be unexported (golint)
  • Line 387: warning: exported method StorageBackendInitWrapper.MultipartBlobCommit should have comment or be unexported (golint)
  • Line 392: warning: exported method StorageBackendInitWrapper.MultipartExpire should have comment or be unexported (golint)
  • Line 397: warning: exported method StorageBackendInitWrapper.RemoveBucket should have comment or be unexported (golint)
  • Line 402: warning: exported method StorageBackendInitWrapper.MakeBucket should have comment or be unexported (golint)
  • Line 407: warning: exported type StorageBackendInitError should have comment or be unexported (golint)
  • Line 412: warning: exported method StorageBackendInitError.Init should have comment or be unexported (golint)
  • Line 416: warning: exported method StorageBackendInitError.Delegate should have comment or be unexported (golint)
  • Line 420: warning: exported method StorageBackendInitError.Capabilities should have comment or be unexported (golint)
  • Line 424: warning: exported method StorageBackendInitError.Bucket should have comment or be unexported (golint)
  • Line 424: warning: receiver name s should be consistent with previous receiver name e for StorageBackendInitError (golint)
  • Line 428: warning: exported method StorageBackendInitError.HeadBlob should have comment or be unexported (golint)
  • Line 438: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 443: warning: exported method StorageBackendInitError.ListBlobs should have comment or be unexported (golint)
  • Line 455: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 460: warning: exported method StorageBackendInitError.DeleteBlob should have comment or be unexported (golint)
  • Line 464: warning: exported method StorageBackendInitError.DeleteBlobs should have comment or be unexported (golint)
  • Line 468: warning: exported method StorageBackendInitError.RenameBlob should have comment or be unexported (golint)
  • Line 472: warning: exported method StorageBackendInitError.CopyBlob should have comment or be unexported (golint)
  • Line 476: warning: exported method StorageBackendInitError.GetBlob should have comment or be unexported (golint)
  • Line 490: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 495: warning: exported method StorageBackendInitError.PutBlob should have comment or be unexported (golint)
  • Line 499: warning: exported method StorageBackendInitError.MultipartBlobBegin should have comment or be unexported (golint)
  • Line 503: warning: exported method StorageBackendInitError.MultipartBlobAdd should have comment or be unexported (golint)
  • Line 507: warning: exported method StorageBackendInitError.MultipartBlobAbort should have comment or be unexported (golint)
  • Line 511: warning: exported method StorageBackendInitError.MultipartBlobCommit should have comment or be unexported (golint)
  • Line 515: warning: exported method StorageBackendInitError.MultipartExpire should have comment or be unexported (golint)
  • Line 519: warning: exported method StorageBackendInitError.RemoveBucket should have comment or be unexported (golint)
  • Line 523: warning: exported method StorageBackendInitError.MakeBucket should have comment or be unexported (golint)

• goofys/internal/dir.go
  • Line 33: warning: exported type DirInodeData should have comment or be unexported (golint)
  • Line 46: warning: exported type DirHandleEntry should have comment or be unexported (golint)
  • Line 93: warning: exported type DirHandle should have comment or be unexported (golint)
  • Line 106: warning: exported function NewDirHandle should have comment or be unexported (golint)
  • Line 111: warning: exported method Inode.OpenDir should have comment or be unexported (golint)
  • Line 429: warning: comment on exported method DirHandle.ReadDir should be of the form "ReadDir ..." (golint)
  • Line 606: warning: exported method DirHandle.CloseDir should have comment or be unexported (golint)
  • Line 612: warning: receiver name dir should be consistent with previous receiver name inode for Inode (golint)
  • Line 732: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 753: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 761: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 767: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 782: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 794: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 816: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 824: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 831: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 853: warning: exported method Inode.LookUp should have comment or be unexported (golint)
  • Line 853: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 864: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 867: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 872: warning: exported method Inode.Unlink should have comment or be unexported (golint)
  • Line 872: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 901: warning: exported method Inode.Create should have comment or be unexported (golint)
  • Line 901: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 928: warning: exported method Inode.MkDir should have comment or be unexported (golint)
  • Line 928: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 971: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 1001: warning: exported method Inode.RmDir should have comment or be unexported (golint)
  • Line 1001: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 1037: warning: comment on exported method Inode.Rename should be of the form "Rename ..." (golint)
  • Line 1045: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 1086: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 1114: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 1147: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 1164: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 1245: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 1261: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 1291: warning: exported method Inode.LookUpInodeNotDir should have comment or be unexported (golint)
  • Line 1291: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 1305: warning: exported method Inode.LookUpInodeDir should have comment or be unexported (golint)
  • Line 1305: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 1324: warning: comment on exported method Inode.LookUpInodeMaybeDir should be of the form "LookUpInodeMaybeDir ..." (golint)
  • Line 1325: warning: receiver name parent should be consistent with previous receiver name inode for Inode (golint)
  • Line 1392: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• goofys/internal/goofys.go
  • Line 18: warning: should not use dot imports (golint)
  • Line 50: warning: exported type Goofys should have comment or be unexported (golint)
  • Line 100: warning: exported function NewBackend should have comment or be unexported (golint)
  • Line 124: warning: exported type BucketSpec should have comment or be unexported (golint)
  • Line 130: warning: exported function ParseBucketSpec should have comment or be unexported (golint)
  • Line 165: warning: exported function NewGoofys should have comment or be unexported (golint)
  • Line 241: warning: comment on exported function RandStringBytesMaskImprSrc should be of the form "RandStringBytesMaskImprSrc ..." (golint)
  • Line 267: warning: exported method Goofys.SigUsr1 should have comment or be unexported (golint)
  • Line 288: warning: exported type Mount should have comment or be unexported (golint)
  • Line 368: warning: exported method Goofys.MountAll should have comment or be unexported (golint)
  • Line 378: warning: exported method Goofys.Mount should have comment or be unexported (golint)
  • Line 385: warning: exported method Goofys.Unmount should have comment or be unexported (golint)
  • Line 405: warning: exported method Goofys.StatFS should have comment or be unexported (golint)
  • Line 423: warning: exported method Goofys.GetInodeAttributes should have comment or be unexported (golint)
  • Line 440: warning: exported method Goofys.GetXattr should have comment or be unexported (golint)
  • Line 463: warning: exported method Goofys.ListXattr should have comment or be unexported (golint)
  • Line 493: warning: exported method Goofys.RemoveXattr should have comment or be unexported (golint)
  • Line 504: warning: exported method Goofys.SetXattr should have comment or be unexported (golint)
  • Line 558: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 595: warning: exported method Goofys.LookUpInode should have comment or be unexported (golint)
  • Line 753: warning: exported method Goofys.ForgetInode should have comment or be unexported (golint)
  • Line 782: warning: exported method Goofys.OpenDir should have comment or be unexported (golint)
  • Line 814: warning: exported method Goofys.ReadDir should have comment or be unexported (golint)
  • Line 859: warning: exported method Goofys.ReleaseDirHandle should have comment or be unexported (golint)
  • Line 876: warning: exported method Goofys.OpenFile should have comment or be unexported (golint)
  • Line 921: warning: exported method Goofys.ReadFile should have comment or be unexported (golint)
  • Line 934: warning: exported method Goofys.SyncFile should have comment or be unexported (golint)
  • Line 943: warning: exported method Goofys.FlushFile should have comment or be unexported (golint)
  • Line 991: warning: exported method Goofys.ReleaseFileHandle should have comment or be unexported (golint)
  • Line 1008: warning: exported method Goofys.CreateFile should have comment or be unexported (golint)
  • Line 1044: warning: exported method Goofys.MkDir should have comment or be unexported (golint)
  • Line 1074: warning: exported method Goofys.RmDir should have comment or be unexported (golint)
  • Line 1087: warning: exported method Goofys.SetInodeAttributes should have comment or be unexported (golint)
  • Line 1103: warning: exported method Goofys.WriteFile should have comment or be unexported (golint)
  • Line 1120: warning: exported method Goofys.Unlink should have comment or be unexported (golint)
  • Line 1132: warning: comment on exported method Goofys.Rename should be of the form "Rename ..." (golint)

• goofys/internal/utils.go
  • Line 26: warning: exported var TIME_MAX should have comment or be unexported (golint)
  • Line 28: warning: exported function MaxInt should have comment or be unexported (golint)
  • Line 31: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 36: warning: exported function MinInt should have comment or be unexported (golint)
  • Line 39: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 44: warning: exported function MaxInt64 should have comment or be unexported (golint)
  • Line 47: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 52: warning: exported function MinInt64 should have comment or be unexported (golint)
  • Line 55: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 60: warning: exported function MaxUInt32 should have comment or be unexported (golint)
  • Line 63: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 68: warning: exported function MinUInt32 should have comment or be unexported (golint)
  • Line 71: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 76: warning: exported function MaxUInt64 should have comment or be unexported (golint)
  • Line 79: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 84: warning: exported function MinUInt64 should have comment or be unexported (golint)
  • Line 87: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 92: warning: exported function PBool should have comment or be unexported (golint)
  • Line 96: warning: exported function PInt32 should have comment or be unexported (golint)
  • Line 100: warning: exported function PUInt32 should have comment or be unexported (golint)
  • Line 104: warning: exported function PInt64 should have comment or be unexported (golint)
  • Line 108: warning: exported function PUInt64 should have comment or be unexported (golint)
  • Line 112: warning: exported function PString should have comment or be unexported (golint)
  • Line 116: warning: exported function PTime should have comment or be unexported (golint)
  • Line 120: warning: exported function NilStr should have comment or be unexported (golint)
  • Line 123: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 142: warning: exported function Dup should have comment or be unexported (golint)
  • Line 148: warning: exported function TryUnmount should have comment or be unexported (golint)

• goofys/internal/v2signer.go
  • Line 79: warning: comment on exported function SignV2 should be of the form "SignV2 ..." (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign81%

IneffAssign detects ineffectual assignments in Go code.

• goofys/internal/handles.go
  • Line 342: warning: ineffectual assignment to fullName (ineffassign)

• goofys/internal/goofys_test.go
  • Line 913: warning: ineffectual assignment to err (ineffassign)
  • Line 2169: warning: ineffectual assignment to err (ineffassign)
  • Line 2217: warning: ineffectual assignment to value (ineffassign)
  • Line 2276: warning: ineffectual assignment to err (ineffassign)
  • Line 2287: warning: ineffectual assignment to err (ineffassign)
  • Line 2333: warning: ineffectual assignment to err (ineffassign)
  • Line 2335: warning: ineffectual assignment to err (ineffassign)

• goofys/api/common/conf_azure.go
  • Line 383: warning: ineffectual assignment to key (ineffassign)

• goofys/api/api.go
  • Line 98: warning: ineffectual assignment to bucketName (ineffassign)

• goofys/internal/backend_adlv1.go
  • Line 322: warning: ineffectual assignment to err (ineffassign)
  • Line 358: warning: ineffectual assignment to err (ineffassign)

• goofys/internal/backend_azblob.go
  • Line 615: warning: ineffectual assignment to err (ineffassign)

misspell93%

Misspell Finds commonly misspelled English words

• goofys/internal/goofys_test.go
  • Line 2332: warning: "excercise" is a misspelling of "exercise" (misspell)

• goofys/internal/handles.go
  • Line 56: warning: "comparisions" is a misspelling of "comparisons" (misspell)
  • Line 58: warning: "comparision" is a misspelling of "comparison" (misspell)