Removed unnecessary ItemList constructor iteration.

This commit is contained in:
MobiusDevelopment
2019-11-07 17:08:56 +00:00
parent 89246e697a
commit df4a4e1321

View File

@@ -16,9 +16,6 @@
*/ */
package org.l2jmobius.gameserver.network.serverpackets; package org.l2jmobius.gameserver.network.serverpackets;
import java.util.ArrayList;
import java.util.List;
import org.l2jmobius.commons.network.PacketWriter; import org.l2jmobius.commons.network.PacketWriter;
import org.l2jmobius.gameserver.model.actor.instance.PlayerInstance; import org.l2jmobius.gameserver.model.actor.instance.PlayerInstance;
import org.l2jmobius.gameserver.model.items.instance.ItemInstance; import org.l2jmobius.gameserver.model.items.instance.ItemInstance;
@@ -27,18 +24,14 @@ import org.l2jmobius.gameserver.network.OutgoingPackets;
public class ItemList extends AbstractItemPacket public class ItemList extends AbstractItemPacket
{ {
private final PlayerInstance _player; private final PlayerInstance _player;
private final List<ItemInstance> _items = new ArrayList<>(); private final ItemInstance[] _items;
private final boolean _showWindow; private final boolean _showWindow;
public ItemList(PlayerInstance player, boolean showWindow) public ItemList(PlayerInstance player, boolean showWindow)
{ {
_player = player; _player = player;
_showWindow = showWindow; _showWindow = showWindow;
_items = player.getInventory().getItems();
for (ItemInstance item : player.getInventory().getItems())
{
_items.add(item);
}
} }
@Override @Override
@@ -46,7 +39,7 @@ public class ItemList extends AbstractItemPacket
{ {
OutgoingPackets.ITEM_LIST.writeId(packet); OutgoingPackets.ITEM_LIST.writeId(packet);
packet.writeH(_showWindow ? 0x01 : 0x00); packet.writeH(_showWindow ? 0x01 : 0x00);
packet.writeH(_items.size()); packet.writeH(_items.length);
for (ItemInstance item : _items) for (ItemInstance item : _items)
{ {
writeItem(packet, item); writeItem(packet, item);