Revision: 20781
          http://sourceforge.net/p/jmol/code/20781
Author:   hansonr
Date:     2015-09-21 01:43:27 +0000 (Mon, 21 Sep 2015)
Log Message:
-----------
Jmol.___JmolVersion="14.3.16_2015.09.21"

bug fix: print within(0, "XXX", ...) ignores the "XXX" entirely. Should throw 
an error (Ron Mignery)
bug fix: print within(0, "vdw", ...) ignores "vdw"; should accept that as 
"vanderwaal"
bug fix: print within(-1.0, "vanderwaal", ...) ignores "vanderwaal" and does a 
fractional-coordinate calculation  

Modified Paths:
--------------
    trunk/Jmol/src/org/jmol/dssx/DSSR1.java
    trunk/Jmol/src/org/jmol/scriptext/MathExt.java
    trunk/Jmol/src/org/jmol/viewer/Jmol.properties

Modified: trunk/Jmol/src/org/jmol/dssx/DSSR1.java
===================================================================
--- trunk/Jmol/src/org/jmol/dssx/DSSR1.java     2015-09-20 21:01:08 UTC (rev 
20780)
+++ trunk/Jmol/src/org/jmol/dssx/DSSR1.java     2015-09-21 01:43:27 UTC (rev 
20781)
@@ -86,8 +86,8 @@
       "..helices.pairs.nt*" +
       "..iloops.nts_long" +
       "..isocanonpairs.nt*" +
+      "..junctions.nts_long" +
       "..kissingloops.hairpins.nts_long" +
-      "..junctions.nts_long" +
       "..multiplets.nts_long" +
       "..nonstacks.nts_long" +
       "..nts.nt_id" +

Modified: trunk/Jmol/src/org/jmol/scriptext/MathExt.java
===================================================================
--- trunk/Jmol/src/org/jmol/scriptext/MathExt.java      2015-09-20 21:01:08 UTC 
(rev 20780)
+++ trunk/Jmol/src/org/jmol/scriptext/MathExt.java      2015-09-21 01:43:27 UTC 
(rev 20781)
@@ -2805,7 +2805,6 @@
     ModelSet ms = vwr.ms;
     boolean isWithinModelSet = false;
     boolean isWithinGroup = false;
-    boolean isWithinUnitcell = false;
     boolean isDistance = (isVdw || tok == T.decimal || tok == T.integer);
     RadiusData rd = null;
     switch (tok) {
@@ -2845,6 +2844,8 @@
           tok == T.search ? JC.SMILES_TYPE_SMARTS : JC.SMILES_TYPE_SMILES,
           mp.asBitSet, false));
     }
+    
+    
     if (withinSpec instanceof String) {
       if (tok == T.nada) {
         tok = T.spec_seqcode;
@@ -2869,22 +2870,26 @@
         String s = SV.sValue(args[1]);
         if (s.startsWith("$"))
           return mp.addXBs(getAtomsNearSurface(distance, s.substring(1)));
-        isWithinGroup = (s.equalsIgnoreCase("group"));
-        isVdw = (!isWithinGroup && s.equalsIgnoreCase("vanderwaals"));
-        isWithinUnitcell = (!isWithinGroup && s.equalsIgnoreCase("unitcell"));
-        if (isVdw) {
+        if (s.equalsIgnoreCase("group")) {
+          isWithinGroup = true;
+          tok = T.group;
+        } else
+        if (s.equalsIgnoreCase("vanderwaals") || s.equalsIgnoreCase("vdw")) {
           withinSpec = null;
+          isVdw = true;
           tok = T.vanderwaals;
-        } else if (isWithinUnitcell) {
+        } else if (s.equalsIgnoreCase("unitcell")) {
           tok = T.unitcell;
         } else {
-          tok = T.group;
+          return false;
         }
         break;
       }
     } else {
       return false;
     }
+    
+    
     P3 pt = null;
     P4 plane = null;
     switch (i) {
@@ -2977,9 +2982,12 @@
       return mp.addXBs(vwr.ms.getAtoms(tok, bs));
     if (isWithinGroup)
       return mp.addXBs(vwr.getGroupsWithin((int) distance, bs));
-    if (isVdw)
+    if (isVdw) {
       rd = new RadiusData(null, (distance > 10 ? distance / 100 : distance),
           (distance > 10 ? EnumType.FACTOR : EnumType.OFFSET), VDW.AUTO);
+      if (distance < 0)
+        distance = 0; // not used, but this prevents a diversion
+    }
     return mp.addXBs(vwr.ms.getAtomsWithinRadius(distance, bs,
         isWithinModelSet, rd));
   }

Modified: trunk/Jmol/src/org/jmol/viewer/Jmol.properties
===================================================================
--- trunk/Jmol/src/org/jmol/viewer/Jmol.properties      2015-09-20 21:01:08 UTC 
(rev 20780)
+++ trunk/Jmol/src/org/jmol/viewer/Jmol.properties      2015-09-21 01:43:27 UTC 
(rev 20781)
@@ -61,8 +61,15 @@
 
 TODO: Switch to dssr1 prior to full release
 
-Jmol.___JmolVersion="14.3.16_2015.09.20"
+Jmol.___JmolVersion="14.3.16_2015.09.21"
 
+bug fix: print within(0, "XXX", ...) ignores the "XXX" entirely. Should throw 
an error (Ron Mignery)
+bug fix: print within(0, "vdw", ...) ignores "vdw"; should accept that as 
"vanderwaal"
+bug fix: print within(-1.0, "vanderwaal", ...) ignores "vanderwaal" and does a 
fractional-coordinate calculation  
+
+
+JmolVersion="14.3.16_2015.09.20"
+
 FEATURE CHANGE: 3DNA DSSR has switched over to JSON; some array names are 
changed.
 
 new feature: print @1.label("%[unitid]")

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
_______________________________________________
Jmol-commits mailing list
Jmol-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jmol-commits

Reply via email to