Fixed NullPointerException by creating an empty list of generators, a…

…nd return a list too, rather than a null
cdk · May 19, 2012 · d13b1a2 · d13b1a2
1 parent 32ca540
commit d13b1a2
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/main/org/openscience/cdk/renderer/MoleculeSetRenderer.java b/src/main/org/openscience/cdk/renderer/MoleculeSetRenderer.java
@@ -23,6 +23,8 @@
 
 import java.awt.Rectangle;
 import java.awt.geom.Rectangle2D;
+import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 
 import javax.vecmath.Point2d;
@@ -121,6 +123,7 @@ public MoleculeSetRenderer(List<IGenerator<IAtomContainer>> generators, IFontMan
             rendererModel.registerParameters(generator);
         }
         atomContainerRenderer = new AtomContainerRenderer(generators, fontManager);
+        this.generators = Collections.emptyList();
         this.setup();
     }
 
@@ -284,8 +287,7 @@ public double calculateScaleForBondLength(double modelBondLength) {
 	}
 
     public List<IGenerator<IMoleculeSet>> getGenerators() {
-        // TODO Auto-generated method stub
+        return new ArrayList<IGenerator<IMoleculeSet>>(generators);
-        return null;
     }
 
 }