Also the last parameter must be an IAtom

Change-Id: Ieaf5ae528623e558b2ab0eb5d240d9cfa9035bff
cdk · May 15, 2012 · 3d0b0e5 · 3d0b0e5
1 parent 2a2aecc
commit 3d0b0e5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/main/org/openscience/cdk/DefaultChemObjectBuilder.java b/src/main/org/openscience/cdk/DefaultChemObjectBuilder.java
@@ -421,7 +421,8 @@ private <T extends ICDKObject>T newElectronContainerInstance(
 	        } else {
 	            // the IBond(IAtom[]) constructor
 	            boolean allIAtom = true;
-	            for (int i=0; i<(params.length-1) && allIAtom; i++) {
+	            for (int i=0; i<params.length && allIAtom; i++) {
+	            	System.out.println(params[i]);
 	                if (!(params[i] instanceof IAtom)) allIAtom = false;
 	            }
 	            if (allIAtom) {

diff --git a/src/main/org/openscience/cdk/silent/SilentChemObjectBuilder.java b/src/main/org/openscience/cdk/silent/SilentChemObjectBuilder.java
@@ -414,7 +414,7 @@ private <T extends ICDKObject>T newElectronContainerInstance(
 	        } else {
 	            // the IBond(IAtom[]) constructor
 	            boolean allIAtom = true;
-	            for (int i=0; i<(params.length-1) && allIAtom; i++) {
+	            for (int i=0; i<params.length && allIAtom; i++) {
 	                if (!(params[i] instanceof IAtom)) allIAtom = false;
 	            }
 	            if (allIAtom) {