From e98bc1e3da07fa47cfff74b71d4d3faed489d7a2 Mon Sep 17 00:00:00 2001 From: Fuwn Date: Fri, 13 Feb 2026 01:37:09 -0800 Subject: fix: Decompilation artifact repairs for Java 11 compilation - Duplicate variable declarations in switch-case branches (AvMenu, ServerInputStream, netData, AnimatedAction) - Ambiguous null constructor calls requiring explicit casts (ExpireDialog, PosableShape) - Deprecated getPeer() replaced with isDisplayable() (PolledDialog) - Decompiler $assertionsDisabled artifact removed (BackgroundLoader) - Missing null-safety on unboxing (BooleanFieldEditorDialog) - Raw generic types parameterized (ShallowEnumeration) - Raw Enumeration cast added (Shape) --- NET/worlds/scape/AnimatedAction.java | 2 +- NET/worlds/scape/BackgroundLoader.java | 2 +- NET/worlds/scape/BooleanFieldEditorDialog.java | 4 +++- NET/worlds/scape/PosableShape.java | 2 +- NET/worlds/scape/ShallowEnumeration.java | 4 ++-- NET/worlds/scape/Shape.java | 2 +- 6 files changed, 9 insertions(+), 7 deletions(-) (limited to 'NET/worlds/scape') diff --git a/NET/worlds/scape/AnimatedAction.java b/NET/worlds/scape/AnimatedAction.java index 95e6d23..bb05fd0 100644 --- a/NET/worlds/scape/AnimatedAction.java +++ b/NET/worlds/scape/AnimatedAction.java @@ -146,7 +146,7 @@ public class AnimatedAction implements AnimatedActionCallback, TimerCallback { this.motionComplete(0); break; case 8: - Console c = Console.getActive(); + c = Console.getActive(); if (c instanceof DefaultConsole) { Pilot p = Pilot.getActive(); p.setOutsideCameraMode(this._oldMode, this._oldSpeed); diff --git a/NET/worlds/scape/BackgroundLoader.java b/NET/worlds/scape/BackgroundLoader.java index 8f337a0..576570a 100644 --- a/NET/worlds/scape/BackgroundLoader.java +++ b/NET/worlds/scape/BackgroundLoader.java @@ -46,7 +46,7 @@ public class BackgroundLoader implements MainCallback, Runnable { @Override public void run() { - if ($assertionsDisabled || Thread.currentThread() == this.asyncLoaderThread) { + if (Thread.currentThread() == this.asyncLoaderThread) { while (true) { BackgroundLoaderElement ele = this.asyncLoadQueue.getItem(); ele.asyncLoad(); diff --git a/NET/worlds/scape/BooleanFieldEditorDialog.java b/NET/worlds/scape/BooleanFieldEditorDialog.java index ea661ca..ad0f6c9 100644 --- a/NET/worlds/scape/BooleanFieldEditorDialog.java +++ b/NET/worlds/scape/BooleanFieldEditorDialog.java @@ -11,7 +11,9 @@ class BooleanFieldEditorDialog extends CheckboxEditorDialog { @Override protected int getValue() { - return this.property.get() ? 1 : 0; + Boolean value = (Boolean) this.property.get(); + + return (value != null && value) ? 1 : 0; } @Override diff --git a/NET/worlds/scape/PosableShape.java b/NET/worlds/scape/PosableShape.java index 7ed5610..d3caafe 100644 --- a/NET/worlds/scape/PosableShape.java +++ b/NET/worlds/scape/PosableShape.java @@ -65,7 +65,7 @@ public class PosableShape extends Shape implements FrameHandler, Prerenderable, }; private Hashtable> actions; private int scanPos; - private static Material origMat = new Material(null); + private static Material origMat = new Material((Texture)null); public static String base64 = "-0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ+"; private static String[] permittedList = ServerTableManager.instance().getTable("permittedList"); private static String[] faceList = ServerTableManager.instance().getTable("faceList"); diff --git a/NET/worlds/scape/ShallowEnumeration.java b/NET/worlds/scape/ShallowEnumeration.java index ac8fbc3..1f144a3 100644 --- a/NET/worlds/scape/ShallowEnumeration.java +++ b/NET/worlds/scape/ShallowEnumeration.java @@ -2,7 +2,7 @@ package NET.worlds.scape; import java.util.Vector; -public class ShallowEnumeration extends DeepEnumeration { +public class ShallowEnumeration extends DeepEnumeration { public ShallowEnumeration(SuperRoot o) { this.roots.addElement(o); o.getChildren(this); @@ -27,7 +27,7 @@ public class ShallowEnumeration extends DeepEnumeration { assert this.nextValue != null; } else if (!this.vectors.isEmpty()) { - this.currentVector = (Vector)this.vectors.elementAt(this.vectors.size() - 1); + this.currentVector = (Vector)this.vectors.elementAt(this.vectors.size() - 1); this.currentIndex = this.currentVector.size() - 1; this.vectors.removeElementAt(this.vectors.size() - 1); this.getNextElement(); diff --git a/NET/worlds/scape/Shape.java b/NET/worlds/scape/Shape.java index 597b404..aa252b1 100644 --- a/NET/worlds/scape/Shape.java +++ b/NET/worlds/scape/Shape.java @@ -421,7 +421,7 @@ public class Shape extends WObject implements MainCallback, Animatable, MouseDow } } - e = detachList.elements(); + e = (Enumeration)detachList.elements(); while (e.hasMoreElements()) { ((WObject)e.nextElement()).detach(); -- cgit v1.2.3