Make sure try/catch/finally is used instead of try/finally [#421 stat…

…e:resolved]
juzna · Mar 19, 2009 · c7a5d34 · c7a5d34
1 parent 1a375da
commit c7a5d34
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,4 +1,6 @@
-* . Use `in` operator when accessing property of a nodelist to prevent Safari <=2.0.4 from crashing (kangax)
+* Make sure try/catch/finally is used instead of try/finally for clients without support for the latter one (e.g. Blackberry, IE) (Ville Koskinen, kangax)
+
+* Use `in` operator when accessing property of a nodelist to prevent Safari <=2.0.4 from crashing (kangax)
 
 * Add Element#clone as a safe wrapper of native `cloneNode`. (Andrew Dupont, kangax)
 

diff --git a/src/lang/periodical_executer.js b/src/lang/periodical_executer.js
@@ -52,9 +52,12 @@ var PeriodicalExecuter = Class.create({
       try {
         this.currentlyExecuting = true;
         this.execute();
-      } finally {
+      } catch(e) {
+        /* empty catch for clients that don't support try/finally */
+      }
+      finally {
         this.currentlyExecuting = false;
       }
     }
   }
-});
+});