Bug 12036

Summary: Update embedded JavaScriptCore/pcre library from 6.2 to 6.4
Product: WebKit Reporter: David Kilzer (:ddkilzer) <ddkilzer>
Component: JavaScriptCoreAssignee: David Kilzer (:ddkilzer) <ddkilzer>
Status: RESOLVED FIXED    
Severity: Normal CC: ggaren, mitz
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://www.pcre.org/
Attachments:
Description Flags
Full pcre 6.2 to 6.3 diff
none
Full pcre 6.3 to 6.4 dif
none
Patch v1 ggaren: review+

David Kilzer (:ddkilzer)
Reported 2006-12-30 07:49:07 PST
Update embedded JavaScriptCore/pcre library from version 6.2 to 6.4. Note that version 6.3 had no changes that affected the embedded WebKit pcre library.
Attachments
Full pcre 6.2 to 6.3 diff (189.98 KB, patch)
2006-12-30 07:50 PST, David Kilzer (:ddkilzer) 		no flags
DetailsFormatted DiffDiff
Full pcre 6.3 to 6.4 dif (101.57 KB, patch)
2006-12-30 07:51 PST, David Kilzer (:ddkilzer) 		no flags
DetailsFormatted DiffDiff
Patch v1 (61.77 KB, patch)
2006-12-30 08:45 PST, David Kilzer (:ddkilzer) 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
David Kilzer (:ddkilzer)
Comment 1 2006-12-30 07:50:16 PST
Created attachment 12119 [details] Full pcre 6.2 to 6.3 diff Full diff between pcre 6.2 and 6.3.
David Kilzer (:ddkilzer)
Comment 2 2006-12-30 07:51:34 PST
Created attachment 12120 [details] Full pcre 6.3 to 6.4 dif Full diff between pcre 6.3 and 6.4.
David Kilzer (:ddkilzer)
Comment 3 2006-12-30 08:45:05 PST
Created attachment 12122 [details] Patch v1 Patch to bring embedded JavaScriptCore/pcre from 6.2 to 6.4. No real code changes. All layout tests and javascriptcore tests pass.
Geoffrey Garen
Comment 4 2006-12-30 11:27:18 PST
Comment on attachment 12122 [details] Patch v1 We're inching along!
David Kilzer (:ddkilzer)
Comment 5 2006-12-31 21:24:47 PST
Committed revision 18498. Mitz: I've got changes from pcre-6.5 applied locally, but I'll wait until your stuff lands to post a patch. Have you considered contributing the fix for Bug 11849 back to pcre?
Note You need to log in before you can comment on or make changes to this bug.