Review Request: Have RegExp.prototype.test set lastIndex like Regexp.prototype.exec

[Maksim Orlovich]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3327/#review4588
-----------------------------------------------------------

Ship it!


Looks good, thanks! Do you have an SVN account, or would you like me to commit it? Also, do you by any chance have a testcase?

- Maksim


On 2010-03-20 22:35:31, pcc wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3327/
> -----------------------------------------------------------
> 
> (Updated 2010-03-20 22:35:31)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> Have RegExp.prototype.test set lastIndex like Regexp.prototype.exec
> 
> The EMCAScript specification states that Regexp.prototype.test
> acts exactly like Regexp.prototype.exec except for returning a
> boolean result.  The kjs implementation currently does not conform
> to this because it does not set lastIndex if the global flag is set.
> This patch causes it to do so.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdelibs/kjs/regexp_object.cpp 1105683 
> 
> Diff: http://reviewboard.kde.org/r/3327/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> pcc
> 
>