Page MenuHomePhabricator

D11471.id27578.diff
No OneTemporary

D11471.id27578.diff

diff --git a/src/error/PhutilErrorTrap.php b/src/error/PhutilErrorTrap.php
--- a/src/error/PhutilErrorTrap.php
+++ b/src/error/PhutilErrorTrap.php
@@ -23,17 +23,18 @@
* throw new Exception('proc_open() failed: '.$err);
* }
*
- * NOTE: You must explicitly destroy traps because they register themselves with
- * @{class:PhutilErrorHandler}, and thus will not be destroyed when `unset()`.
+ * IMPORTANT: You must explicitly destroy traps because they register
+ * themselves with @{class:PhutilErrorHandler}, and thus will not be destroyed
+ * when `unset()`.
*
* Some notes on traps:
*
* - Traps catch all errors, including those silenced by `@`.
* - Traps do not prevent errors from reaching other standard handlers. You
* can use `@` to keep errors out of the logs while still trapping them.
- * - Traps capture all errors until they are destroyed, usually by leaving
- * scope. This means that you should not create long-lived traps, or they
- * may consume unbounded amounts of memory to hold the error log.
+ * - Traps capture all errors until they are explicitly destroyed. This means
+ * that you should not create long-lived traps, or they may consume
+ * unbounded amounts of memory to hold the error log.
*/
final class PhutilErrorTrap extends Phobject {

File Metadata

Mime Type
text/plain
Expires
Fri, Jan 24, 10:31 PM (4 h, 51 m)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7044457
Default Alt Text
D11471.id27578.diff (1 KB)

Event Timeline