From 53452331d8e69dc3bcd40a56a4f00f9380140e73 Mon Sep 17 00:00:00 2001 From: RobertObkircher Date: Sun, 16 Feb 2025 20:31:23 +0100 Subject: Cleanup (#162) * remove unused method lazyFRef * remove unused method copyF(Node, Node) * make TypeMemPtr constants final * make ConstantNode value final --- .../src/main/java/com/seaofnodes/simple/node/BoolNode.java | 1 - .../src/main/java/com/seaofnodes/simple/node/ConstantNode.java | 2 +- .../src/main/java/com/seaofnodes/simple/type/TypeMemPtr.java | 10 +++++----- 3 files changed, 6 insertions(+), 7 deletions(-) (limited to 'chapter15/src/main/java/com/seaofnodes') diff --git a/chapter15/src/main/java/com/seaofnodes/simple/node/BoolNode.java b/chapter15/src/main/java/com/seaofnodes/simple/node/BoolNode.java index 238be00e..aa9fd312 100644 --- a/chapter15/src/main/java/com/seaofnodes/simple/node/BoolNode.java +++ b/chapter15/src/main/java/com/seaofnodes/simple/node/BoolNode.java @@ -49,7 +49,6 @@ abstract public class BoolNode extends Node { TypeInteger doOp(TypeInteger t1, TypeInteger t2) { throw Utils.TODO(); } boolean doOp(double lhs, double rhs) { throw Utils.TODO(); } - Node copyF(Node lhs, Node rhs) { return null; } @Override public Node idealize() { diff --git a/chapter15/src/main/java/com/seaofnodes/simple/node/ConstantNode.java b/chapter15/src/main/java/com/seaofnodes/simple/node/ConstantNode.java index 845b9d08..d760c0df 100644 --- a/chapter15/src/main/java/com/seaofnodes/simple/node/ConstantNode.java +++ b/chapter15/src/main/java/com/seaofnodes/simple/node/ConstantNode.java @@ -18,7 +18,7 @@ import java.util.BitSet; * The Constant's value is the value stored in it. */ public class ConstantNode extends Node { - Type _con; + final Type _con; public ConstantNode( Type type ) { super(Parser.START); _con = type; diff --git a/chapter15/src/main/java/com/seaofnodes/simple/type/TypeMemPtr.java b/chapter15/src/main/java/com/seaofnodes/simple/type/TypeMemPtr.java index 8b484235..4b70546c 100644 --- a/chapter15/src/main/java/com/seaofnodes/simple/type/TypeMemPtr.java +++ b/chapter15/src/main/java/com/seaofnodes/simple/type/TypeMemPtr.java @@ -37,13 +37,13 @@ public class TypeMemPtr extends Type { // An abstract pointer, pointing to either a Struct or an Array. // Can also be null or not. - public static TypeMemPtr BOT = make(TypeStruct.BOT,true); - public static TypeMemPtr TOP = BOT.dual(); + public static final TypeMemPtr BOT = make(TypeStruct.BOT,true); + public static final TypeMemPtr TOP = BOT.dual(); // An abstract null (can be Struct or Array) or not-null C void* - public static TypeMemPtr NULLPTR = make(TypeStruct.TOP,true); - public static TypeMemPtr VOIDPTR = NULLPTR.dual(); // A bottom mix of not-null ptrs, like C's void* but not null + public static final TypeMemPtr NULLPTR = make(TypeStruct.TOP,true); + public static final TypeMemPtr VOIDPTR = NULLPTR.dual(); // A bottom mix of not-null ptrs, like C's void* but not null - public static TypeMemPtr TEST= make(TypeStruct.TEST,false); + public static final TypeMemPtr TEST= make(TypeStruct.TEST,false); public static void gather(ArrayList ts) { ts.add(NULLPTR); ts.add(BOT); ts.add(TEST); } @Override -- cgit v1.2.3