Remove PHP4 constructors in wp-includes/pomo/streams.php?

[andg]

Hi,

in the wp-includes/pomo/streams.php file there's a bunch of PHP4 constructors that could generate a warning when on PHP7, since "methods with the same name as their class will not be constructors in a future version of PHP".

Should these be removed, leaving only their PHP5 counterparts?

Also related, the constructor for the POMO_CachedIntFileReader class isn't really needed, since all it does is calling its parent class' constructor.

Thoughts?

[swissspidy]

It's an external library, see ​https://github.com/LeoColomb/pomo.

Though in the past lots of changes have been made to it, some have been merged upstream.

In the current version, the library uses namespaces and PHP5-style constructors only.

I think it's safe to remove the old constructors in this case, as it doesn't make maintenance any harder.

[dd32]

I believe all of these constructors were dealt with in [32990].

Instead of removing them (which we can't really do for backwards compatibility reasons) we chose to instead ensure that a __construct() appears before the class_name() constructor. This avoids the PHP7 deprecated warnings, and even if PHP-future stops calling the class_name() methods they'll continue to work (as they now wrap the __construct() constructor).

Do there still exist any classes which don't follow the new-then-old style syntax?

[ocean90]

See [32990].