Re: RFC [Discussion]: Improve unserialize() error handling
| From: | Tim Düsterhus | Date: | Wed, 28 Sep 2022 18:37:55 +0000 |
| Subject: | Re: RFC [Discussion]: Improve unserialize() error handling | ||
| References: | 1 2 3 4 5 6 7 8 9 10 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-118700@lists.php.net to get a copy of this message | ||
Hi
On 9/28/22 17:42, Tim Düsterhus wrote:
Would the increase of E_NOTICE to E_WARNING for the two cases that currently emit E_NOTICE be something that even requires a vote or is this something that can simply be decided by merging a PR? [1] In that case the second and third vote could be simplified to "Convert E_* to Exception" with the regular 2/3 majority required. [1] I would hope that unifying E_NOTICE/E_WARNING to E_WARNING everywhere is uncontroversial.I've now created a standalone PR for this: https://github.com/php/php-src/pull/9629 While creating that one, I also came across this: https://github.com/php/php-src/pull/9630 Best regards Tim Düsterhus