On Wed, Aug 9, 2017 at 7:23 PM, Christoph M. Becker <cmbecke...@gmx.de> wrote:
> On 09.08.2017 at 17:42, Nikita Popov wrote: > > > On Wed, Aug 9, 2017 at 5:23 PM, Christoph M. Becker <cmbecke...@gmx.de> > > wrote: > > > >> I assume that the $errcontext parameter of error handlers will not be > >> deprecated in PHP 7.2, but a later version. Would it make sense to > >> already document that the parameter shouldn't be used anymore (aka. soft > >> deprecation)? > > > > I'm not sure on this one. I think it might actually be preferable to keep > > this as a documentation-only deprecation (and then directly remove it in > > PHP 8). I'm not really happy with the proposed action from the RFC (even > > though I suggested it myself), both in terms of behavior and > implementation > > complexity. There's a number of fun edge-cases there, like a __call() > > method being used as error handler -- in that case the formal number of > > parameters is 2, but more might actually be used and we have no way to > > know. I don't even know what's supposed to happen in that case under the > > RFC. > > Thanks for the explanation. In this case I'm fine with a "soft" > deprecation, but it should be mentioned in UPGRADING. > I've added a note in UPGRADING in https://github.com/php/php-src/commit/b202587e099312e433c123331bb5fece94ec9670 and moved the RFC into the Implemented section. Nikita
