removed all 'structcheck' and 'unused' lint errors (#839)

d2core/d2asset/asset_manager.go

@@ -425,6 +425,9 @@ func (am *AssetManager) loadDCC(path string,
 	}
 
 	animation, err := newDCCAnimation(dcc, palette, effect)
+	if err != nil {
+		return nil, err
+	}
 
 	return animation, nil
 }

d2core/d2item/diablo2item/item.go

@@ -53,6 +53,7 @@ const (
 var _ d2item.Item = &Item{}
 
 // Item is a representation of a diablo2 item
+// nolint:structcheck,unused // WIP
 type Item struct {
 	factory *ItemFactory
 	name    string
@@ -83,6 +84,7 @@ type Item struct {
 	sockets []*d2item.Item // there will be checks for handling the craziness this might entail
 }
 
+// nolint:structcheck,unused // WIP
 type itemAttributes struct {
 	worldSprite     *d2ui.Sprite
 	inventorySprite *d2ui.Sprite
@@ -118,7 +120,7 @@ type itemAttributes struct {
 type minMaxEnhanceable struct {
 	min     int
 	max     int
-	enhance int
+	enhance int // nolint:structcheck,unused // WIP
 }
 
 // Label returns the item name

d2core/d2map/d2mapentity/animated_entity.go

@@ -7,6 +7,7 @@ import (
 )
 
 // AnimatedEntity represents an animation that can be projected onto the map.
+// nolint:structcheck,unused // variables are unused because it is incomplete
 type AnimatedEntity struct {
 	mapEntity
 	animation d2interface.Animation

d2game/d2gamescreen/main_menu.go

@@ -116,7 +116,7 @@ type MainMenu struct {
 	inputManager  d2interface.InputManager
 	renderer      d2interface.Renderer
 	audioProvider d2interface.AudioProvider
-	scriptEngine  *d2script.ScriptEngine
+	scriptEngine  *d2script.ScriptEngine // nolint:structcheck,unused // it will be used...
 	navigator     d2interface.Navigator
 	uiManager     *d2ui.UIManager
 	heroState     *d2hero.HeroStateFactory

d2networking/d2client/d2remoteclient/remote_client_connection.go

@@ -36,6 +36,7 @@ func Create(asset *d2asset.AssetManager) (*RemoteClientConnection, error) {
 	}
 
 	result := &RemoteClientConnection{
+		asset:     asset,
 		heroState: heroStateFactory,
 		uniqueID:  uuid.New().String(),
 	}
@@ -149,6 +150,7 @@ func (r *RemoteClientConnection) serverListener() {
 }
 
 // bytesToJSON reads the packet type, decompresses the packet and returns a JSON string.
+// nolint:unused // WIP
 func (r *RemoteClientConnection) bytesToJSON(buffer []byte) (string, d2netpackettype.NetPacketType, error) {
 	packet, err := d2netpacket.UnmarshalNetPacket(buffer)
 	if err != nil {