Bug 7486

Summary:

a first step in improving String/StringImpl API -- use "s" and "l" less

Product:

WebKit

Reporter:

Darin Adler <darin>

Component:

WebKit Misc.

Assignee:

Darin Adler <darin>

Status:

RESOLVED FIXED

Severity:

Minor

Priority:

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
remove most uses of s and l, especially the two places that set them directly	mjs: review+

Darin Adler

Reported 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().

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
View All Add attachment proposed patch, testcase, etc.

Darin Adler

Comment 1 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

Maciej Stachowiak

Comment 2 2006-02-26 23:32:31 PST

r=me

Note You need to log in before you can comment on or make changes to this bug.