IS_INSATURATED is unambiguous so support this in loose mode.

johnmay · johnmay · commit 2c761577174c · 2019-08-08T15:53:15.000+01:00
diff --git a/tool/smarts/src/main/java/org/openscience/cdk/smarts/Smarts.java b/tool/smarts/src/main/java/org/openscience/cdk/smarts/Smarts.java
@@ -1189,7 +1189,7 @@ else if (num > 1)
                         expr = new Expr(Expr.Type.HYBRIDISATION_NUMBER, num);
                         break;
                     case 'i':
-                        if (!isFlavor(FLAVOR_MOE | FLAVOR_CACTVS))
+                        if (!isFlavor(FLAVOR_MOE | FLAVOR_CACTVS | FLAVOR_LOOSE))
                             return false;
                         num = nextUnsignedInt();
                         if (num <= 0 || num > 8)
diff --git a/tool/smarts/src/test/java/org/openscience/cdk/smarts/SmartsExprReadTest.java b/tool/smarts/src/test/java/org/openscience/cdk/smarts/SmartsExprReadTest.java
@@ -950,6 +950,14 @@ public void ringSmallestSizeCountRange() {
                                   expr(RING_SMALLEST, 7)))));
     }
 
+    @Test
+    public void supportInsaturatedByDefault() {
+        Expr expr = getAtomExpr("[Oi]");
+        assertThat(expr, is(or(expr(RING_SMALLEST, 5),
+                               or(expr(RING_SMALLEST, 6),
+                                  expr(RING_SMALLEST, 7)))));
+    }
+
     @Test
     public void supportHGt() {
         Expr expr = getAtomExpr("[H>1]");
