Simplify Opcache enabled check

lucasnetau · Jun 29, 2023 · 2f36140 · 2f36140
1 parent 41b7f5b
commit 2f36140
Showing 1 changed file with 2 additions and 16 deletions.
diff --git a/src/Scheduler.php b/src/Scheduler.php
@@ -835,8 +835,8 @@ protected function restoreState() : void
      */
     public function run() : void
     {
-        if (!$this->isOpcacheEnabled())
-        {
+        /** Opcache is disabled if either the extension isn't loaded or opcache_get_status returns false */
+        if (!(function_exists('opcache_get_status') && opcache_get_status(false) !== false)) {
             $this->logger->warning("*** Opcache is not enabled. This will reduce performance and increase memory usage ***");
         }
 
@@ -1220,20 +1220,6 @@ protected function memoryReclaim() : void {
         }
     }
 
-    /**
-     * @return bool
-     */
-    protected function isOpcacheEnabled() : bool
-    {
-        if (function_exists('opcache_get_status'))
-        {
-            $status = opcache_get_status(false);
-
-            return (false !== $status);
-        }
-        return false;
-    }
-
     /**
      * Enable or disable the experimental management server
      * @param bool $enable