Resolve #523 15 Remove duplicate check

johnmay · johnmay · commit 998b83c3fc49 · 2019-01-02T22:24:12.000Z
diff --git a/legacy/src/main/java/org/openscience/cdk/iupac/parser/MoleculeBuilder.java b/legacy/src/main/java/org/openscience/cdk/iupac/parser/MoleculeBuilder.java
@@ -186,7 +186,7 @@ else if ("oyl chloride".equals(funGroupToken)) {
         else if ("chloro".equals(funGroupToken)) {
             //If functional group hasn't had a location specified:
             if (addPos < 0) {
-                addAtom("Cl", currentMolecule.isEmpty() ? null : currentMolecule.isEmpty() ? null : currentMolecule.getAtom(0), IBond.Order.SINGLE, 0);
+                addAtom("Cl", currentMolecule.isEmpty() ? null : currentMolecule.getAtom(0), IBond.Order.SINGLE, 0);
             } else {
                 addAtom("Cl", currentMolecule.getAtom(addPos), IBond.Order.SINGLE, 0);
             }

Original file line number	Diff line number	Diff line change
`@@ -186,7 +186,7 @@ else if ("oyl chloride".equals(funGroupToken)) {`
`186`	`186`	`else if ("chloro".equals(funGroupToken)) {`
`187`	`187`	`//If functional group hasn't had a location specified:`
`188`	`188`	`if (addPos < 0) {`
`189`		`- addAtom("Cl", currentMolecule.isEmpty() ? null : currentMolecule.isEmpty() ? null : currentMolecule.getAtom(0), IBond.Order.SINGLE, 0);`
	`189`	`+ addAtom("Cl", currentMolecule.isEmpty() ? null : currentMolecule.getAtom(0), IBond.Order.SINGLE, 0);`
`190`	`190`	`} else {`
`191`	`191`	`addAtom("Cl", currentMolecule.getAtom(addPos), IBond.Order.SINGLE, 0);`
`192`	`192`	`}`