Go Report Card

Preparing report...

Report for github.com/ShadowJonathan/MOpher

B    Not bad!    Found 90 issues across 109 files

Tweet
Last refresh:

Refresh now

gofmt70%

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

go_vet100%

go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.

No problems detected. Good job!

gocyclo78%

Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.

    • MOpher/main/main.go
    • Line 231: warning: cyclomatic complexity 89 of function callback() is high (> 15) (gocyclo)
    • Line 30: warning: cyclomatic complexity 31 of function main() is high (> 15) (gocyclo)
    • MOpher/navigate.go
    • Line 578: warning: cyclomatic complexity 83 of function (*Path).NextNode() is high (> 15) (gocyclo)
    • Line 776: warning: cyclomatic complexity 81 of function solidwhole() is high (> 15) (gocyclo)
    • Line 19: warning: cyclomatic complexity 41 of function NAVtoNearest() is high (> 15) (gocyclo)
    • Line 1034: warning: cyclomatic complexity 36 of function (*Tile).PathNeighbors() is high (> 15) (gocyclo)
    • Line 251: warning: cyclomatic complexity 21 of function Moveto() is high (> 15) (gocyclo)
    • MOpher/play.go
    • Line 15: warning: cyclomatic complexity 30 of function Dig() is high (> 15) (gocyclo)
    • Line 302: warning: cyclomatic complexity 21 of function Typeof() is high (> 15) (gocyclo)
    • Line 101: warning: cyclomatic complexity 19 of function dig() is high (> 15) (gocyclo)
    • MOpher/chunk.go
    • Line 239: warning: cyclomatic complexity 24 of function updateLight() is high (> 15) (gocyclo)
    • Line 525: warning: cyclomatic complexity 23 of function (*chunk).postLoad() is high (> 15) (gocyclo)
    • MOpher/network.go
    • Line 573: warning: cyclomatic complexity 23 of function (*NetworkManager).Connect() is high (> 15) (gocyclo)
    • Line 59: warning: cyclomatic complexity 17 of function (handler).ServerMessage() is high (> 15) (gocyclo)
    • MOpher/blockimpl.go
    • Line 1394: warning: cyclomatic complexity 17 of function (slabVariant).String() is high (> 15) (gocyclo)
    • Line 755: warning: cyclomatic complexity 17 of function (color).String() is high (> 15) (gocyclo)

golint25%

Golint is a linter for Go source code.

    • MOpher/Protocol/versions/340/play_serverbound.go
    • Line 3: warning: don't use an underscore in package name (golint)
    • Line 9: warning: comment on exported type TeleConfirm should be of the form "TeleConfirm ..." (with optional leading article) (golint)
    • Line 154: warning: comment on exported type VehicleDrive should be of the form "VehicleDrive ..." (with optional leading article) (golint)
    • Line 162: warning: comment on exported type SteerBoat should be of the form "SteerBoat ..." (with optional leading article) (golint)
    • Line 169: warning: comment on exported type CraftReceipeRequest should be of the form "CraftReceipeRequest ..." (with optional leading article) (golint)
    • Line 216: warning: comment on exported type CraftingBookData should be of the form "CraftingBookData ..." (with optional leading article) (golint)
    • Line 233: warning: comment on exported type AdvancementTab should be of the form "AdvancementTab ..." (with optional leading article) (golint)
    • MOpher/Protocol/login.go
    • Line 19: warning: error should be the last type when returning multiple items (golint)
    • Line 47: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 87: warning: error var OFFLINE_ERR should have name of the form ErrFoo (golint)
    • Line 87: warning: exported var OFFLINE_ERR should have comment or be unexported (golint)
    • Line 89: warning: error should be the last type when returning multiple items (golint)
    • Line 94: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • MOpher/blockimpl.go
    • Line 15: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 152: warning: receiver name g should be consistent with previous receiver name b for blockGrass (golint)
    • Line 156: warning: receiver name g should be consistent with previous receiver name b for blockGrass (golint)
    • MOpher/navigate.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 19: warning: error should be the last type when returning multiple items (golint)
    • Line 19: warning: exported function NAVtoNearest should have comment or be unexported (golint)
    • Line 177: warning: exported function NAV should have comment or be unexported (golint)
    • Line 230: warning: error var alreadyAtDest should have name of the form errFoo (golint)
    • Line 233: warning: exported const RadToDeg should have comment (or a comment on this block) or be unexported (golint)
    • Line 250: warning: comment on exported function Moveto should be of the form "Moveto ..." (golint)
    • Line 347: warning: exported function MoveSpecial should have comment or be unexported (golint)
    • Line 356: warning: exported var SLOW should have comment or be unexported (golint)
    • Line 358: warning: exported function Jumpto should have comment or be unexported (golint)
    • Line 411: warning: exported function Flallto should have comment or be unexported (golint)
    • Line 464: warning: exported type RelCoord should have comment or be unexported (golint)
    • Line 469: warning: exported var FallFrames should have comment or be unexported (golint)
    • Line 489: warning: exported var Jumpframes should have comment or be unexported (golint)
    • Line 511: warning: exported function Navigate should have comment or be unexported (golint)
    • Line 517: warning: exported function Show should have comment or be unexported (golint)
    • Line 528: warning: exported function RemoveShow should have comment or be unexported (golint)
    • Line 536: warning: exported function Move should have comment or be unexported (golint)
    • Line 555: warning: exported type Path should have comment or be unexported (golint)
    • Line 564: warning: exported function FindPath should have comment or be unexported (golint)
    • Line 578: warning: exported method Path.NextNode should have comment or be unexported (golint)
    • Line 717: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 722: warning: exported method Path.Print should have comment or be unexported (golint)
    • Line 744: warning: comment on exported function CheckWalkable should be of the form "CheckWalkable ..." (golint)
    • Line 758: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 772: warning: error var CHECK_ERR_ABOVE_NON_AIR should have name of the form ErrFoo (golint)
    • Line 772: warning: exported var CHECK_ERR_ABOVE_NON_AIR should have comment or be unexported (golint)
    • Line 773: warning: error var CHECK_ERR_DEFINED_NON_AIR should have name of the form ErrFoo (golint)
    • Line 773: warning: exported var CHECK_ERR_DEFINED_NON_AIR should have comment or be unexported (golint)
    • Line 774: warning: error var CHECK_ERR_BELOW_NON_SOLID should have name of the form ErrFoo (golint)
    • Line 774: warning: exported var CHECK_ERR_BELOW_NON_SOLID should have comment or be unexported (golint)
    • Line 959: warning: comment on exported type BlockLake should be of the form "BlockLake ..." (with optional leading article) (golint)
    • Line 964: warning: exported method BlockLake.Fill should have comment or be unexported (golint)
    • Line 974: warning: exported method BlockLake.Orient should have comment or be unexported (golint)
    • Line 985: warning: exported type Tiles should have comment or be unexported (golint)
    • Line 987: warning: exported method Tiles.SetTile should have comment or be unexported (golint)
    • Line 1005: warning: exported var IdS should have comment or be unexported (golint)
    • Line 1007: warning: exported method Tiles.NEWID should have comment or be unexported (golint)
    • Line 1014: warning: exported const Xaxis should have comment (or a comment on this block) or be unexported (golint)
    • Line 1025: warning: exported type Tile should have comment or be unexported (golint)
    • Line 1034: warning: exported method Tile.PathNeighbors should have comment or be unexported (golint)
    • Line 1161: warning: exported method Tile.Get should have comment or be unexported (golint)
    • Line 1178: warning: exported method Tile.PathNeighborCost should have comment or be unexported (golint)
    • Line 1195: warning: exported method Tile.PathEstimatedCost should have comment or be unexported (golint)
    • Line 1203: warning: exported type Snappoint should have comment or be unexported (golint)
    • Line 1206: warning: exported const TopLeft should have comment (or a comment on this block) or be unexported (golint)
    • MOpher/Protocol/versions/335/338.go
    • Line 3: warning: don't use an underscore in package name (golint)
    • Line 9: warning: exported type This should have comment or be unexported (golint)
    • Line 14: warning: exported var P should have comment or be unexported (golint)
    • Line 16: warning: exported method This.Packets should have comment or be unexported (golint)
    • Line 20: warning: exported method This.Translate should have comment or be unexported (golint)
    • Line 24: warning: exported method This.Back should have comment or be unexported (golint)
    • Line 28: warning: exported method This.InitTranslate should have comment or be unexported (golint)
    • Line 33: warning: exported function Version should have comment or be unexported (golint)
    • MOpher/network.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 230: warning: exported function RawPitch should have comment or be unexported (golint)
    • Line 234: warning: exported function Refpitch should have comment or be unexported (golint)
    • Line 558: warning: exported type NetworkManager should have comment or be unexported (golint)
    • Line 573: warning: exported method NetworkManager.Connect should have comment or be unexported (golint)
    • Line 685: warning: exported method NetworkManager.SignalClose should have comment or be unexported (golint)
    • Line 715: warning: exported method NetworkManager.Close should have comment or be unexported (golint)
    • MOpher/Protocol/translate_335.go
    • Line 10: warning: exported function Translate_335 should have comment or be unexported (golint)
    • Line 10: warning: don't use underscores in Go names; func Translate_335 should be Translate335 (golint)
    • Line 335: warning: exported function Back_335 should have comment or be unexported (golint)
    • Line 335: warning: don't use underscores in Go names; func Back_335 should be Back335 (golint)
    • MOpher/Protocol/translate_338.go
    • Line 10: warning: exported function Translate_338 should have comment or be unexported (golint)
    • Line 10: warning: don't use underscores in Go names; func Translate_338 should be Translate338 (golint)
    • Line 324: warning: exported function Back_338 should have comment or be unexported (golint)
    • Line 324: warning: don't use underscores in Go names; func Back_338 should be Back338 (golint)
    • MOpher/Protocol/versions/335/play_serverbound.go
    • Line 3: warning: don't use an underscore in package name (golint)
    • Line 9: warning: comment on exported type TeleConfirm should be of the form "TeleConfirm ..." (with optional leading article) (golint)
    • Line 16: warning: comment on exported type PrepareCraftingGrid should be of the form "PrepareCraftingGrid ..." (with optional leading article) (golint)
    • Line 25: warning: comment on exported type ReturnEntry should be of the form "ReturnEntry ..." (with optional leading article) (golint)
    • Line 32: warning: comment on exported type PrepareEntry should be of the form "PrepareEntry ..." (with optional leading article) (golint)
    • Line 177: warning: comment on exported type VehicleDrive should be of the form "VehicleDrive ..." (with optional leading article) (golint)
    • Line 185: warning: comment on exported type SteerBoat should be of the form "SteerBoat ..." (with optional leading article) (golint)
    • Line 231: warning: comment on exported type CraftingBookData should be of the form "CraftingBookData ..." (with optional leading article) (golint)
    • Line 247: warning: comment on exported type AdvancementTab should be of the form "AdvancementTab ..." (with optional leading article) (golint)
    • MOpher/Protocol/versions/340/play_clientbound.go
    • Line 3: warning: don't use an underscore in package name (golint)
    • Line 363: warning: exported type BlockEntity should have comment or be unexported (golint)
    • Line 481: warning: comment on exported type VehicleMove should be of the form "VehicleMove ..." (with optional leading article) (golint)
    • Line 498: warning: comment on exported type CraftReceipeResponse should be of the form "CraftReceipeResponse ..." (with optional leading article) (golint)
    • Line 575: warning: comment on exported type UnlockReceipes should be of the form "UnlockReceipes ..." (with optional leading article) (golint)
    • Line 629: warning: comment on exported type SelectAdvancementTab should be of the form "SelectAdvancementTab ..." (with optional leading article) (golint)
    • Line 739: warning: comment on exported type Passengers should be of the form "Passengers ..." (with optional leading article) (golint)
    • Line 805: warning: comment on exported type HardSoundEffect should be of the form "HardSoundEffect ..." (with optional leading article) (golint)
    • Line 845: warning: comment on exported type Advancements should be of the form "Advancements ..." (with optional leading article) (golint)
    • Line 854: warning: comment on exported type AdvancementMappingItem should be of the form "AdvancementMappingItem ..." (with optional leading article) (golint)
    • Line 860: warning: comment on exported type Advancement should be of the form "Advancement ..." (with optional leading article) (golint)
    • Line 870: warning: comment on exported type AdvancementDisplay should be of the form "AdvancementDisplay ..." (with optional leading article) (golint)
    • Line 881: warning: comment on exported type AdvancementRequirements should be of the form "AdvancementRequirements ..." (with optional leading article) (golint)
    • Line 886: warning: comment on exported type ProgressMappingItem should be of the form "ProgressMappingItem ..." (with optional leading article) (golint)
    • Line 892: warning: comment on exported type AdvancementProgress should be of the form "AdvancementProgress ..." (with optional leading article) (golint)
    • Line 897: warning: comment on exported type ProgressCriteria should be of the form "ProgressCriteria ..." (with optional leading article) (golint)
    • Line 903: warning: comment on exported type CriterionProgress should be of the form "CriterionProgress ..." (with optional leading article) (golint)
    • MOpher/entity.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 91: warning: exported type Entity should have comment or be unexported (golint)
    • MOpher/Protocol/versions/315/315.go
    • Line 3: warning: don't use an underscore in package name (golint)
    • Line 9: warning: exported type This should have comment or be unexported (golint)
    • Line 14: warning: exported var P should have comment or be unexported (golint)
    • Line 16: warning: exported method This.Packets should have comment or be unexported (golint)
    • Line 20: warning: exported method This.Translate should have comment or be unexported (golint)
    • Line 24: warning: exported method This.Back should have comment or be unexported (golint)
    • Line 28: warning: exported method This.InitTranslate should have comment or be unexported (golint)
    • Line 33: warning: exported function Version should have comment or be unexported (golint)
    • MOpher/Protocol/lib/types.go
    • Line 34: warning: exported const Clientbound should have comment (or a comment on this block) or be unexported (golint)
    • Line 37: warning: exported const MaxPacketCount should have comment or be unexported (golint)
    • Line 100: warning: exported type MetaPacket should have comment or be unexported (golint)
    • MOpher/Protocol/versions/315/play_serverbound.go
    • Line 3: warning: don't use an underscore in package name (golint)
    • Line 7: warning: comment on exported type TeleConfirm should be of the form "TeleConfirm ..." (with optional leading article) (golint)
    • Line 152: warning: comment on exported type VehicleDrive should be of the form "VehicleDrive ..." (with optional leading article) (golint)
    • Line 160: warning: comment on exported type SteerBoat should be of the form "SteerBoat ..." (with optional leading article) (golint)
    • MOpher/Protocol/play_serverbound.go
    • Line 9: warning: comment on exported type TeleConfirm should be of the form "TeleConfirm ..." (with optional leading article) (golint)
    • Line 16: warning: comment on exported type PrepareCraftingGrid should be of the form "PrepareCraftingGrid ..." (with optional leading article) (golint)
    • Line 25: warning: comment on exported type ReturnEntry should be of the form "ReturnEntry ..." (with optional leading article) (golint)
    • Line 32: warning: comment on exported type PrepareEntry should be of the form "PrepareEntry ..." (with optional leading article) (golint)
    • Line 177: warning: comment on exported type VehicleDrive should be of the form "VehicleDrive ..." (with optional leading article) (golint)
    • Line 185: warning: comment on exported type SteerBoat should be of the form "SteerBoat ..." (with optional leading article) (golint)
    • Line 241: warning: comment on exported type CraftingBookData should be of the form "CraftingBookData ..." (with optional leading article) (golint)
    • Line 258: warning: comment on exported type AdvancementTab should be of the form "AdvancementTab ..." (with optional leading article) (golint)
    • MOpher/Protocol/translate_340.go
    • Line 10: warning: exported function Translate_340 should have comment or be unexported (golint)
    • Line 10: warning: don't use underscores in Go names; func Translate_340 should be Translate340 (golint)
    • Line 324: warning: exported function Back_340 should have comment or be unexported (golint)
    • Line 324: warning: don't use underscores in Go names; func Back_340 should be Back340 (golint)
    • MOpher/Protocol/versions/315/play_clientbound.go
    • Line 3: warning: don't use an underscore in package name (golint)
    • Line 363: warning: exported type BlockEntity should have comment or be unexported (golint)
    • Line 481: warning: comment on exported type VehicleMove should be of the form "VehicleMove ..." (with optional leading article) (golint)
    • Line 714: warning: comment on exported type Passengers should be of the form "Passengers ..." (with optional leading article) (golint)
    • Line 780: warning: comment on exported type HardSoundEffect should be of the form "HardSoundEffect ..." (with optional leading article) (golint)
    • MOpher/Protocol/translate_315.go
    • Line 10: warning: exported function Translate_315 should have comment or be unexported (golint)
    • Line 10: warning: don't use underscores in Go names; func Translate_315 should be Translate315 (golint)
    • Line 301: warning: exported function Back_315 should have comment or be unexported (golint)
    • Line 301: warning: don't use underscores in Go names; func Back_315 should be Back315 (golint)
    • MOpher/client.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 34: warning: exported type ClientState should have comment or be unexported (golint)
    • Line 181: warning: exported method ClientState.UpdateHealth should have comment or be unexported (golint)
    • Line 185: warning: exported method ClientState.Connect should have comment or be unexported (golint)
    • Line 193: warning: exported function Start should have comment or be unexported (golint)
    • Line 206: warning: exported function CoTicker should have comment or be unexported (golint)
    • Line 217: warning: exported function Tick should have comment or be unexported (golint)
    • Line 327: warning: exported method ClientState.TargetBlock should have comment or be unexported (golint)
    • MOpher/script_window.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 9: warning: don't use underscores in Go names; func lua_window_loader should be luaWindowLoader (golint)
    • Line 24: warning: don't use underscores in Go names; func lua_w_current_type should be luaWCurrentType (golint)
    • Line 33: warning: don't use underscores in Go names; func lua_w_inv should be luaWInv (golint)
    • Line 55: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 60: warning: don't use underscores in Go names; func lua_w_TT should be luaWTT (golint)
    • Line 71: warning: don't use underscores in Go names; func lua_w_TF should be luaWTF (golint)
    • Line 82: warning: don't use underscores in Go names; func lua_w_TC should be luaWTC (golint)
    • MOpher/Protocol/play_clientbound.go
    • Line 363: warning: exported type BlockEntity should have comment or be unexported (golint)
    • Line 481: warning: comment on exported type VehicleMove should be of the form "VehicleMove ..." (with optional leading article) (golint)
    • Line 568: warning: comment on exported type UnlockReceipes should be of the form "UnlockReceipes ..." (with optional leading article) (golint)
    • Line 622: warning: comment on exported type SelectAdvancementTab should be of the form "SelectAdvancementTab ..." (with optional leading article) (golint)
    • Line 732: warning: comment on exported type Passengers should be of the form "Passengers ..." (with optional leading article) (golint)
    • Line 798: warning: comment on exported type HardSoundEffect should be of the form "HardSoundEffect ..." (with optional leading article) (golint)
    • Line 838: warning: comment on exported type Advancements should be of the form "Advancements ..." (with optional leading article) (golint)
    • Line 846: warning: comment on exported type AdvancementMappingItem should be of the form "AdvancementMappingItem ..." (with optional leading article) (golint)
    • Line 852: warning: comment on exported type Advancement should be of the form "Advancement ..." (with optional leading article) (golint)
    • Line 862: warning: comment on exported type AdvancementDisplay should be of the form "AdvancementDisplay ..." (with optional leading article) (golint)
    • Line 873: warning: comment on exported type AdvancementRequirements should be of the form "AdvancementRequirements ..." (with optional leading article) (golint)
    • Line 878: warning: comment on exported type ProgressMappingItem should be of the form "ProgressMappingItem ..." (with optional leading article) (golint)
    • Line 884: warning: comment on exported type AdvancementProgress should be of the form "AdvancementProgress ..." (with optional leading article) (golint)
    • Line 889: warning: comment on exported type ProgressCriteria should be of the form "ProgressCriteria ..." (with optional leading article) (golint)
    • Line 895: warning: comment on exported type CriterionProgress should be of the form "CriterionProgress ..." (with optional leading article) (golint)
    • MOpher/blocksign.go
    • Line 15: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 30: warning: exported type SignComponent should have comment or be unexported (golint)
    • MOpher/encoding/nbt/nbt.go
    • Line 10: warning: exported type TypeID should have comment or be unexported (golint)
    • Line 13: warning: exported const TagEnd should have comment (or a comment on this block) or be unexported (golint)
    • Line 28: warning: exported var ErrInvalidCompound should have comment or be unexported (golint)
    • Line 31: warning: exported type Compound should have comment or be unexported (golint)
    • Line 36: warning: exported function NewCompound should have comment or be unexported (golint)
    • Line 42: warning: exported method Compound.Serialize should have comment or be unexported (golint)
    • Line 128: warning: exported method Compound.Deserialize should have comment or be unexported (golint)
    • Line 159: warning: exported type List should have comment or be unexported (golint)
    • MOpher/Protocol/versions/338/335.go
    • Line 3: warning: don't use an underscore in package name (golint)
    • Line 9: warning: exported type This should have comment or be unexported (golint)
    • Line 14: warning: exported var P should have comment or be unexported (golint)
    • Line 16: warning: exported method This.Packets should have comment or be unexported (golint)
    • Line 20: warning: exported method This.Translate should have comment or be unexported (golint)
    • Line 24: warning: exported method This.Back should have comment or be unexported (golint)
    • Line 28: warning: exported method This.InitTranslate should have comment or be unexported (golint)
    • Line 33: warning: exported function Version should have comment or be unexported (golint)
    • MOpher/Protocol/connection.go
    • Line 263: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 274: warning: exported function MakeByte should have comment or be unexported (golint)
    • Line 372: warning: error should be the last type when returning multiple items (golint)
    • Line 372: warning: exported method Conn.ResolveConnectable should have comment or be unexported (golint)
    • MOpher/Protocol/versions/340/340.go
    • Line 3: warning: don't use an underscore in package name (golint)
    • Line 9: warning: exported type This should have comment or be unexported (golint)
    • Line 14: warning: exported var P should have comment or be unexported (golint)
    • Line 16: warning: exported method This.Packets should have comment or be unexported (golint)
    • Line 20: warning: exported method This.Translate should have comment or be unexported (golint)
    • Line 24: warning: exported method This.Back should have comment or be unexported (golint)
    • Line 28: warning: exported method This.InitTranslate should have comment or be unexported (golint)
    • Line 33: warning: exported function Version should have comment or be unexported (golint)
    • MOpher/MO.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 11: warning: exported var NetworkLogLevel should have comment or be unexported (golint)
    • Line 17: warning: exported function C should have comment or be unexported (golint)
    • Line 24: warning: exported function CS should have comment or be unexported (golint)
    • Line 28: warning: exported function L should have comment or be unexported (golint)
    • Line 35: warning: exported function LS should have comment or be unexported (golint)
    • Line 47: warning: exported function Chat should have comment or be unexported (golint)
    • Line 51: warning: exported var T should have comment or be unexported (golint)
    • Line 52: warning: exported var ST should have comment or be unexported (golint)
    • MOpher/structs.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 28: warning: exported type MetadataComponent should have comment or be unexported (golint)
    • Line 43: warning: exported type NetworkComponent should have comment or be unexported (golint)
    • Line 63: warning: exported type PositionComponent should have comment or be unexported (golint)
    • Line 87: warning: exported type TargetPositionComponent should have comment or be unexported (golint)
    • Line 107: warning: exported type RotationComponent should have comment or be unexported (golint)
    • Line 133: warning: exported type TargetRotationComponent should have comment or be unexported (golint)
    • Line 148: warning: exported type SizeComponent should have comment or be unexported (golint)
    • Line 165: warning: exported type PlayerComponent should have comment or be unexported (golint)
    • Line 180: warning: exported type DebugComponent should have comment or be unexported (golint)
    • Line 226: warning: exported type Inventory should have comment or be unexported (golint)
    • Line 232: warning: exported function NewInventory should have comment or be unexported (golint)
    • Line 239: warning: exported method Inventory.Close should have comment or be unexported (golint)
    • Line 258: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 283: warning: exported type ItemStack should have comment or be unexported (golint)
    • Line 292: warning: exported method ItemStack.StackTo should have comment or be unexported (golint)
    • Line 304: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 313: warning: exported method ItemStack.PopTo should have comment or be unexported (golint)
    • Line 343: warning: exported method ItemStack.RightGrabToCursor should have comment or be unexported (golint)
    • Line 370: warning: exported function ItemStackFromProtocol should have comment or be unexported (golint)
    • Line 390: warning: exported function ItemStackToProtocol should have comment or be unexported (golint)
    • Line 402: warning: exported type ItemType should have comment or be unexported (golint)
    • Line 412: warning: exported function ItemById should have comment or be unexported (golint)
    • Line 452: warning: exported type DisplayTag should have comment or be unexported (golint)
    • Line 463: warning: exported function ItemOfBlock should have comment or be unexported (golint)
    • Line 514: warning: exported type ItemDamagable should have comment or be unexported (golint)
    • Line 587: warning: comment on exported type BaseBlock should be of the form "BaseBlock ..." (with optional leading article) (golint)
    • Line 606: warning: exported method BaseBlock.BlockSet should have comment or be unexported (golint)
    • Line 626: warning: exported method BaseBlock.StepSound should have comment or be unexported (golint)
    • Line 627: warning: exported method BaseBlock.DigSound should have comment or be unexported (golint)
    • Line 628: warning: exported method BaseBlock.BreakSound should have comment or be unexported (golint)
    • Line 630: warning: exported method BaseBlock.NameLocaleKey should have comment or be unexported (golint)
    • Line 634: warning: exported method BaseBlock.Hardness should have comment or be unexported (golint)
    • Line 642: warning: exported method BaseBlock.Plugin should have comment or be unexported (golint)
    • Line 646: warning: exported method BaseBlock.Name should have comment or be unexported (golint)
    • Line 650: warning: exported method BaseBlock.SID should have comment or be unexported (golint)
    • Line 654: warning: exported method BaseBlock.Collidable should have comment or be unexported (golint)
    • Line 658: warning: exported method BaseBlock.CollisionBounds should have comment or be unexported (golint)
    • Line 667: warning: exported method BaseBlock.Renderable should have comment or be unexported (golint)
    • Line 671: warning: exported method BaseBlock.ModelName should have comment or be unexported (golint)
    • Line 674: warning: exported method BaseBlock.ModelVariant should have comment or be unexported (golint)
    • Line 678: warning: exported method BaseBlock.LightReduction should have comment or be unexported (golint)
    • Line 685: warning: exported method BaseBlock.LightEmitted should have comment or be unexported (golint)
    • Line 689: warning: exported method BaseBlock.ShouldCullAgainst should have comment or be unexported (golint)
    • Line 693: warning: exported method BaseBlock.ForceShade should have comment or be unexported (golint)
    • Line 697: warning: exported method BaseBlock.TintImage should have comment or be unexported (golint)
    • Line 701: warning: exported method BaseBlock.TintColor should have comment or be unexported (golint)
    • Line 705: warning: exported method BaseBlock.IsTranslucent should have comment or be unexported (golint)
    • Line 709: warning: exported method BaseBlock.UpdateState should have comment or be unexported (golint)
    • Line 713: warning: exported method BaseBlock.Set should have comment or be unexported (golint)
    • Line 903: warning: exported method BlockSet.Stringify should have comment or be unexported (golint)
    • Line 947: warning: exported type Position should have comment or be unexported (golint)
    • Line 951: warning: exported method Position.Shift should have comment or be unexported (golint)
    • Line 955: warning: exported method Position.ShiftDir should have comment or be unexported (golint)
    • Line 959: warning: exported method Position.Get should have comment or be unexported (golint)
    • Line 963: warning: exported method Position.Vec should have comment or be unexported (golint)
    • MOpher/Protocol/versions/335/play_clientbound.go
    • Line 3: warning: don't use an underscore in package name (golint)
    • Line 363: warning: exported type BlockEntity should have comment or be unexported (golint)
    • Line 481: warning: comment on exported type VehicleMove should be of the form "VehicleMove ..." (with optional leading article) (golint)
    • Line 498: warning: comment on exported type CraftReceipeResponse should be of the form "CraftReceipeResponse ..." (with optional leading article) (golint)
    • Line 575: warning: comment on exported type UnlockReceipes should be of the form "UnlockReceipes ..." (with optional leading article) (golint)
    • Line 629: warning: comment on exported type SelectAdvancementTab should be of the form "SelectAdvancementTab ..." (with optional leading article) (golint)
    • Line 739: warning: comment on exported type Passengers should be of the form "Passengers ..." (with optional leading article) (golint)
    • Line 805: warning: comment on exported type HardSoundEffect should be of the form "HardSoundEffect ..." (with optional leading article) (golint)
    • Line 845: warning: comment on exported type Advancements should be of the form "Advancements ..." (with optional leading article) (golint)
    • Line 853: warning: comment on exported type AdvancementMappingItem should be of the form "AdvancementMappingItem ..." (with optional leading article) (golint)
    • Line 859: warning: comment on exported type Advancement should be of the form "Advancement ..." (with optional leading article) (golint)
    • Line 869: warning: comment on exported type AdvancementDisplay should be of the form "AdvancementDisplay ..." (with optional leading article) (golint)
    • Line 880: warning: comment on exported type AdvancementRequirements should be of the form "AdvancementRequirements ..." (with optional leading article) (golint)
    • Line 885: warning: comment on exported type ProgressMappingItem should be of the form "ProgressMappingItem ..." (with optional leading article) (golint)
    • Line 891: warning: comment on exported type AdvancementProgress should be of the form "AdvancementProgress ..." (with optional leading article) (golint)
    • Line 896: warning: comment on exported type ProgressCriteria should be of the form "ProgressCriteria ..." (with optional leading article) (golint)
    • Line 902: warning: comment on exported type CriterionProgress should be of the form "CriterionProgress ..." (with optional leading article) (golint)
    • MOpher/inventory.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 11: warning: exported type InventoryInterface should have comment or be unexported (golint)
    • Line 16: warning: exported var II should have comment or be unexported (golint)
    • Line 26: warning: exported const CURSORSLOT should have comment or be unexported (golint)
    • Line 42: warning: comment on exported method InventoryInterface.PickUp should be of the form "PickUp ..." (golint)
    • Line 80: warning: comment on exported method InventoryInterface.Drop should be of the form "Drop ..." (golint)
    • Line 115: warning: exported method InventoryInterface.Swap should have comment or be unexported (golint)
    • Line 143: warning: exported method InventoryInterface.RightClick should have comment or be unexported (golint)
    • Line 180: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 228: warning: exported method InventoryInterface.CursorIsHolding should have comment or be unexported (golint)
    • Line 232: warning: exported method InventoryInterface.DropCursor should have comment or be unexported (golint)
    • Line 244: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • MOpher/play.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 13: warning: comment on exported function Dig should be of the form "Dig ..." (golint)
    • Line 191: warning: exported function Use should have comment or be unexported (golint)
    • Line 206: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 268: warning: error var notool should have name of the form errFoo (golint)
    • Line 270: warning: exported function Issword should have comment or be unexported (golint)
    • Line 274: warning: exported function Isaxe should have comment or be unexported (golint)
    • Line 278: warning: exported function Ispick should have comment or be unexported (golint)
    • Line 282: warning: exported function Thepick should have comment or be unexported (golint)
    • Line 298: warning: exported function Isshovel should have comment or be unexported (golint)
    • Line 302: warning: exported function Typeof should have comment or be unexported (golint)
    • Line 318: warning: comment on exported function Swap should be of the form "Swap ..." (golint)
    • Line 324: warning: error var DEFBLOCKAIR should have name of the form ErrFoo (golint)
    • Line 324: warning: exported var DEFBLOCKAIR should have comment or be unexported (golint)
    • Line 325: warning: error var NOTMINABLE should have name of the form ErrFoo (golint)
    • MOpher/type/vmath/frustum.go
    • Line 23: warning: exported type Frustum should have comment or be unexported (golint)
    • Line 44: warning: exported function NewFrustum should have comment or be unexported (golint)
    • Line 56: warning: exported method Frustum.SetPerspective should have comment or be unexported (golint)
    • Line 69: warning: exported method Frustum.SetCamera should have comment or be unexported (golint)
    • Line 107: warning: exported method Frustum.IsSphereInside should have comment or be unexported (golint)
    • Line 118: warning: exported method Frustum.IsAABBInside should have comment or be unexported (golint)
    • MOpher/Protocol/versions/338/play_clientbound.go
    • Line 3: warning: don't use an underscore in package name (golint)
    • Line 363: warning: exported type BlockEntity should have comment or be unexported (golint)
    • Line 481: warning: comment on exported type VehicleMove should be of the form "VehicleMove ..." (with optional leading article) (golint)
    • Line 498: warning: comment on exported type CraftReceipeResponse should be of the form "CraftReceipeResponse ..." (with optional leading article) (golint)
    • Line 575: warning: comment on exported type UnlockReceipes should be of the form "UnlockReceipes ..." (with optional leading article) (golint)
    • Line 629: warning: comment on exported type SelectAdvancementTab should be of the form "SelectAdvancementTab ..." (with optional leading article) (golint)
    • Line 739: warning: comment on exported type Passengers should be of the form "Passengers ..." (with optional leading article) (golint)
    • Line 805: warning: comment on exported type HardSoundEffect should be of the form "HardSoundEffect ..." (with optional leading article) (golint)
    • Line 845: warning: comment on exported type Advancements should be of the form "Advancements ..." (with optional leading article) (golint)
    • Line 854: warning: comment on exported type AdvancementMappingItem should be of the form "AdvancementMappingItem ..." (with optional leading article) (golint)
    • Line 860: warning: comment on exported type Advancement should be of the form "Advancement ..." (with optional leading article) (golint)
    • Line 870: warning: comment on exported type AdvancementDisplay should be of the form "AdvancementDisplay ..." (with optional leading article) (golint)
    • Line 881: warning: comment on exported type AdvancementRequirements should be of the form "AdvancementRequirements ..." (with optional leading article) (golint)
    • Line 886: warning: comment on exported type ProgressMappingItem should be of the form "ProgressMappingItem ..." (with optional leading article) (golint)
    • Line 892: warning: comment on exported type AdvancementProgress should be of the form "AdvancementProgress ..." (with optional leading article) (golint)
    • Line 897: warning: comment on exported type ProgressCriteria should be of the form "ProgressCriteria ..." (with optional leading article) (golint)
    • Line 903: warning: comment on exported type CriterionProgress should be of the form "CriterionProgress ..." (with optional leading article) (golint)
    • MOpher/Protocol/lib/io.go
    • Line 11: warning: exported const VarPart should have comment or be unexported (golint)
    • Line 12: warning: exported const VarPartLong should have comment or be unexported (golint)
    • Line 23: warning: exported function VarIntSize should have comment or be unexported (golint)
    • MOpher/blocks.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 9: warning: comment on exported var Blocks should be of the form "Blocks ..." (golint)
    • Line 226: warning: exported var Hardness should have comment or be unexported (golint)
    • MOpher/chunk.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 18: warning: exported var ChunkMap should have comment or be unexported (golint)
    • MOpher/script.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 13: warning: exported var SE should have comment or be unexported (golint)
    • Line 15: warning: don't use underscores in Go names; var lua_cmds should be luaCmds (golint)
    • Line 17: warning: don't use underscores in Go names; var lua_started should be luaStarted (golint)
    • Line 19: warning: don't use underscores in Go names; func lua_onload should be luaOnload (golint)
    • Line 39: warning: exported function Lua_eval should have comment or be unexported (golint)
    • Line 39: warning: don't use underscores in Go names; func Lua_eval should be LuaEval (golint)
    • Line 46: warning: don't use underscores in Go names; var curr_path should be currPath (golint)
    • Line 64: warning: don't use underscores in Go names; func lua_bot_loader should be luaBotLoader (golint)
    • Line 71: warning: don't use underscores in Go names; func lua_inv_loader should be luaInvLoader (golint)
    • Line 97: warning: don't use underscores in Go names; func lua_NAV should be luaNAV (golint)
    • Line 109: warning: don't use underscores in Go names; func lua_LOG should be luaLOG (golint)
    • Line 131: warning: don't use underscores in Go names; func lua_BLOCK should be luaBLOCK (golint)
    • Line 137: warning: don't use underscores in Go names; func lua_POS should be luaPOS (golint)
    • Line 146: warning: don't use underscores in Go names; func lua_INV should be luaINV (golint)
    • Line 169: warning: don't use underscores in Go names; func lua_CURSORITEM should be luaCURSORITEM (golint)
    • Line 191: warning: don't use underscores in Go names; func lua_NEARITEMS should be luaNEARITEMS (golint)
    • Line 208: warning: don't use underscores in Go names; func inv_pickup should be invPickup (golint)
    • Line 213: warning: don't use underscores in Go names; func inv_drop should be invDrop (golint)
    • Line 218: warning: don't use underscores in Go names; func inv_swap should be invSwap (golint)
    • Line 223: warning: don't use underscores in Go names; func inv_cursorIsHolding should be invCursorIsHolding (golint)
    • MOpher/window.go
    • Line 1: warning: don't use MixedCaps in package name; MO should be mo (golint)
    • Line 11: warning: error var closedError should have name of the form errFoo (golint)
    • Line 12: warning: error var cannotSwap should have name of the form errFoo (golint)
    • Line 13: warning: error var invalidInventorySlot should have name of the form errFoo (golint)
    • Line 14: warning: error var invalidWindowSlot should have name of the form errFoo (golint)
    • Line 106: warning: exported type WindowType should have comment or be unexported (golint)
    • Line 182: warning: exported const ContainerWindow should have comment (or a comment on this block) or be unexported (golint)
    • Line 198: warning: exported type Window should have comment or be unexported (golint)
    • Line 211: warning: exported var OpenWindow should have comment or be unexported (golint)
    • Line 213: warning: exported type BaseWindow should have comment or be unexported (golint)
    • Line 280: warning: exported method BaseWindow.GetInterface should have comment or be unexported (golint)
    • Line 284: warning: exported method BaseWindow.GetType 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 296: warning: exported method BaseWindow.GetInventory should have comment or be unexported (golint)
    • Line 300: warning: exported method BaseWindow.Close should have comment or be unexported (golint)
    • Line 308: warning: exported method BaseWindow.IsClosed should have comment or be unexported (golint)
    • Line 312: warning: exported method BaseWindow.TransferComplex should have comment or be unexported (golint)
    • Line 336: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 345: warning: exported type WChest should have comment or be unexported (golint)
    • Line 349: warning: exported method WChest.TransferTo should have comment or be unexported (golint)
    • Line 365: warning: exported method WChest.TransferFrom should have comment or be unexported (golint)
    • Line 381: warning: exported type WCraft should have comment or be unexported (golint)
    • Line 385: warning: exported method WCraft.TransferTo should have comment or be unexported (golint)
    • Line 401: warning: exported method WCraft.TransferFrom should have comment or be unexported (golint)
    • MOpher/Protocol/versions/338/play_serverbound.go
    • Line 3: warning: don't use an underscore in package name (golint)
    • Line 9: warning: comment on exported type TeleConfirm should be of the form "TeleConfirm ..." (with optional leading article) (golint)
    • Line 154: warning: comment on exported type VehicleDrive should be of the form "VehicleDrive ..." (with optional leading article) (golint)
    • Line 162: warning: comment on exported type SteerBoat should be of the form "SteerBoat ..." (with optional leading article) (golint)
    • Line 169: warning: comment on exported type CraftReceipeRequest should be of the form "CraftReceipeRequest ..." (with optional leading article) (golint)
    • Line 216: warning: comment on exported type CraftingBookData should be of the form "CraftingBookData ..." (with optional leading article) (golint)
    • Line 233: warning: comment on exported type AdvancementTab should be of the form "AdvancementTab ..." (with optional leading article) (golint)
    • MOpher/type/bit/map.go
    • Line 19: warning: exported type Map should have comment or be unexported (golint)
    • Line 25: warning: exported function NewMap should have comment or be unexported (golint)
    • Line 33: warning: exported function NewMapFromRaw should have comment or be unexported (golint)
    • Line 41: warning: exported method Map.ResizeBits should have comment or be unexported (golint)
    • Line 49: warning: exported method Map.Set should have comment or be unexported (golint)
    • Line 66: warning: exported method Map.Get should have comment or be unexported (golint)
    • Line 75: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • MOpher/type/vmath/aabb.go
    • Line 23: warning: exported type AABB should have comment or be unexported (golint)
    • Line 28: warning: exported function NewAABB should have comment or be unexported (golint)
    • Line 35: warning: exported method AABB.RotateX should have comment or be unexported (golint)
    • Line 44: warning: exported method AABB.RotateY should have comment or be unexported (golint)
    • Line 61: warning: exported method AABB.Intersects should have comment or be unexported (golint)
    • Line 70: warning: exported method AABB.IntersectsLine should have comment or be unexported (golint)
    • Line 124: warning: exported method AABB.Shift should have comment or be unexported (golint)
    • Line 134: warning: exported method AABB.Grow should have comment or be unexported (golint)
    • Line 144: warning: exported method AABB.MoveOutOf should have comment or be unexported (golint)

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!

license0%

Checks whether your project has a LICENSE file.

misspell87%

Misspell Finds commonly misspelled English words

    • MOpher/nav_test.go
    • Line 10: warning: "GLOBALY" is a misspelling of "GLOBALLY" (misspell)
    • Line 14: warning: "GLOBALY" is a misspelling of "GLOBALLY" (misspell)