Correct addition of atoms to a list. Unlike Vector, a List does not l…

…et you use set(int, Obj) unless that index already has an item. Change-Id: I96004873d392e572efebb94f34f040fb79112238 Signed-off-by: Egon Willighagen <egonw@users.sourceforge.net>
cdk · Jun 5, 2013 · 939f03f · 939f03f
1 parent 14219a7
commit 939f03f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/main/org/openscience/cdk/ringsearch/FiguerasSSSRFinder.java b/src/main/org/openscience/cdk/ringsearch/FiguerasSSSRFinder.java
@@ -218,10 +218,10 @@ private IRing getRing(IAtom rootNode, IAtomContainer molecule)
 		List<List<IAtom>> path = new ArrayList<List<IAtom>>(OKatoms);
 		List<IAtom> intersection = new ArrayList<IAtom>();
 		List<IAtom> ring = new ArrayList<IAtom>();
-		for (int f = 0; f < OKatoms; f++)
+		for (final IAtom atom : molecule.atoms())
 		{
-			path.set(f, new ArrayList<IAtom>());
-			((List<IAtom>)molecule.getAtom(f).getProperty(PATH)).clear();
+			path.add(new ArrayList<IAtom>());
+            atom.getProperty(PATH, List.class).clear();
 		}
 		// Initialize the queue with nodes attached to rootNode
 		neighbors = molecule.getConnectedAtomsList(rootNode);