Changeset 26676 in webkit for trunk/JavaScriptCore/kjs/ustring.cpp

Timestamp:

Oct 16, 2007, 1:13:24 PM (18 years ago)

Author:

darin

Message:

JavaScriptCore:

Reviewed by Maciej and Geoff (and looked over by Eric).

• ​http://bugs.webkit.org/show_bug.cgi?id=15519 eliminate use of <ctype.h> for processing ASCII

• wtf/ASCIICType.h: Added.
• wtf/DisallowCType.h: Added.

• kjs/config.h: Include DisallowCType.h.

• kjs/date_object.cpp: (KJS::skipSpacesAndComments): (KJS::findMonth): (KJS::parseDate):
• kjs/function.cpp: (KJS::decode):
• kjs/ustring.cpp: (KJS::UString::toDouble): Use ASCIICType.h functions instead of ctype.h ones.

WebCore:

Reviewed by Maciej and Geoff (and looked over by Eric).

• ​http://bugs.webkit.org/show_bug.cgi?id=15519 eliminate use of <ctype.h> for processing ASCII

• ForwardingHeaders/wtf/ASCIICType.h: Added.
• ForwardingHeaders/wtf/DisallowCType.h: Added.

• WebCorePrefix.h: Get rid of inclusion of <ctype.h>.
• config.h: Include DisallowCType.h.

• css/CSSParser.cpp: (WebCore::ParseString::lower):
• css/CSSPrimitiveValue.cpp: (WebCore::isCSSTokenizerIdentifier):
• css/CSSStyleDeclaration.cpp: (WebCore::propertyID):
• html/HTMLSelectElement.cpp: (WebCore::stripLeadingWhiteSpace):
• html/HTMLTokenizer.cpp: (WebCore::tagMatch):
• loader/FTPDirectoryParser.cpp: (WebCore::parseOneFTPLine):
• loader/TextResourceDecoder.cpp: (WebCore::TextResourceDecoder::checkForHeadCharset):
• platform/DeprecatedCString.cpp: (WebCore::DeprecatedCString::lower): (WebCore::DeprecatedCString::upper): (WebCore::DeprecatedCString::find): (WebCore::DeprecatedCString::contains):
• platform/DeprecatedString.cpp: (WebCore::equalCaseInsensitive): (WebCore::isCharacterAllowedInBase): (WebCore::DeprecatedString::find): (WebCore::DeprecatedString::contains): (WebCore::toIntegralType):
• platform/DeprecatedString.h: (WebCore::DeprecatedChar::isSpace): (WebCore::DeprecatedChar::lower): (WebCore::DeprecatedChar::upper):
• platform/KURL.cpp: (WebCore::KURL::parse):
• platform/StringImpl.cpp: (WebCore::isSpace): (WebCore::StringImpl::containsOnlyWhitespace): (WebCore::StringImpl::isLower): (WebCore::StringImpl::lower): (WebCore::StringImpl::find): (WebCore::StringImpl::reverseFind): (WebCore::equalIgnoringCase):
• platform/TextEncodingRegistry.cpp: (WebCore::TextEncodingNameHash::equal): (WebCore::TextEncodingNameHash::hash): (WebCore::atomicCanonicalTextEncodingName):
• platform/mac/KeyEventMac.mm: (WebCore::keyIdentifierForKeyEvent):
• platform/win/KeyEventWin.cpp: (WebCore::keyIdentifierForWindowsKeyCode):
• platform/win/PopupMenuWin.cpp: (WebCore::isASCIIPrintable): (WebCore::PopupWndProc):
• plugins/win/PluginViewWin.cpp: (WebCore::capitalizeRFC822HeaderFieldName):
• rendering/RenderText.cpp: (WebCore::RenderText::widthFromCache): Use ASCIICType.h functions instead of ctype.h ones.

WebKit:

Reviewed by Maciej and Geoff (and looked over by Eric).

• ​http://bugs.webkit.org/show_bug.cgi?id=15519 eliminate use of <ctype.h> for processing ASCII

• ForwardingHeaders/wtf/ASCIICType.h: Added.
• ForwardingHeaders/wtf/DisallowCType.h: Added.

• WebKitPrefix.h: Include DisallowCType.h.

• Misc/WebNSURLExtras.mm: (-[NSURL _web_URLWithLowercasedScheme]): Use toASCIILower.
• WebView/WebHTMLView.mm: (-[WebHTMLView callWebCoreCommand:]): Use toASCIIUpper. (-[WebTextCompleteController filterKeyDown:]): Add a list of specific character codes, instead of using ispunct.

File:

• trunk/JavaScriptCore/kjs/ustring.cpp (modified) (4 diffs)

trunk/JavaScriptCore/kjs/ustring.cpp

@@ -1 +1 @@
 // -*- c-basic-offset: 2 -*-
 /*
- *  This file is part of the KDE libraries
  *  Copyright (C) 1999-2000 Harri Porten ([email protected])
  *  Copyright (C) 2004, 2005, 2006, 2007 Apple Inc. All rights reserved.
@@ -48 +47 @@
 #endif
 
-using std::max;
-using std::min;
+using namespace WTF;
+using namespace std;
 
 namespace KJS {
@@ -955 +954 @@
 
   // skip leading white space
-  while (isspace(*c))
+  while (isASCIISpace(*c))
     c++;
 
@@ -1010 +1009 @@
 
   // allow trailing white space
-  while (isspace(*c))
+  while (isASCIISpace(*c))
     c++;
   // don't allow anything after - unless tolerant=true