Summary: | Update String::containsOnlyLatin1() to avoid converting to 16 bits | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Benjamin Poulain <benjamin> | ||||
Component: | Web Template Framework | Assignee: | Benjamin Poulain <benjamin> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | kling, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Benjamin Poulain
2011-12-04 21:29:35 PST
Created attachment 117839 [details]
Patch
Comment on attachment 117839 [details]
Patch
Delightful!
Comment on attachment 117839 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=117839&action=review > Source/JavaScriptCore/wtf/text/WTFString.h:-479 > -inline bool charactersAreAllLatin1(const UChar* characters, size_t length) Why remove this? Why not just call it? > > Source/JavaScriptCore/wtf/text/WTFString.h:-479
> > -inline bool charactersAreAllLatin1(const UChar* characters, size_t length)
>
> Why remove this? Why not just call it?
I thought removing it would reduce the complexity here since it is never used.
(In reply to comment #4) > > > Source/JavaScriptCore/wtf/text/WTFString.h:-479 > > > -inline bool charactersAreAllLatin1(const UChar* characters, size_t length) > > > > Why remove this? Why not just call it? > > I thought removing it would reduce the complexity here since it is never used. Do you want me to add it back? I don't mind. Comment on attachment 117839 [details] Patch Clearing flags on attachment: 117839 Committed r102028: <http://trac.webkit.org/changeset/102028> All reviewed patches have been landed. Closing bug. |