Bug 7486 - a first step in improving String/StringImpl API -- use "s" and "l" less
Summary: a first step in improving String/StringImpl API -- use "s" and "l" less
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 420+
Hardware: Macintosh OS X 10.4
: P3 Minor
Assignee: Darin Adler
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-02-26 18:37 PST by Darin Adler
Modified: 2006-02-27 08:52 PST (History)
0 users

See Also:


Attachments
remove most uses of s and l, especially the two places that set them directly (22.86 KB, patch)
2006-02-26 18:37 PST, Darin Adler
mjs: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Adler 2006-02-26 18:37:02 PST
I have a patch that replaces many uses of StringImpl::s and StringImpl::l with StringImpl::unicode() and StringImpl::length().
Comment 1 Darin Adler 2006-02-26 18:37:43 PST
Created attachment 6750 [details]
remove most uses of s and l, especially the two places that set them directly
Comment 2 Maciej Stachowiak 2006-02-26 23:32:31 PST
r=me