From: gmungoc <g.m.carstairs@dundee.ac.uk>
Date: Wed, 20 May 2020 07:51:38 +0000 (+0100)
Subject: JAL-3626 use Viewer.scriptWait() to send Jmol commands (Bob Hanson)
X-Git-Tag: Release_2_11_2_0~37^2~11
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=4a67afac64ed574a5b79c7d33c97e58e26f60f30;p=jalview.git

JAL-3626 use Viewer.scriptWait() to send Jmol commands (Bob Hanson)
---

diff --git a/src/jalview/ext/jmol/JalviewJmolBinding.java b/src/jalview/ext/jmol/JalviewJmolBinding.java
index 54eb599..038ca48 100644
--- a/src/jalview/ext/jmol/JalviewJmolBinding.java
+++ b/src/jalview/ext/jmol/JalviewJmolBinding.java
@@ -115,7 +115,7 @@ public abstract class JalviewJmolBinding extends AAStructureBindingModel
   private String jmolScript(String script)
   {
     Cache.log.debug(">>Jmol>> " + script);
-    String s = jmolViewer.evalStringQuiet(script);
+    String s = jmolViewer.scriptWait(script);
     Cache.log.debug("<<Jmol<< " + s);
 
     return s;
@@ -210,7 +210,10 @@ public abstract class JalviewJmolBinding extends AAStructureBindingModel
           mset.add(filePath);
         }
       }
-      modelFileNames = mset.toArray(new String[mset.size()]);
+      if (!mset.isEmpty())
+      {
+        modelFileNames = mset.toArray(new String[mset.size()]);
+      }
     }
 
     return modelFileNames;
diff --git a/src/jalview/gui/AppJmol.java b/src/jalview/gui/AppJmol.java
index 4058cda..ffc8053 100644
--- a/src/jalview/gui/AppJmol.java
+++ b/src/jalview/gui/AppJmol.java
@@ -344,8 +344,10 @@ public class AppJmol extends StructureViewerBase
       try
       {
         Cache.log.debug("Waiting around for jmb notify.");
-        Thread.sleep(waitFor);
         waitTotal += waitFor;
+
+        // Thread.sleep() throws an exception in JS
+        Thread.sleep(waitFor);
       } catch (Exception e)
       {
       }