Forge Mod detection #292
@ -25,9 +25,7 @@ import java.util.List;
|
||||
|
||||
public class FMLPing extends ServerPing {
|
||||
|
||||
|
zOnlyKroks markierte diese Unterhaltung als gelöst
Veraltet
|
||||
static {
|
||||
ForgeData forgeData = new ForgeData();
|
||||
}
|
||||
private static final ForgeData data = new ForgeData();
|
||||
|
Lixfel markierte diese Unterhaltung als gelöst
Veraltet
YoyoNow
hat
Also von einem static block war nicht die rede nur den static modifier. Also sowas wie 'private static ForgeData forgeData = new ForgeData();'. Also von einem static block war nicht die rede nur den static modifier. Also sowas wie 'private static ForgeData forgeData = new ForgeData();'.
YoyoNow
hat
Wo wird das überhaupt genutzt? Wo wird das überhaupt genutzt?
Lixfel
hat
GSON-Serializer des ServerListPings. Anmerkung meinerseits: Ob das korrekt serialisiert wird, wenn die ForgeData als static final deklariert wird, muss getestet werden. GSON-Serializer des ServerListPings.
Anmerkung meinerseits: Ob das korrekt serialisiert wird, wenn die ForgeData als static final deklariert wird, muss getestet werden.
YoyoNow
hat
Naja es ist static also gar nicht serializiert? Ansonsten mach halt mit einem instance ansatz, im ForgeData selber. Naja es ist static also gar nicht serializiert? Ansonsten mach halt mit einem instance ansatz, im ForgeData selber.
Lixfel
hat
Ist unbekannt ob, muss daher getestet werden. Ist unbekannt ob, muss daher getestet werden.
Lixfel
hat
Ich glaube, das muss weiterhin forgeData heißen, ansonsten erkennt Forge die Modliste nicht... du solltest auch mal probieren, mit Minecraft-Versionen anders als 1.15.2, ob dann immer noch die Kompatibilität gemeldet wird (da wir ja minecraft 1.15.2 als Mod drin haben... Ich glaube, das muss weiterhin forgeData heißen, ansonsten erkennt Forge die Modliste nicht... du solltest auch mal probieren, mit Minecraft-Versionen anders als 1.15.2, ob dann immer noch die Kompatibilität gemeldet wird (da wir ja minecraft 1.15.2 als Mod drin haben...
zOnlyKroks
hat
static mact hier in diesem Falle nur probleme static mact hier in diesem Falle nur probleme
zOnlyKroks
hat
Kompatibilität wird verneint. Forge 1.16.5 Kompatibilität wird verneint. Forge 1.16.5
|
||||
|
||||
public FMLPing(ServerPing existing) {
|
||||
super(existing.getVersion(), existing.getPlayers(), existing.getDescriptionComponent(), existing.getFaviconObject());
|
||||
|
||||
Man könnte noch argumentieren, dass es äußerst schade ist, dass wir hier für jeden Ping die ForeData neu erstellen und danach wieder verwerfen, aber das ist so schon in Ordnung.
Könnte man das nicht einfach mit einem 'static' beheben?
wird morgn als erledigt markiert.