Remove some more unncessary/doubled map check

common/src/main/java/us/myles/ViaVersion/protocols/protocol1_11to1_10/storage/EntityTracker.java

@@ -33,9 +33,7 @@ public class EntityTracker extends StoredObject {
     }
 
     public Optional<Entity1_11Types.EntityType> get(int id) {
-        if (!has(id))
+        return Optional.fromNullable(clientEntityTypes.get(id));
-            return Optional.absent();
-        return Optional.of(clientEntityTypes.get(id));
     }
 
     public void addHologram(int entId) {

common/src/main/java/us/myles/ViaVersion/protocols/protocol1_12to1_11_1/storage/EntityTracker.java

@@ -28,9 +28,7 @@ public class EntityTracker extends StoredObject {
     }
 
     public Optional<Entity1_12Types.EntityType> get(int id) {
-        if (!has(id))
+        return Optional.fromNullable(clientEntityTypes.get(id));
-            return Optional.absent();
-        return Optional.of(clientEntityTypes.get(id));
     }
 
 }

common/src/main/java/us/myles/ViaVersion/protocols/protocol1_13to1_12_2/providers/PaintingProvider.java

@@ -46,12 +46,6 @@ public class PaintingProvider implements Provider {
         // Handle older versions
         if (!motive.startsWith("minecraft:"))
             motive = "minecraft:" + motive.toLowerCase();
-
+        return Optional.fromNullable(paintings.get(motive));
-        Integer index = paintings.get(motive);
-        if (index != null) {
-            return Optional.of(index);
-        }
-
-        return Optional.absent();
     }
 }

common/src/main/java/us/myles/ViaVersion/protocols/protocol1_13to1_12_2/storage/EntityTracker.java

@@ -28,9 +28,7 @@ public class EntityTracker extends StoredObject {
     }
 
     public Optional<Entity1_13Types.EntityType> get(int id) {
-        if (!has(id))
+        return Optional.fromNullable(clientEntityTypes.get(id));
-            return Optional.absent();
-        return Optional.of(clientEntityTypes.get(id));
     }
 
 }

common/src/main/java/us/myles/ViaVersion/protocols/protocol1_9to1_8/metadata/MetaIndex.java

@@ -173,12 +173,7 @@ public enum MetaIndex {
 
     private static Optional<MetaIndex> getIndex(Entity1_10Types.EntityType type, int index) {
         Pair pair = new Pair<>(type, index);
-        MetaIndex metaIndex = metadataRewrites.get(pair);
+        return Optional.fromNullable(metadataRewrites.get(pair));
-        if (metaIndex != null) {
-            return Optional.of(metaIndex);
-        }
-
-        return Optional.absent();
     }
 
     public static MetaIndex searchIndex(Entity1_10Types.EntityType type, int index) {

common/src/main/java/us/myles/ViaVersion/protocols/protocol1_9to1_8/storage/CommandBlockStorage.java

@@ -57,11 +57,14 @@ public class CommandBlockStorage extends StoredObject {
         if (blocks == null)
             return Optional.absent();
 
-        CompoundTag tag = blocks.get(position).clone();
+        CompoundTag tag = blocks.get(position);
+        if (tag == null)
+            return Optional.absent();
+
+        tag = tag.clone();
         tag.put(new ByteTag("powered", (byte) 0));
         tag.put(new ByteTag("auto", (byte) 0));
         tag.put(new ByteTag("conditionMet", (byte) 0));
-
         return Optional.of(tag);
     }
 

common/src/main/java/us/myles/ViaVersion/util/Config.java

@@ -183,7 +183,7 @@ public abstract class Config implements ConfigurationProvider {
     public List<Integer> getIntegerList(String key) {
         Object o = this.config.get(key);
         if (o != null) {
-            return (List<Integer>) this.config.get(key);
+            return (List<Integer>) o;
         } else {
             return new ArrayList<>();
         }