Bug 16438

Summary: another round of PCRE clean-up
Product: WebKit Reporter: Darin Adler <darin>
Component: JavaScriptCoreAssignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Enhancement    
Priority: P3    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
patch (all regression tests pass)
none
patch (all regression tests pass)
none
patch with "ignore whitespace differences" on -- may be easier to review
none
patch rebased to r28788 mjs: review+

Description Darin Adler 2007-12-14 10:45:59 PST
While looking into a regression, which I'll fix soon, I did another round of PCRE code tweaks.
Comment 1 Darin Adler 2007-12-14 11:08:11 PST
Created attachment 17896 [details]
patch (all regression tests pass)
Comment 2 Darin Adler 2007-12-15 20:01:45 PST
Created attachment 17921 [details]
patch (all regression tests pass)

Earlier patch had an extra ChangeLog entry.
Comment 3 Darin Adler 2007-12-15 20:02:11 PST
Created attachment 17922 [details]
patch with "ignore whitespace differences" on -- may be easier to review
Comment 4 Darin Adler 2007-12-16 17:36:06 PST
Created attachment 17947 [details]
patch rebased to r28788
Comment 5 Maciej Stachowiak 2007-12-16 18:04:38 PST
Comment on attachment 17947 [details]
patch rebased to r28788

r=me
Comment 6 Darin Adler 2007-12-16 20:20:38 PST
Committed revision 28793.