Update String::containsOnlyLatin1() to avoid converting to 16 bits
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.