Unindent some code in Watchdog::shouldTerminate().
authormark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 10 May 2017 00:03:10 +0000 (00:03 +0000)
committermark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 10 May 2017 00:03:10 +0000 (00:03 +0000)
https://bugs.webkit.org/show_bug.cgi?id=171896

Rubber stamped by Keith Miller.

I should have done this before I landed r213107, but I forgot.  Unindenting it now.

* runtime/Watchdog.cpp:
(JSC::Watchdog::shouldTerminate):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@216558 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/runtime/Watchdog.cpp

index a4781cf..360f9cf 100644 (file)
@@ -1,3 +1,15 @@
+2017-05-09  Mark Lam  <mark.lam@apple.com>
+
+        Unindent some code in Watchdog::shouldTerminate().
+        https://bugs.webkit.org/show_bug.cgi?id=171896
+
+        Rubber stamped by Keith Miller.
+
+        I should have done this before I landed r213107, but I forgot.  Unindenting it now.
+
+        * runtime/Watchdog.cpp:
+        (JSC::Watchdog::shouldTerminate):
+
 2017-05-09  Michael Saboff  <msaboff@apple.com>
 
         Cap the number of FTL compilation threads on iOS to 2
index 5734263..267121c 100644 (file)
@@ -69,21 +69,18 @@ void Watchdog::setTimeLimit(std::chrono::microseconds limit,
 bool Watchdog::shouldTerminate(ExecState* exec)
 {
     ASSERT(m_vm->currentThreadIsHoldingAPILock());
-    // FIXME: Will unindent the following before landing. Leaving indented for now to minimize the code diff.
-    {
-        if (currentWallClockTime() < m_wallClockDeadline)
-            return false; // Just a stale timer firing. Nothing to do.
-
-        // Set m_wallClockDeadline to noTimeLimit here so that we can reject all future
-        // spurious wakes.
-        m_wallClockDeadline = noTimeLimit;
-
-        auto cpuTime = currentCPUTime();
-        if (cpuTime < m_cpuDeadline) {
-            auto remainingCPUTime = m_cpuDeadline - cpuTime;
-            startTimer(remainingCPUTime);
-            return false;
-        }
+    if (currentWallClockTime() < m_wallClockDeadline)
+        return false; // Just a stale timer firing. Nothing to do.
+
+    // Set m_wallClockDeadline to noTimeLimit here so that we can reject all future
+    // spurious wakes.
+    m_wallClockDeadline = noTimeLimit;
+
+    auto cpuTime = currentCPUTime();
+    if (cpuTime < m_cpuDeadline) {
+        auto remainingCPUTime = m_cpuDeadline - cpuTime;
+        startTimer(remainingCPUTime);
+        return false;
     }
 
     // Note: we should not be holding the lock while calling the callbacks. The callbacks may
@@ -96,23 +93,21 @@ bool Watchdog::shouldTerminate(ExecState* exec)
     if (needsTermination)
         return true;
 
-    // FIXME: Will unindent the following before landing. Leaving indented for now to minimize the code diff.
-    {
-        // If we get here, then the callback above did not want to terminate execution. As a
-        // result, the callback may have done one of the following:
-        //   1. cleared the time limit (i.e. watchdog is disabled),
-        //   2. set a new time limit via Watchdog::setTimeLimit(), or
-        //   3. did nothing (i.e. allow another cycle of the current time limit).
-        //
-        // In the case of 1, we don't have to do anything.
-        // In the case of 2, Watchdog::setTimeLimit() would already have started the timer.
-        // In the case of 3, we need to re-start the timer here.
-
-        ASSERT(m_hasEnteredVM);
-        bool callbackAlreadyStartedTimer = (m_cpuDeadline != noTimeLimit);
-        if (hasTimeLimit() && !callbackAlreadyStartedTimer)
-            startTimer(m_timeLimit);
-    }
+    // If we get here, then the callback above did not want to terminate execution. As a
+    // result, the callback may have done one of the following:
+    //   1. cleared the time limit (i.e. watchdog is disabled),
+    //   2. set a new time limit via Watchdog::setTimeLimit(), or
+    //   3. did nothing (i.e. allow another cycle of the current time limit).
+    //
+    // In the case of 1, we don't have to do anything.
+    // In the case of 2, Watchdog::setTimeLimit() would already have started the timer.
+    // In the case of 3, we need to re-start the timer here.
+
+    ASSERT(m_hasEnteredVM);
+    bool callbackAlreadyStartedTimer = (m_cpuDeadline != noTimeLimit);
+    if (hasTimeLimit() && !callbackAlreadyStartedTimer)
+        startTimer(m_timeLimit);
+
     return false;
 }