OK, so basically it is impossible to fix the current regex to match what we
want (I had a quick chat with the PCRE author and he thinks the same).
I propose the following patch:
http://web.ist.utl.pt/nuno.lopes/php_run_tests_%s.txt

The main changes are:
'%s' => '[^\r\n]+'

Nice patch! The only thing i wonder about is why you disallow /r. Shouldn't we allow it to prevent problems? Or is it enough to take care of it only at
line endings, and if so where are we doing that?

It's mainly because of the old Macs that used \r as line-ending. Anyway I think that probably you'll never want to match a single \r. If you want, well you can always use the EXPECTREGEX section, or the %c (match a char) or the new %a (match anything).

Nuno
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to