Re: Re: cvs: php-src /ext/pcre TODO
| From: | Derick Rethans | Date: | Sun, 09 Apr 2006 13:31:49 +0000 |
| Subject: | Re: Re: cvs: php-src /ext/pcre TODO | ||
| References: | 1 2 3 4 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-22688@lists.php.net to get a copy of this message | ||
On Sun, 9 Apr 2006, Nuno Lopes wrote:
> OK, I've made a patch for you:
> http://mega.ist.utl.pt/~ncpl/php_pcre_match_limit.txt
> adds: pcre.match_limit and pcre.match_limit_recursion.
>
> BTW, I think we should print a warning when pcre_exec() fails. Currently we
> just fail silently.
We should only do that is the regexp is the *only* parameter that can
make pcre_exec fail. If the subject string can influence this we should
not show a warning as you'd have to silence that by hand then.
Derick