Bug 6119 - split() function ignores case insensitive modifier
Summary: split() function ignores case insensitive modifier
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Geoffrey Garen
URL: not applicable
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-17 05:11 PST by rahul abrol
Modified: 2007-12-04 13:29 PST (History)
0 users

See Also:


Attachments
Fix (6.43 KB, patch)
2005-12-17 16:28 PST, Geoffrey Garen
no flags Details | Formatted Diff | Diff
Fix (6.25 KB, patch)
2005-12-17 16:31 PST, Geoffrey Garen
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description rahul abrol 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.
Comment 1 Alexey Proskuryakov 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.
Comment 2 Geoffrey Garen 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.
Comment 3 Geoffrey Garen 2005-12-17 16:31:09 PST
Created attachment 5126 [details]
Fix

Last patch had extra ChangeLog junk in it.
Comment 4 Darin Adler 2005-12-17 18:57:35 PST
Comment on attachment 5126 [details]
Fix

Clear and consistent, great! r=me