Antony Dovgal wrote: > On 10.01.2006 19:13, Thiago Silva wrote: > >> Hi, >> can anyone review this? >> >> It's a one-line patch for the silent operator (@), wich currently >> supress fatal error messages (undesirable for "@include" pehaps, wich >> makes difficult for debugging scripts). > > > With this patch you basically change whole @ functionality, thus > breaking both BC and expected behavior in the same time. > And even in the case with include() - this is _expected_. If you don't > want to see all error messages silenced, just use set_error_reporting() > to set desired error level.
Suppression of fatals is an irritating misfeature. There's nothing worse than trying to work out where in tens of thousands of lines of code some idiot developer put an @ before a function invocation where the function is only available on some systems. In MediaWiki we use wfSuppressWarnings() and wfRestoreWarnings() instead of @, but of course that's slower to type and to execute, so the temptation is always there to use the evil @. If suppression of fatals could be switched off, even as an INI setting off by default, it would be a great help to developers. -- Tim Starling -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
