Upstream/ViaVersion
1
0
Fork 0
mirror of https://github.com/ViaVersion/ViaVersion.git synced 2024-11-24 19:15:32 +01:00
Code Issues Projects Releases Wiki Activity

Go back to fast entity data list iteration and mutation

Browse Source 
This is a fairly hot method and the array list cannot be modified outside of the already managed event cancellation and provided extra list
This commit is contained in:
Nassim Jahnke 2024-08-25 18:59:48 +02:00
parent 2a71612aea
commit 4f66fd793b
No known key found for this signature in database
GPG Key ID: EF6771C01F6EF02F
1 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
common/src/main/java/com/viaversion/viaversion/rewriter/EntityRewriter.java
View File

@ -110,7 +110,11 @@ public abstract class EntityRewriter<C extends ClientboundPacketType, T extends
public void handleEntityData(final int entityId, final List<EntityData> dataList, final UserConnection connection) {
final TrackedEntity entity = tracker(connection).entity(entityId);
final EntityType type = entity != null ? entity.entityType() : null;
for (final EntityData entityData : dataList.toArray(EMPTY_ARRAY)) { // Copy the list to allow mutation
// Iterate over indexed list to allow for removal and addition of elements, decrease current index and size if an element is removed
int size = dataList.size();
for (int i = 0; i < size; i++) {
final EntityData entityData = dataList.get(i);
EntityDataHandlerEvent event = null;
for (final EntityDataFilter filter : entityDataFilters) {
if (!filter.isFiltered(type, entityData)) {
@ -125,13 +129,15 @@ public abstract class EntityRewriter<C extends ClientboundPacketType, T extends
filter.handler().handle(event, entityData);
} catch (final Exception e) {
logException(e, type, dataList, entityData);
dataList.remove(entityData);
dataList.remove(i--);
size--;
break;
}
if (event.cancelled()) {
// Remove entity data, and break current filter loop
dataList.remove(entityData);
dataList.remove(i--);
size--;
break;
}
}