Bug 6119

Summary: split() function ignores case insensitive modifier
Product: WebKit Reporter: rahul abrol <solushex>
Component: JavaScriptCoreAssignee: Geoffrey Garen <ggaren>
Status: CLOSED FIXED    
Severity: Normal    
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: not applicable
Attachments:
Description Flags
Fix
none
Fix darin: review+

rahul abrol
Reported 2005-12-17 05:11:04 PST
"HelloGoodbye".split(/o/i) ... properly splits into 3 element list "HelloGOOdbye".split(/o/i) ... only splits into 2 i've used /regex/i successfully with other string functions, and split() seems to work fine in firefox.
Attachments
Fix (6.43 KB, patch)
2005-12-17 16:28 PST, Geoffrey Garen 		no flags
DetailsFormatted DiffDiff
Fix (6.25 KB, patch)
2005-12-17 16:31 PST, Geoffrey Garen 		darin: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2005-12-17 07:46:44 PST
Confirmed (by entering javascript:alert("HelloGOOdbye".split(/o/i)) in the address bar). Opera also gives the expected result.
Geoffrey Garen
Comment 2 2005-12-17 16:28:32 PST
Created attachment 5125 [details] Fix Less code = more worky. No change either way in JSC test suite results.
Geoffrey Garen
Comment 3 2005-12-17 16:31:09 PST
Created attachment 5126 [details] Fix Last patch had extra ChangeLog junk in it.
Darin Adler
Comment 4 2005-12-17 18:57:35 PST
Comment on attachment 5126 [details] Fix Clear and consistent, great! r=me
Note You need to log in before you can comment on or make changes to this bug.