Define concrete type in order to avoid accidental re-assignment with another collection
This commit is contained in:
@@ -12,7 +12,6 @@ import net.minecraftforge.common.util.Constants;
|
|||||||
|
|
||||||
import javax.annotation.Nullable;
|
import javax.annotation.Nullable;
|
||||||
import java.util.Collection;
|
import java.util.Collection;
|
||||||
import java.util.Map;
|
|
||||||
import java.util.concurrent.ConcurrentHashMap;
|
import java.util.concurrent.ConcurrentHashMap;
|
||||||
import java.util.function.Function;
|
import java.util.function.Function;
|
||||||
|
|
||||||
@@ -24,7 +23,7 @@ public class NetworkNodeManager extends WorldSavedData implements INetworkNodeMa
|
|||||||
private static final String NBT_NODE_DATA = "Data";
|
private static final String NBT_NODE_DATA = "Data";
|
||||||
private static final String NBT_NODE_POS = "Pos";
|
private static final String NBT_NODE_POS = "Pos";
|
||||||
|
|
||||||
private Map<BlockPos, INetworkNode> nodes = new ConcurrentHashMap<>();
|
private ConcurrentHashMap<BlockPos, INetworkNode> nodes = new ConcurrentHashMap<>();
|
||||||
|
|
||||||
public NetworkNodeManager(String s) {
|
public NetworkNodeManager(String s) {
|
||||||
super(s);
|
super(s);
|
||||||
@@ -32,7 +31,7 @@ public class NetworkNodeManager extends WorldSavedData implements INetworkNodeMa
|
|||||||
|
|
||||||
@Override
|
@Override
|
||||||
public void readFromNBT(NBTTagCompound tag) {
|
public void readFromNBT(NBTTagCompound tag) {
|
||||||
Map<BlockPos, INetworkNode> newNodes = new ConcurrentHashMap<>();
|
ConcurrentHashMap<BlockPos, INetworkNode> newNodes = new ConcurrentHashMap<>();
|
||||||
|
|
||||||
if (tag.hasKey(NBT_NODES)) {
|
if (tag.hasKey(NBT_NODES)) {
|
||||||
NBTTagList list = tag.getTagList(NBT_NODES, Constants.NBT.TAG_COMPOUND);
|
NBTTagList list = tag.getTagList(NBT_NODES, Constants.NBT.TAG_COMPOUND);
|
||||||
|
|||||||
@@ -16,7 +16,7 @@ import java.util.concurrent.CopyOnWriteArrayList;
|
|||||||
|
|
||||||
public class StorageCacheFluid implements IStorageCache<FluidStack> {
|
public class StorageCacheFluid implements IStorageCache<FluidStack> {
|
||||||
private INetwork network;
|
private INetwork network;
|
||||||
private List<IStorage<FluidStack>> storages = new CopyOnWriteArrayList<>();
|
private CopyOnWriteArrayList<IStorage<FluidStack>> storages = new CopyOnWriteArrayList<>();
|
||||||
private IStackList<FluidStack> list = API.instance().createFluidStackList();
|
private IStackList<FluidStack> list = API.instance().createFluidStackList();
|
||||||
|
|
||||||
public StorageCacheFluid(INetwork network) {
|
public StorageCacheFluid(INetwork network) {
|
||||||
|
|||||||
@@ -16,7 +16,7 @@ import java.util.concurrent.CopyOnWriteArrayList;
|
|||||||
|
|
||||||
public class StorageCacheItem implements IStorageCache<ItemStack> {
|
public class StorageCacheItem implements IStorageCache<ItemStack> {
|
||||||
private INetwork network;
|
private INetwork network;
|
||||||
private List<IStorage<ItemStack>> storages = new CopyOnWriteArrayList<>();
|
private CopyOnWriteArrayList<IStorage<ItemStack>> storages = new CopyOnWriteArrayList<>();
|
||||||
private IStackList<ItemStack> list = API.instance().createItemStackList();
|
private IStackList<ItemStack> list = API.instance().createItemStackList();
|
||||||
|
|
||||||
public StorageCacheItem(INetwork network) {
|
public StorageCacheItem(INetwork network) {
|
||||||
|
|||||||
Reference in New Issue
Block a user