Cleaner calling, we should not add an isotope if the count is 0.

johnmay · johnmay · commit 87a2c0807e74 · 2019-04-13T13:21:27.000+01:00
diff --git a/base/data/src/main/java/org/openscience/cdk/formula/MolecularFormula.java b/base/data/src/main/java/org/openscience/cdk/formula/MolecularFormula.java
@@ -113,6 +113,8 @@ public IMolecularFormula addIsotope(IIsotope isotope) {
      */
     @Override
     public IMolecularFormula addIsotope(IIsotope isotope, int count) {
+        if (count == 0)
+            return this;
         boolean flag = false;
         for (IIsotope thisIsotope : isotopes()) {
             if (isTheSame(thisIsotope, isotope)) {
diff --git a/base/silent/src/main/java/org/openscience/cdk/silent/MolecularFormula.java b/base/silent/src/main/java/org/openscience/cdk/silent/MolecularFormula.java
@@ -110,6 +110,8 @@ public IMolecularFormula addIsotope(IIsotope isotope) {
      */
     @Override
     public IMolecularFormula addIsotope(IIsotope isotope, int count) {
+        if (count == 0)
+            return this;
         boolean flag = false;
         for (IIsotope thisIsotope : isotopes()) {
             if (isTheSame(thisIsotope, isotope)) {
