Invalidate pack caches directly instead of iterating resource manager pack list
This commit is contained in:
parent
d97da8fe4c
commit
d427c5e050
|
|
@ -1,28 +1,18 @@
|
||||||
package org.embeddedt.modernfix.common.mixin.perf.resourcepacks;
|
package org.embeddedt.modernfix.common.mixin.perf.resourcepacks;
|
||||||
|
|
||||||
import net.minecraft.server.packs.PackResources;
|
|
||||||
import net.minecraft.server.packs.resources.SimpleReloadableResourceManager;
|
import net.minecraft.server.packs.resources.SimpleReloadableResourceManager;
|
||||||
import org.embeddedt.modernfix.ModernFix;
|
import org.embeddedt.modernfix.ModernFix;
|
||||||
import org.embeddedt.modernfix.resources.ICachingResourcePack;
|
import org.embeddedt.modernfix.resources.PackResourcesCacheEngine;
|
||||||
import org.spongepowered.asm.mixin.Final;
|
|
||||||
import org.spongepowered.asm.mixin.Mixin;
|
import org.spongepowered.asm.mixin.Mixin;
|
||||||
import org.spongepowered.asm.mixin.Shadow;
|
|
||||||
import org.spongepowered.asm.mixin.injection.At;
|
import org.spongepowered.asm.mixin.injection.At;
|
||||||
import org.spongepowered.asm.mixin.injection.Inject;
|
import org.spongepowered.asm.mixin.injection.Inject;
|
||||||
import org.spongepowered.asm.mixin.injection.callback.CallbackInfoReturnable;
|
import org.spongepowered.asm.mixin.injection.callback.CallbackInfoReturnable;
|
||||||
|
|
||||||
import java.util.List;
|
|
||||||
|
|
||||||
@Mixin(SimpleReloadableResourceManager.class)
|
@Mixin(SimpleReloadableResourceManager.class)
|
||||||
public class ReloadableResourceManagerMixin {
|
public class ReloadableResourceManagerMixin {
|
||||||
@Shadow @Final private List<PackResources> packs;
|
|
||||||
|
|
||||||
@Inject(method = "createReload", at = @At("HEAD"))
|
@Inject(method = "createReload", at = @At("HEAD"))
|
||||||
private void invalidateResourceCaches(CallbackInfoReturnable<?> cir) {
|
private void invalidateResourceCaches(CallbackInfoReturnable<?> cir) {
|
||||||
ModernFix.LOGGER.info("Invalidating pack caches");
|
ModernFix.LOGGER.info("Invalidating pack caches");
|
||||||
for(PackResources pack : this.packs) {
|
PackResourcesCacheEngine.invalidate();
|
||||||
if(pack instanceof ICachingResourcePack)
|
|
||||||
((ICachingResourcePack)pack).invalidateCache();
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -154,4 +154,20 @@ public class PackResourcesCacheEngine {
|
||||||
}
|
}
|
||||||
return resources;
|
return resources;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private static final WeakHashMap<ICachingResourcePack, Boolean> cachingPacks = new WeakHashMap<>();
|
||||||
|
public static void track(ICachingResourcePack pack) {
|
||||||
|
synchronized (cachingPacks) {
|
||||||
|
cachingPacks.put(pack, Boolean.TRUE);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
public static void invalidate() {
|
||||||
|
synchronized (cachingPacks) {
|
||||||
|
cachingPacks.keySet().forEach(pack -> {
|
||||||
|
if(pack != null)
|
||||||
|
pack.invalidateCache();
|
||||||
|
});
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -29,6 +29,7 @@ public abstract class ModNioResourcePackMixin implements ICachingResourcePack {
|
||||||
@Inject(method = "<init>", at = @At("RETURN"))
|
@Inject(method = "<init>", at = @At("RETURN"))
|
||||||
private void cacheResources(CallbackInfo ci) {
|
private void cacheResources(CallbackInfo ci) {
|
||||||
invalidateCache();
|
invalidateCache();
|
||||||
|
PackResourcesCacheEngine.track(this);
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
|
|
|
||||||
|
|
@ -34,6 +34,7 @@ public abstract class ModFileResourcePackMixin implements ICachingResourcePack {
|
||||||
@Inject(method = "<init>", at = @At("TAIL"))
|
@Inject(method = "<init>", at = @At("TAIL"))
|
||||||
private void cacheResources(ModFile modFile, CallbackInfo ci) {
|
private void cacheResources(ModFile modFile, CallbackInfo ci) {
|
||||||
invalidateCache();
|
invalidateCache();
|
||||||
|
PackResourcesCacheEngine.track(this);
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user