Summary: | REGRESSION (r27299-r27305): RegExp leaks on Tiger | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | mitz | ||||
Component: | JavaScriptCore | Assignee: | Alexey Proskuryakov <ap> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ap | ||||
Priority: | P1 | Keywords: | Regression | ||||
Version: | 523.x (Safari 3) | ||||||
Hardware: | Mac | ||||||
OS: | OS X 10.4 | ||||||
URL: | http://build.webkit.org/results/trunk-mac-intel-debug/1950/DumpRenderTree2-leaks.txt | ||||||
Attachments: |
|
Description
mitz
2007-11-02 11:03:26 PDT
Created attachment 17021 [details]
proposed fix
Comment on attachment 17021 [details]
proposed fix
Looks good.
Why did my code change cause the regression? Could we use OwnPtr here?
r=me
(In reply to comment #2) > Why did my code change cause the regression? It was my change that added RegExp.compile - I wrongly assumed setRegExp deleted the old value. > Could we use OwnPtr here? Oops, forgot about it somehow. Changed to use OwnPtr. > r=me Committed revision 27393. |