Move `addUse()` return statement to `keep()` (#243)

1 files changed, 3 insertions, 6 deletions

diff --git a/chapter10/src/main/java/com/seaofnodes/simple/node/Node.java b/chapter10/src/main/java/com/seaofnodes/simple/node/Node.java
index bb698860..c4889231 100644
--- a/chapter10/src/main/java/com/seaofnodes/simple/node/Node.java
+++ b/chapter10/src/main/java/com/seaofnodes/simple/node/Node.java
@@ -207,7 +207,7 @@ public abstract class Node {
     }
 
     // Breaks the edge invariants, used temporarily
-    protected <N extends Node> N addUse(Node n) { _outputs.add(n); return (N)this; }
+    protected <N extends Node> void addUse(N n) { _outputs.add(n); }
 
     // Remove node 'use' from 'def's (i.e. our) output list, by compressing the list in-place.
     // Return true if the output list is empty afterward.
@@ -254,12 +254,9 @@ public abstract class Node {
 
     // Shortcuts to stop DCE mid-parse
     // Add bogus null use to keep node alive
-    public <N extends Node> N keep() { return addUse(null); }
+    public <N extends Node> N keep() { addUse(null); return (N)this; }
     // Remove bogus null.
-    public <N extends Node> N unkeep() {
-        delUse(null);
-        return (N)this;
-    }
+    public <N extends Node> N unkeep() { delUse(null); return (N)this; }
     // Test "keep" status
     public boolean iskeep() { return Utils.find(_outputs,null) != -1; }