JAL-4083 only set globalUseScriptWait if needed
authorBen Soares <b.soares@dundee.ac.uk>
Thu, 27 Oct 2022 12:23:16 +0000 (13:23 +0100)
committerBen Soares <b.soares@dundee.ac.uk>
Thu, 27 Oct 2022 12:23:16 +0000 (13:23 +0100)
src/jalview/ext/jmol/JalviewJmolBinding.java

index a40da35..a529fae 100644 (file)
@@ -269,11 +269,15 @@ public abstract class JalviewJmolBinding extends AAStructureBindingModel
   {
     if (atoms != null)
     {
+      // we set the globalUseScriptWait (and reset to previous value) only if
+      // we're going to use it here
       boolean useScriptWait = atoms.size() > 1;
-      boolean prevUseScriptWait = setGlobalUseScriptWait(useScriptWait);
+      boolean prevUseScriptWait = false;
+      if (useScriptWait)
+        prevUseScriptWait = setGlobalUseScriptWait(useScriptWait);
       if (resetLastRes.length() > 0)
       {
-        jmolScript(resetLastRes.toString());
+        jmolScript(resetLastRes.toString(), useScriptWait);
         resetLastRes.setLength(0);
       }
       for (AtomSpec atom : atoms)
@@ -281,7 +285,8 @@ public abstract class JalviewJmolBinding extends AAStructureBindingModel
         highlightAtom(atom.getAtomIndex(), atom.getPdbResNum(),
                 atom.getChain(), atom.getPdbFile(), useScriptWait);
       }
-      setGlobalUseScriptWait(prevUseScriptWait);
+      if (useScriptWait)
+        setGlobalUseScriptWait(prevUseScriptWait);
     }
   }