From 00edbfd0df7651df048fc1f8819d1ab327e34f02 Mon Sep 17 00:00:00 2001
From: jochen <jochen@379699f6-c40d-0410-875b-85095c16579e>
Date: Wed, 14 Jul 1999 16:35:28 +0000
Subject: [PATCH] fillInGenSet now takes Collection instead Set as parameter

git-svn-id: https://svn.code.sf.net/p/jode/code/trunk@1069 379699f6-c40d-0410-875b-85095c16579e
---
 jode/jode/expr/CheckNullOperator.java   | 8 ++++----
 jode/jode/expr/ConstructorOperator.java | 6 +++---
 jode/jode/expr/Expression.java          | 8 ++++----
 jode/jode/expr/LocalStoreOperator.java  | 1 +
 jode/jode/expr/Operator.java            | 8 ++++----
 5 files changed, 16 insertions(+), 15 deletions(-)

diff --git a/jode/jode/expr/CheckNullOperator.java b/jode/jode/expr/CheckNullOperator.java
index bf6e9a8..1f90207 100644
--- a/jode/jode/expr/CheckNullOperator.java
+++ b/jode/jode/expr/CheckNullOperator.java
@@ -23,9 +23,9 @@ import jode.decompiler.LocalInfo;
 import jode.decompiler.TabbedPrintWriter;
 
 ///#ifdef JDK12
-///import java.util.Set;
+///import java.util.Collection;
 ///#else
-import jode.util.Set;
+import jode.util.Collection;
 ///#endif
 
 /**
@@ -74,13 +74,13 @@ public class CheckNullOperator extends Operator {
 	local.remove();
     }
 
-    public void fillInGenSet(Set in, Set gen) {
+    public void fillInGenSet(Collection in, Collection gen) {
 	if (gen != null)
 	    gen.add(local);
 	super.fillInGenSet(in, gen);
     }
 
-    public void fillDeclarables(Set used) {
+    public void fillDeclarables(Collection used) {
 	used.add(local);
 	super.fillDeclarables(used);
     }
diff --git a/jode/jode/expr/ConstructorOperator.java b/jode/jode/expr/ConstructorOperator.java
index 5ebb177..4b4918e 100644
--- a/jode/jode/expr/ConstructorOperator.java
+++ b/jode/jode/expr/ConstructorOperator.java
@@ -33,9 +33,9 @@ import jode.decompiler.Scope;
 
 import java.lang.reflect.Modifier;
 ///#ifdef JDK12
-///import java.util.Set;
+///import java.util.Collection;
 ///#else
-import jode.util.Set;
+import jode.util.Collection;
 ///#endif
 
 public class ConstructorOperator extends Operator 
@@ -165,7 +165,7 @@ public class ConstructorOperator extends Operator
      * We add the named method scoped classes to the declarables, and
      * only fillDeclarables on the parameters we will print.
      */
-    public void fillDeclarables(Set used) {
+    public void fillDeclarables(Collection used) {
 	ClassInfo clazz = getClassInfo();
 	InnerClassInfo outer = getOuterClassInfo(clazz);
 	ClassAnalyzer clazzAna = methodAnalyzer.getClassAnalyzer(clazz);
diff --git a/jode/jode/expr/Expression.java b/jode/jode/expr/Expression.java
index afdbe00..63c8289 100644
--- a/jode/jode/expr/Expression.java
+++ b/jode/jode/expr/Expression.java
@@ -23,9 +23,9 @@ import jode.GlobalOptions;
 import jode.decompiler.TabbedPrintWriter;
 
 ///#ifdef JDK12
-///import java.util.Set;
+///import java.util.Collection;
 ///#else
-import jode.util.Set;
+import jode.util.Collection;
 ///#endif
 
 public abstract class Expression {
@@ -228,10 +228,10 @@ public abstract class Expression {
         return true;
     }
 
-    public void fillInGenSet(Set in, Set gen) {
+    public void fillInGenSet(Collection in, Collection gen) {
     }
 
-    public void fillDeclarables(Set used) {
+    public void fillDeclarables(Collection used) {
     }
 
     public abstract void dumpExpression(TabbedPrintWriter writer) 
diff --git a/jode/jode/expr/LocalStoreOperator.java b/jode/jode/expr/LocalStoreOperator.java
index 44e58ce..45dfcf5 100644
--- a/jode/jode/expr/LocalStoreOperator.java
+++ b/jode/jode/expr/LocalStoreOperator.java
@@ -35,6 +35,7 @@ public class LocalStoreOperator extends LValueExpression
     }
 
     public boolean isRead() {
+	/* if it is part of a += operator, this is a read. */
         return parent != null && parent.getOperatorIndex() != ASSIGN_OP;
     }
 
diff --git a/jode/jode/expr/Operator.java b/jode/jode/expr/Operator.java
index 950ebed..aafb43b 100644
--- a/jode/jode/expr/Operator.java
+++ b/jode/jode/expr/Operator.java
@@ -23,9 +23,9 @@ import jode.GlobalOptions;
 import jode.decompiler.TabbedPrintWriter;
 
 ///#ifdef JDK12
-///import java.util.Set;
+///import java.util.Collection;
 ///#else
-import jode.util.Set;
+import jode.util.Collection;
 ///#endif
 
 public abstract class Operator extends Expression {
@@ -163,7 +163,7 @@ public abstract class Operator extends Expression {
 	return this;
     }
 
-    public void fillInGenSet(Set in, Set gen) {
+    public void fillInGenSet(Collection in, Collection gen) {
 	if (this instanceof LocalVarOperator) {
             LocalVarOperator varOp = (LocalVarOperator) this;
             if (varOp.isRead() && in != null)
@@ -175,7 +175,7 @@ public abstract class Operator extends Expression {
 	    subExpressions[i].fillInGenSet(in,gen);
     }
 
-    public void fillDeclarables(Set used) {
+    public void fillDeclarables(Collection used) {
 	if (this instanceof LocalVarOperator) {
 	    used.add(((LocalVarOperator) this).getLocalInfo());
 	}