Correct check for cyclic bonds.

Signed-off-by: Egon Willighagen <egonw@users.sourceforge.net>
cdk · Oct 22, 2013 · 43352de · 43352de
1 parent 3e04187
commit 43352de
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/main/org/openscience/cdk/aromaticity/AtomTypeModel.java b/src/main/org/openscience/cdk/aromaticity/AtomTypeModel.java
@@ -139,8 +139,7 @@ final class AtomTypeModel extends ElectronDonation {
                 int u = indexMap.get(a1);
                 int v = indexMap.get(a2);
 
-                // XXX: bond may be between rings, need patch under review to change for this
-                if (!ringSearch.cyclic(u) || !ringSearch.cyclic(v)) {
+                if (!ringSearch.cyclic(u, v)) {
 
                     // XXX: single exception - we could make this more general but
                     // for now this mirrors the existing behavior