Antony Dovgal wrote:
Lukas, I thought we already discussed and agreed that the only
acceptable solution is NOT to add any E_STRICT messages if the
recommended way didn't exist in first release of the major version.
This apparently requires versioning and its support in PEAR.
And personally I think this is the only solution we can accept.
That would work fine as I pointed out in the email, but would limit the
scope of E_STRICT. The point of E_STRICT was to quickly let people know
about things that are no longer recommended/have become deprecated.
Waiting until the next major version for new features would diminish that.
Adding some kind of hypercomplicated API for "filtering" E_STRICT is
definitely huge overkill.
Not sure if I explained things this badly. All I requested was that the
error message contain some "header" with the php version that notice was
added.
This requires no changes in any infrastructure and the point was exactly
to make it feasible to write such a filter function for custom error
handler. The alternative would require keeping an array of the internal
error messages with the version they were added. Which would obviously
break apart if we do a simple language or other change to the given notice.
So all I am asking is that E_STRICT notices look like the following (or
something of that sort):
Strict Standards since PHP 5.0.0: Declaration of Europe::get_countries()
should be compatible with that of Scandinavia::get_countries() in - on
line 14
Instead of:
Strict Standards: Declaration of Europe::get_countries() should be
compatible with that of Scandinavia::get_countries() in - on line 14
So I dont really see where its killing a fly with a tank either. All
that people then need to do is write a simple regexp, that filters out
the version number in the E_STRICT notice and pass it off to
version_compare(). Very trivial to implement in userland, nothing more
than changing a few strings in PHP core. End of story, problem solved.
regards,
Lukas
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
