Re: [PHP-DEV] Kill ereg with fire
[email protected] (Ferenc Kovacs)
| Newsgroups | php.internals |
|---|---|
| Message-ID | <CAH-PCH64H0NL_vDbDSMsDBZ9pDXDHJEzdpnU7NT79ed3u+zdSQ@mail.gmail.com> |
On Sun, Aug 31, 2014 at 7:08 AM, Tjerk Meesters <[email protected]> wrote: > Hi internals (again), > > Recently I’ve done a small assessment on how feasible it is to remove > ext/ereg from the project for the next major version. This is the result > (so far): > > https://github.com/datibbaw/php-src/compare/kill-ereg > > I’ve replaced two instances of ereg with their pcre equivalents, OPcache > and PGSQL. > > Btw, simply using ‘pcreposix.h’ didn’t work out for OPcache, I would get > this error: > > Error Blacklist compilation: invalid argument to regex routine > > Perhaps I’ve missed something obvious, but porting it wasn’t hard either. > > Thoughts? Afair we have a patch floating around, but never made it into the repo, but everybody seemed to agree to remove the core dependency of ereg and make it optional to enable. -- Ferenc Kovács @Tyr43l - http://tyrael.hu