Req #53238 [Opn->Asn]: Make third parameter of preg_match_all optional
| From: | aharvey@php.net | Date: | Fri, 05 Nov 2010 04:23:45 +0000 |
| Subject: | Req #53238 [Opn->Asn]: Make third parameter of preg_match_all optional | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-155781@lists.php.net to get a copy of this message | ||
Edit report at http://bugs.php.net/bug.php?id=53238&edit=1
ID: 53238
Updated by: aharvey@php.net
Reported by: giovanni at giacobbi dot net
Summary: Make third parameter of preg_match_all optional
-Status: Open
+Status: Assigned
Type: Feature/Change Request
Package: PCRE related
Operating System: any
PHP Version: 5.3.3
-Assigned To:
+Assigned To: aharvey
Block user comment: N
Previous Comments:
------------------------------------------------------------------------
[2010-11-03 19:32:23] giovanni at giacobbi dot net
Description:
------------
I had to count the time a regular expression is in a string, and the
correct way to do this is with the return value of preg_match_all()
because preg_match() will always return either 0 or 1.
The problem is that preg_match_all() enforces the third parameter. My
request is to align the prototype of preg_match_all() to the one of
preg_match(), i.e. making the third parameter optional.
Test script:
---------------
$cnt = preg_match_all('/<br\s*\/?>/i', "this<br/>my<br>text");
var_dump($cnt);
Expected result:
----------------
int(2)
Actual result:
--------------
Warning: preg_match_all() expects at least 3 parameters, 2 given in - on
line 2
bool(false)
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/bug.php?id=53238&edit=1