contains does an identity check and is therefore redunant - unless so…

…me extra logic was intended here? Signed-off-by: Egon Willighagen <egonw@users.sourceforge.net>
cdk · May 4, 2015 · cb28cde · cb28cde
1 parent 1d76286
commit cb28cde
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/tool/pcore/src/main/java/org/openscience/cdk/pharmacophore/PharmacophoreMatcher.java b/tool/pcore/src/main/java/org/openscience/cdk/pharmacophore/PharmacophoreMatcher.java
@@ -422,8 +422,7 @@ private IAtomContainer getPharmacophoreMolecule(IAtomContainer input) throws CDK
                     PharmacophoreAtom patom = new PharmacophoreAtom(smarts, qatom.getSymbol(), coords);
                     // n.b. mapping[] is reusued for efficient to need to make an explict copy 
                     patom.setMatchingAtoms(Arrays.copyOf(mapping, mapping.length));
-                    if (!pharmacophoreMolecule.contains(patom))
-                        pharmacophoreMolecule.addAtom(patom);
+                    pharmacophoreMolecule.addAtom(patom);
                     count++;
                 }
             }