From 43352de0070726cedd24a07113ef371d86496244 Mon Sep 17 00:00:00 2001 From: John May Date: Tue, 15 Oct 2013 14:03:17 +0100 Subject: [PATCH] Correct check for cyclic bonds. Signed-off-by: Egon Willighagen --- src/main/org/openscience/cdk/aromaticity/AtomTypeModel.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/main/org/openscience/cdk/aromaticity/AtomTypeModel.java b/src/main/org/openscience/cdk/aromaticity/AtomTypeModel.java index b7f179df5db..47cb73a7a26 100644 --- 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