[Webkit-unassigned] [Bug 38828] REGRESSION(r57955): RegExp literals should not actually be cached, so r57955 should be rolled out.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=38828


Oliver Hunt <oliver at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |InRadar




--- Comment #3 from Oliver Hunt <oliver at apple.com>  2010-05-09 18:32:34 PST ---
<rdar://problem/7961634>


In order to match firefox I made JSC cache regexp literals, however it turns out that this behaviour is part of ES3, and ES5 mandates the we don't cache.  eg. Old JSC was correct per-ES5.  This needs to be reverted due to the following reasons:

1. We did this partially as a compat fix, but firefox is dropping regexp caching in 3.7 and moving to our (old) non-caching behaviour. eg. we will be the only browser that does this.
2. Even in firefox, where regexp literals have always been cached, the caching of regexp literals is their 2nd most frequently duped bug report.
3. Due to the way RegExp.lastIndex works this change in behaviour can result in significantly different regexp behaviour.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.