Error reporting behavior with streams

php.internals

Zeev Suraski

22 years ago
People, When streams were introduced, the format of error messages involving include/require changed significantly, and now emits two errors instead of one. Am I the only one being bugged by this? I think we can simply remove the REPORT_ERRORS option from php_fopen_wrapper_for_zend(). Possibly, if there are cases where php_stream_open_wrapper_as_file() can report something smarter than just the inability to create the stream, we can add an option such as REPORT_NONTRIVIAL_ERRORS, so that if the stream-level error has something smart to say, it won't go unnoticed. Thoughts? Zeev

George Schlossnagle

22 years ago
On Dec 25, 2003, at 12:32 PM, Zeev Suraski wrote:
> People, > > When streams were introduced, the format of error messages involving > include/require changed significantly, and now emits two errors > instead of one. Am I the only one being bugged by this? I think we > can simply remove the REPORT_ERRORS option from > php_fopen_wrapper_for_zend(). Possibly, if there are cases where > php_stream_open_wrapper_as_file() can report something smarter than > just the inability to create the stream, we can add an option such as > REPORT_NONTRIVIAL_ERRORS, so that if the stream-level error has > something smart to say, it won't go unnoticed. > > Thoughts?
I agree with you. Not only are their two errors, but the error for include/require specifically is not as clear as it once was. George