Don't record stack traces in case of AMP_DEBUG not being set

Fixes #217.
amphp · Apr 21, 2018 · 8399301 · 8399301
1 parent c632e40
commit 8399301
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/Internal/Placeholder.php b/lib/Internal/Placeholder.php
@@ -94,7 +94,7 @@ private function resolve($value = null) {
         }
 
         \assert((function () {
-            $env = \getenv("AMP_DEBUG");
+            $env = \getenv("AMP_DEBUG") ?: "0";
             if (($env !== "0" && $env !== "false") || (\defined("AMP_DEBUG") && \AMP_DEBUG)) {
                 $trace = \debug_backtrace(\DEBUG_BACKTRACE_IGNORE_ARGS);
                 \array_shift($trace); // remove current closure

diff --git a/lib/Internal/Producer.php b/lib/Internal/Producer.php
@@ -154,7 +154,7 @@ private function complete() {
         }
 
         \assert((function () {
-            $env = \getenv("AMP_DEBUG");
+            $env = \getenv("AMP_DEBUG") ?: "0";
             if (($env !== "0" && $env !== "false") || (\defined("AMP_DEBUG") && \AMP_DEBUG)) {
                 $trace = \debug_backtrace(\DEBUG_BACKTRACE_IGNORE_ARGS);
                 \array_shift($trace); // remove current closure