2006-03-27 Eric Seidel <eseidel@apple.com>
authoreseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Mar 2006 18:34:44 +0000 (18:34 +0000)
committereseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Mar 2006 18:34:44 +0000 (18:34 +0000)
        Reviewed by darin.

        Replace more DeprecatedString with String.
        Add String::sprintf() and String::number()
        http://bugzilla.opendarwin.org/show_bug.cgi?id=8009

        * bindings/objc/DOMHTML.mm:
        (-[DOMHTMLTextAreaElement setCols:]):
        (-[DOMHTMLTextAreaElement setRows:]):
        (-[DOMHTMLOListElement setStart:]):
        (-[DOMHTMLPreElement setWidth:]):
        (-[DOMHTMLImageElement setHeight:]):
        (-[DOMHTMLImageElement setHspace:]):
        (-[DOMHTMLImageElement setVspace:]):
        (-[DOMHTMLImageElement setWidth:]):
        (-[DOMHTMLObjectElement setHspace:]):
        (-[DOMHTMLObjectElement setTabIndex:]):
        (-[DOMHTMLObjectElement setVspace:]):
        (-[DOMHTMLAppletElement setHspace:]):
        (-[DOMHTMLAppletElement setVspace:]):
        (-[DOMHTMLAreaElement setTabIndex:]):
        (-[DOMHTMLTableColElement setSpan:]):
        (-[DOMHTMLTableCellElement setColSpan:]):
        (-[DOMHTMLTableCellElement setRowSpan:]):
        (-[DOMHTMLEmbedElement setHeight:]):
        (-[DOMHTMLEmbedElement setWidth:]):
        * bindings/objc/DOMInternal.mm:
        (String::String):
        * css/CSSComputedStyleDeclaration.cpp:
        (WebCore::numberAsString):
        (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
        * css/css_valueimpl.cpp:
        (WebCore::CSSMutableStyleDeclaration::getPropertyValue):
        (WebCore::CSSPrimitiveValue::cssText):
        * dom/Position.cpp:
        (WebCore::Position::formatForDebugger):
        * dom/Range.cpp:
        (WebCore::Range::formatForDebugger):
        * dom/StyledElement.cpp:
        (WebCore::StyledElement::addCSSColor):
        * dom/xml_tokenizer.cpp:
        (WebCore::XMLTokenizer::error):
        * editing/ApplyStyleCommand.cpp:
        (WebCore::ApplyStyleCommand::applyRelativeFontStyleChange):
        * html/HTMLGenericFormElement.cpp:
        (WebCore::HTMLGenericFormElement::findMatchingState):
        (WebCore::HTMLGenericFormElement::setTabIndex):
        * html/HTMLInputElement.cpp:
        (WebCore::HTMLInputElement::setMaxLength):
        (WebCore::HTMLInputElement::setSize):
        * html/HTMLSelectElement.cpp:
        (WebCore::HTMLSelectElement::setSize):
        * html/HTMLTextAreaElement.cpp:
        (WebCore::HTMLTextAreaElement::setCols):
        (WebCore::HTMLTextAreaElement::setRows):
        * html/html_baseimpl.cpp:
        (WebCore::HTMLBodyElement::insertedIntoDocument):
        * html/html_blockimpl.cpp:
        (WebCore::HTMLHRElement::parseMappedAttribute):
        (WebCore::HTMLPreElement::setWidth):
        * html/html_imageimpl.cpp:
        (WebCore::HTMLImageElement::width):
        (WebCore::HTMLImageElement::height):
        (WebCore::HTMLImageElement::setBorder):
        (WebCore::HTMLImageElement::setHeight):
        (WebCore::HTMLImageElement::setHspace):
        (WebCore::HTMLImageElement::setVspace):
        (WebCore::HTMLImageElement::setWidth):
        (WebCore::HTMLAreaElement::setTabIndex):
        * html/html_inlineimpl.cpp:
        (WebCore::HTMLAnchorElement::setTabIndex):
        * html/html_listimpl.cpp:
        (WebCore::HTMLOListElement::setStart):
        (WebCore::HTMLLIElement::setValue):
        * html/html_objectimpl.cpp:
        (WebCore::HTMLObjectElement::setTabIndex):
        * html/html_tableimpl.cpp:
        (WebCore::HTMLTableElement::parseMappedAttribute):
        (WebCore::HTMLTableCellElement::setColSpan):
        (WebCore::HTMLTableCellElement::setRowSpan):
        (WebCore::HTMLTableColElement::setSpan):
        * khtml/ecma/kjs_window.cpp:
        (KJS::Window::isSafeScript):
        (KJS::Location::getValueProperty):
        * ksvg2/css/SVGCSSParser.cpp:
        (WebCore::CSSParser::parseSVGPaint):
        (WebCore::CSSParser::parseSVGColor):
        * ksvg2/svg/SVGAngle.cpp:
        (SVGAngle::setValueAsString):
        (SVGAngle::valueAsString):
        * ksvg2/svg/SVGAngle.h:
        * ksvg2/svg/SVGLength.cpp:
        (SVGLength::setValueAsString):
        (SVGLength::valueAsString):
        * ksvg2/svg/SVGLength.h:
        * ksvg2/svg/SVGPathSeg.h:
        (WebCore::SVGPathSeg::pathSegTypeAsLetter):
        (WebCore::SVGPathSeg::toString):
        * ksvg2/svg/SVGPathSegArc.h:
        (WebCore::SVGPathSegArcAbs::pathSegTypeAsLetter):
        (WebCore::SVGPathSegArcAbs::toString):
        (WebCore::SVGPathSegArcRel::pathSegTypeAsLetter):
        (WebCore::SVGPathSegArcRel::toString):
        * ksvg2/svg/SVGPathSegClosePath.h:
        (WebCore::SVGPathSegClosePath::pathSegTypeAsLetter):
        (WebCore::SVGPathSegClosePath::toString):
        * ksvg2/svg/SVGPathSegCurvetoCubic.h:
        (WebCore::SVGPathSegCurvetoCubicAbs::pathSegTypeAsLetter):
        (WebCore::SVGPathSegCurvetoCubicAbs::toString):
        (WebCore::SVGPathSegCurvetoCubicRel::pathSegTypeAsLetter):
        (WebCore::SVGPathSegCurvetoCubicRel::toString):
        * ksvg2/svg/SVGPathSegCurvetoCubicSmooth.h:
        (WebCore::SVGPathSegCurvetoCubicSmoothAbs::pathSegTypeAsLetter):
        (WebCore::SVGPathSegCurvetoCubicSmoothAbs::toString):
        (WebCore::SVGPathSegCurvetoCubicSmoothRel::pathSegTypeAsLetter):
        (WebCore::SVGPathSegCurvetoCubicSmoothRel::toString):
        * ksvg2/svg/SVGPathSegCurvetoQuadratic.h:
        (WebCore::SVGPathSegCurvetoQuadraticAbs::pathSegTypeAsLetter):
        (WebCore::SVGPathSegCurvetoQuadraticAbs::toString):
        (WebCore::SVGPathSegCurvetoQuadraticRel::pathSegTypeAsLetter):
        (WebCore::SVGPathSegCurvetoQuadraticRel::toString):
        * ksvg2/svg/SVGPathSegCurvetoQuadraticSmooth.h:
        (WebCore::SVGPathSegCurvetoQuadraticSmoothAbs::pathSegTypeAsLetter):
        (WebCore::SVGPathSegCurvetoQuadraticSmoothAbs::toString):
        (WebCore::SVGPathSegCurvetoQuadraticSmoothRel::pathSegTypeAsLetter):
        (WebCore::SVGPathSegCurvetoQuadraticSmoothRel::toString):
        * ksvg2/svg/SVGPathSegLineto.h:
        (WebCore::SVGPathSegLinetoAbs::pathSegTypeAsLetter):
        (WebCore::SVGPathSegLinetoAbs::toString):
        (WebCore::SVGPathSegLinetoRel::pathSegTypeAsLetter):
        (WebCore::SVGPathSegLinetoRel::toString):
        * ksvg2/svg/SVGPathSegLinetoHorizontal.h:
        (WebCore::SVGPathSegLinetoHorizontalAbs::pathSegTypeAsLetter):
        (WebCore::SVGPathSegLinetoHorizontalAbs::toString):
        (WebCore::SVGPathSegLinetoHorizontalRel::pathSegTypeAsLetter):
        (WebCore::SVGPathSegLinetoHorizontalRel::toString):
        * ksvg2/svg/SVGPathSegLinetoVertical.h:
        (WebCore::SVGPathSegLinetoVerticalAbs::pathSegTypeAsLetter):
        (WebCore::SVGPathSegLinetoVerticalAbs::toString):
        (WebCore::SVGPathSegLinetoVerticalRel::pathSegTypeAsLetter):
        (WebCore::SVGPathSegLinetoVerticalRel::toString):
        * ksvg2/svg/SVGPathSegMoveto.h:
        (WebCore::SVGPathSegMovetoAbs::pathSegTypeAsLetter):
        (WebCore::SVGPathSegMovetoAbs::toString):
        (WebCore::SVGPathSegMovetoRel::pathSegTypeAsLetter):
        (WebCore::SVGPathSegMovetoRel::toString):
        * ksvg2/svg/SVGPolyElement.cpp:
        (SVGPolyElement::notifyAttributeChange):
        * ksvg2/svg/SVGStopElement.cpp:
        (SVGStopElement::parseMappedAttribute):
        * ksvg2/svg/SVGUseElement.cpp:
        (SVGUseElement::closeRenderer):
        * platform/AtomicString.h:
        (WebCore::AtomicString::toInt):
        * platform/Color.cpp:
        (WebCore::Color::name):
        * platform/Color.h:
        * platform/PlatformString.h:
        * platform/String.cpp:
        (WebCore::String::sprintf):
        (WebCore::String::number):
        * platform/StringImpl.cpp:
        (WebCore::StringImpl::StringImpl):
        (WebCore::StringImpl::initWithChar):
        (WebCore::StringImpl::initWithQChar):
        (WebCore::StringImpl::containsOnlyWhitespace):
        * platform/mac/KeyEventMac.mm:
        (WebCore::keyIdentifierForKeyEvent):
        * rendering/render_form.cpp:
        (WebCore::RenderSlider::updateFromElement):
        (WebCore::RenderSlider::valueChanged):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13532 268f45cc-cd09-0410-ab3c-d52691b4dbfc

50 files changed:
WebCore/ChangeLog
WebCore/bindings/objc/DOMHTML.mm
WebCore/bindings/objc/DOMInternal.mm
WebCore/css/CSSComputedStyleDeclaration.cpp
WebCore/css/css_valueimpl.cpp
WebCore/dom/Position.cpp
WebCore/dom/Range.cpp
WebCore/dom/StyledElement.cpp
WebCore/dom/xml_tokenizer.cpp
WebCore/editing/ApplyStyleCommand.cpp
WebCore/html/HTMLGenericFormElement.cpp
WebCore/html/HTMLInputElement.cpp
WebCore/html/HTMLSelectElement.cpp
WebCore/html/HTMLTextAreaElement.cpp
WebCore/html/html_baseimpl.cpp
WebCore/html/html_blockimpl.cpp
WebCore/html/html_imageimpl.cpp
WebCore/html/html_inlineimpl.cpp
WebCore/html/html_listimpl.cpp
WebCore/html/html_objectimpl.cpp
WebCore/html/html_tableimpl.cpp
WebCore/khtml/ecma/kjs_window.cpp
WebCore/ksvg2/css/SVGCSSParser.cpp
WebCore/ksvg2/svg/SVGAngle.cpp
WebCore/ksvg2/svg/SVGAngle.h
WebCore/ksvg2/svg/SVGLength.cpp
WebCore/ksvg2/svg/SVGLength.h
WebCore/ksvg2/svg/SVGPathSeg.h
WebCore/ksvg2/svg/SVGPathSegArc.h
WebCore/ksvg2/svg/SVGPathSegClosePath.h
WebCore/ksvg2/svg/SVGPathSegCurvetoCubic.h
WebCore/ksvg2/svg/SVGPathSegCurvetoCubicSmooth.h
WebCore/ksvg2/svg/SVGPathSegCurvetoQuadratic.h
WebCore/ksvg2/svg/SVGPathSegCurvetoQuadraticSmooth.h
WebCore/ksvg2/svg/SVGPathSegLineto.h
WebCore/ksvg2/svg/SVGPathSegLinetoHorizontal.h
WebCore/ksvg2/svg/SVGPathSegLinetoVertical.h
WebCore/ksvg2/svg/SVGPathSegMoveto.h
WebCore/ksvg2/svg/SVGPolyElement.cpp
WebCore/ksvg2/svg/SVGStopElement.cpp
WebCore/ksvg2/svg/SVGUseElement.cpp
WebCore/page/Frame.cpp
WebCore/platform/AtomicString.h
WebCore/platform/Color.cpp
WebCore/platform/Color.h
WebCore/platform/PlatformString.h
WebCore/platform/String.cpp
WebCore/platform/StringImpl.cpp
WebCore/platform/mac/KeyEventMac.mm
WebCore/rendering/render_form.cpp

index 546379b412eaa69cb7013efbe5cab6ad859eb7c2..b0225d3abb290d68133365f73b86cfc4fb8c3362 100644 (file)
@@ -1,3 +1,177 @@
+2006-03-27  Eric Seidel  <eseidel@apple.com>
+
+        Reviewed by darin.
+
+        Replace more DeprecatedString with String.
+        Add String::sprintf() and String::number()
+        http://bugzilla.opendarwin.org/show_bug.cgi?id=8009
+
+        * bindings/objc/DOMHTML.mm:
+        (-[DOMHTMLTextAreaElement setCols:]):
+        (-[DOMHTMLTextAreaElement setRows:]):
+        (-[DOMHTMLOListElement setStart:]):
+        (-[DOMHTMLPreElement setWidth:]):
+        (-[DOMHTMLImageElement setHeight:]):
+        (-[DOMHTMLImageElement setHspace:]):
+        (-[DOMHTMLImageElement setVspace:]):
+        (-[DOMHTMLImageElement setWidth:]):
+        (-[DOMHTMLObjectElement setHspace:]):
+        (-[DOMHTMLObjectElement setTabIndex:]):
+        (-[DOMHTMLObjectElement setVspace:]):
+        (-[DOMHTMLAppletElement setHspace:]):
+        (-[DOMHTMLAppletElement setVspace:]):
+        (-[DOMHTMLAreaElement setTabIndex:]):
+        (-[DOMHTMLTableColElement setSpan:]):
+        (-[DOMHTMLTableCellElement setColSpan:]):
+        (-[DOMHTMLTableCellElement setRowSpan:]):
+        (-[DOMHTMLEmbedElement setHeight:]):
+        (-[DOMHTMLEmbedElement setWidth:]):
+        * bindings/objc/DOMInternal.mm:
+        (String::String):
+        * css/CSSComputedStyleDeclaration.cpp:
+        (WebCore::numberAsString):
+        (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
+        * css/css_valueimpl.cpp:
+        (WebCore::CSSMutableStyleDeclaration::getPropertyValue):
+        (WebCore::CSSPrimitiveValue::cssText):
+        * dom/Position.cpp:
+        (WebCore::Position::formatForDebugger):
+        * dom/Range.cpp:
+        (WebCore::Range::formatForDebugger):
+        * dom/StyledElement.cpp:
+        (WebCore::StyledElement::addCSSColor):
+        * dom/xml_tokenizer.cpp:
+        (WebCore::XMLTokenizer::error):
+        * editing/ApplyStyleCommand.cpp:
+        (WebCore::ApplyStyleCommand::applyRelativeFontStyleChange):
+        * html/HTMLGenericFormElement.cpp:
+        (WebCore::HTMLGenericFormElement::findMatchingState):
+        (WebCore::HTMLGenericFormElement::setTabIndex):
+        * html/HTMLInputElement.cpp:
+        (WebCore::HTMLInputElement::setMaxLength):
+        (WebCore::HTMLInputElement::setSize):
+        * html/HTMLSelectElement.cpp:
+        (WebCore::HTMLSelectElement::setSize):
+        * html/HTMLTextAreaElement.cpp:
+        (WebCore::HTMLTextAreaElement::setCols):
+        (WebCore::HTMLTextAreaElement::setRows):
+        * html/html_baseimpl.cpp:
+        (WebCore::HTMLBodyElement::insertedIntoDocument):
+        * html/html_blockimpl.cpp:
+        (WebCore::HTMLHRElement::parseMappedAttribute):
+        (WebCore::HTMLPreElement::setWidth):
+        * html/html_imageimpl.cpp:
+        (WebCore::HTMLImageElement::width):
+        (WebCore::HTMLImageElement::height):
+        (WebCore::HTMLImageElement::setBorder):
+        (WebCore::HTMLImageElement::setHeight):
+        (WebCore::HTMLImageElement::setHspace):
+        (WebCore::HTMLImageElement::setVspace):
+        (WebCore::HTMLImageElement::setWidth):
+        (WebCore::HTMLAreaElement::setTabIndex):
+        * html/html_inlineimpl.cpp:
+        (WebCore::HTMLAnchorElement::setTabIndex):
+        * html/html_listimpl.cpp:
+        (WebCore::HTMLOListElement::setStart):
+        (WebCore::HTMLLIElement::setValue):
+        * html/html_objectimpl.cpp:
+        (WebCore::HTMLObjectElement::setTabIndex):
+        * html/html_tableimpl.cpp:
+        (WebCore::HTMLTableElement::parseMappedAttribute):
+        (WebCore::HTMLTableCellElement::setColSpan):
+        (WebCore::HTMLTableCellElement::setRowSpan):
+        (WebCore::HTMLTableColElement::setSpan):
+        * khtml/ecma/kjs_window.cpp:
+        (KJS::Window::isSafeScript):
+        (KJS::Location::getValueProperty):
+        * ksvg2/css/SVGCSSParser.cpp:
+        (WebCore::CSSParser::parseSVGPaint):
+        (WebCore::CSSParser::parseSVGColor):
+        * ksvg2/svg/SVGAngle.cpp:
+        (SVGAngle::setValueAsString):
+        (SVGAngle::valueAsString):
+        * ksvg2/svg/SVGAngle.h:
+        * ksvg2/svg/SVGLength.cpp:
+        (SVGLength::setValueAsString):
+        (SVGLength::valueAsString):
+        * ksvg2/svg/SVGLength.h:
+        * ksvg2/svg/SVGPathSeg.h:
+        (WebCore::SVGPathSeg::pathSegTypeAsLetter):
+        (WebCore::SVGPathSeg::toString):
+        * ksvg2/svg/SVGPathSegArc.h:
+        (WebCore::SVGPathSegArcAbs::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegArcAbs::toString):
+        (WebCore::SVGPathSegArcRel::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegArcRel::toString):
+        * ksvg2/svg/SVGPathSegClosePath.h:
+        (WebCore::SVGPathSegClosePath::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegClosePath::toString):
+        * ksvg2/svg/SVGPathSegCurvetoCubic.h:
+        (WebCore::SVGPathSegCurvetoCubicAbs::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegCurvetoCubicAbs::toString):
+        (WebCore::SVGPathSegCurvetoCubicRel::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegCurvetoCubicRel::toString):
+        * ksvg2/svg/SVGPathSegCurvetoCubicSmooth.h:
+        (WebCore::SVGPathSegCurvetoCubicSmoothAbs::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegCurvetoCubicSmoothAbs::toString):
+        (WebCore::SVGPathSegCurvetoCubicSmoothRel::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegCurvetoCubicSmoothRel::toString):
+        * ksvg2/svg/SVGPathSegCurvetoQuadratic.h:
+        (WebCore::SVGPathSegCurvetoQuadraticAbs::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegCurvetoQuadraticAbs::toString):
+        (WebCore::SVGPathSegCurvetoQuadraticRel::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegCurvetoQuadraticRel::toString):
+        * ksvg2/svg/SVGPathSegCurvetoQuadraticSmooth.h:
+        (WebCore::SVGPathSegCurvetoQuadraticSmoothAbs::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegCurvetoQuadraticSmoothAbs::toString):
+        (WebCore::SVGPathSegCurvetoQuadraticSmoothRel::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegCurvetoQuadraticSmoothRel::toString):
+        * ksvg2/svg/SVGPathSegLineto.h:
+        (WebCore::SVGPathSegLinetoAbs::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegLinetoAbs::toString):
+        (WebCore::SVGPathSegLinetoRel::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegLinetoRel::toString):
+        * ksvg2/svg/SVGPathSegLinetoHorizontal.h:
+        (WebCore::SVGPathSegLinetoHorizontalAbs::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegLinetoHorizontalAbs::toString):
+        (WebCore::SVGPathSegLinetoHorizontalRel::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegLinetoHorizontalRel::toString):
+        * ksvg2/svg/SVGPathSegLinetoVertical.h:
+        (WebCore::SVGPathSegLinetoVerticalAbs::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegLinetoVerticalAbs::toString):
+        (WebCore::SVGPathSegLinetoVerticalRel::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegLinetoVerticalRel::toString):
+        * ksvg2/svg/SVGPathSegMoveto.h:
+        (WebCore::SVGPathSegMovetoAbs::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegMovetoAbs::toString):
+        (WebCore::SVGPathSegMovetoRel::pathSegTypeAsLetter):
+        (WebCore::SVGPathSegMovetoRel::toString):
+        * ksvg2/svg/SVGPolyElement.cpp:
+        (SVGPolyElement::notifyAttributeChange):
+        * ksvg2/svg/SVGStopElement.cpp:
+        (SVGStopElement::parseMappedAttribute):
+        * ksvg2/svg/SVGUseElement.cpp:
+        (SVGUseElement::closeRenderer):
+        * platform/AtomicString.h:
+        (WebCore::AtomicString::toInt):
+        * platform/Color.cpp:
+        (WebCore::Color::name):
+        * platform/Color.h:
+        * platform/PlatformString.h:
+        * platform/String.cpp:
+        (WebCore::String::sprintf):
+        (WebCore::String::number):
+        * platform/StringImpl.cpp:
+        (WebCore::StringImpl::StringImpl):
+        (WebCore::StringImpl::initWithChar):
+        (WebCore::StringImpl::initWithQChar):
+        (WebCore::StringImpl::containsOnlyWhitespace):
+        * platform/mac/KeyEventMac.mm:
+        (WebCore::keyIdentifierForKeyEvent):
+        * rendering/render_form.cpp:
+        (WebCore::RenderSlider::updateFromElement):
+        (WebCore::RenderSlider::valueChanged):
+
 2006-03-28  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by mjs
index 016485a1a461d4c53712da1a93bd01eaab977a5b..fa37e989f7d622ccfbd98d54ff8235919cb2ae20 100644 (file)
@@ -1478,8 +1478,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setCols:(int)cols
 {
-    String value(DeprecatedString::number(cols));
-    [self _textAreaElement]->setAttribute(colsAttr, value);
+    [self _textAreaElement]->setAttribute(colsAttr, String::number(cols));
 }
 
 - (BOOL)disabled
@@ -1519,8 +1518,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setRows:(int)rows
 {
-    String value(DeprecatedString::number(rows));
-    [self _textAreaElement]->setAttribute(rowsAttr, value);
+    [self _textAreaElement]->setAttribute(rowsAttr, String::number(rows));
 }
 
 - (int)tabIndex
@@ -1777,8 +1775,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setStart:(int)start
 {
-    String value(DeprecatedString::number(start));
-    [self _oListElement]->setAttribute(startAttr, value);
+    [self _oListElement]->setAttribute(startAttr, String::number(start));
 }
 
 - (NSString *)type
@@ -1969,8 +1966,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setWidth:(int)width
 {
-    String string(DeprecatedString::number(width));
-    [self _preElement]->setAttribute(widthAttr, string);
+    [self _preElement]->setAttribute(widthAttr, String::number(width));
 }
 
 @end
@@ -2357,8 +2353,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setHeight:(int)height
 {
-    String string(DeprecatedString::number(height));
-    [self _imageElement]->setAttribute(heightAttr, string);
+    [self _imageElement]->setAttribute(heightAttr, String::number(height));
 }
 
 - (int)hspace
@@ -2368,8 +2363,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setHspace:(int)hspace
 {
-    String string(DeprecatedString::number(hspace));
-    [self _imageElement]->setAttribute(hspaceAttr, string);
+    [self _imageElement]->setAttribute(hspaceAttr, String::number(hspace));
 }
 
 - (BOOL)isMap
@@ -2424,8 +2418,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setVspace:(int)vspace
 {
-    String string(DeprecatedString::number(vspace));
-    [self _imageElement]->setAttribute(vspaceAttr, string);
+    [self _imageElement]->setAttribute(vspaceAttr, String::number(vspace));
 }
 
 - (int)width
@@ -2435,8 +2428,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setWidth:(int)width
 {
-    String string(DeprecatedString::number(width));
-    [self _imageElement]->setAttribute(widthAttr, string);
+    [self _imageElement]->setAttribute(widthAttr, String::number(width));
 }
 
 @end
@@ -2557,8 +2549,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setHspace:(int)hspace
 {
-    String string(DeprecatedString::number(hspace));
-    [self _objectElement]->setAttribute(hspaceAttr, string);
+    [self _objectElement]->setAttribute(hspaceAttr, String::number(hspace));
 }
 
 - (NSString *)name
@@ -2588,8 +2579,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setTabIndex:(int)tabIndex
 {
-    String string(DeprecatedString::number(tabIndex));
-    [self _objectElement]->setAttribute(tabindexAttr, string);
+    [self _objectElement]->setAttribute(tabindexAttr, String::number(tabIndex));
 }
 
 - (NSString *)type
@@ -2619,8 +2609,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setVspace:(int)vspace
 {
-    String string(DeprecatedString::number(vspace));
-    [self _objectElement]->setAttribute(vspaceAttr, string);
+    [self _objectElement]->setAttribute(vspaceAttr, String::number(vspace));
 }
 
 - (NSString *)width
@@ -2769,8 +2758,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setHspace:(int)hspace
 {
-    String string(DeprecatedString::number(hspace));
-    [self _appletElement]->setAttribute(hspaceAttr, string);
+    [self _appletElement]->setAttribute(hspaceAttr, String::number(hspace));
 }
 
 - (NSString *)name
@@ -2800,8 +2788,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setVspace:(int)vspace
 {
-    String string(DeprecatedString::number(vspace));
-    [self _appletElement]->setAttribute(vspaceAttr, string);
+    [self _appletElement]->setAttribute(vspaceAttr, String::number(vspace));
 }
 
 - (NSString *)width
@@ -2926,8 +2913,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setTabIndex:(int)tabIndex
 {
-    String string(DeprecatedString::number(tabIndex));
-    [self _areaElement]->setAttribute(tabindexAttr, string);
+    [self _areaElement]->setAttribute(tabindexAttr, String::number(tabIndex));
 }
 
 - (NSString *)target
@@ -3372,8 +3358,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setSpan:(int)span
 {
-    String string(DeprecatedString::number(span));
-    [self _tableColElement]->setAttribute(spanAttr, string);
+    [self _tableColElement]->setAttribute(spanAttr, String::number(span));
 }
 
 - (NSString *)vAlign
@@ -3562,8 +3547,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setColSpan:(int)colSpan
 {
-    String string(DeprecatedString::number(colSpan));
-    [self _tableCellElement]->setAttribute(colspanAttr, string);
+    [self _tableCellElement]->setAttribute(colspanAttr, String::number(colSpan));
 }
 
 - (NSString *)headers
@@ -3603,8 +3587,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setRowSpan:(int)rowSpan
 {
-    String string(DeprecatedString::number(rowSpan));
-    [self _tableCellElement]->setAttribute(rowspanAttr, string);
+    [self _tableCellElement]->setAttribute(rowspanAttr, String::number(rowSpan));
 }
 
 - (NSString *)scope
@@ -3926,8 +3909,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setHeight:(int)height
 {
-    String string(DeprecatedString::number(height));
-    [self _embedElement]->setAttribute(heightAttr, string);
+    [self _embedElement]->setAttribute(heightAttr, String::number(height));
 }
 
 - (NSString *)name
@@ -3967,8 +3949,7 @@ using namespace WebCore::HTMLNames;
 
 - (void)setWidth:(int)width
 {
-    String string(DeprecatedString::number(width));
-    [self _embedElement]->setAttribute(widthAttr, string);
+    [self _embedElement]->setAttribute(widthAttr, String::number(width));
 }
 
 @end
index 80c2c0a3c5a718b782e98b8018fe2363c63e6bec..d84fd915bd8da524f84942ae914a5dc6056629be 100644 (file)
@@ -118,16 +118,14 @@ String::String(NSString *str)
     else {
         UniChar fixedSizeBuffer[1024];
         UniChar *buffer;
-        if (size > static_cast<CFIndex>(sizeof(fixedSizeBuffer) / sizeof(UniChar))) {
+        if (size > static_cast<CFIndex>(sizeof(fixedSizeBuffer) / sizeof(UniChar)))
             buffer = static_cast<UniChar *>(fastMalloc(size * sizeof(UniChar)));
-        } else {
+        else
             buffer = fixedSizeBuffer;
-        }
         CFStringGetCharacters(reinterpret_cast<CFStringRef>(str), CFRangeMake(0, size), buffer);
         m_impl = new StringImpl(reinterpret_cast<const QChar *>(buffer), (unsigned)size);
-        if (buffer != fixedSizeBuffer) {
+        if (buffer != fixedSizeBuffer)
             fastFree(buffer);
-        }
     }
 }
 
index 9592a7179e49545888938f5373eaa337a38f2e28..418b74825792beb0e7faef700a3b63e8aae1bcf6 100644 (file)
@@ -297,10 +297,10 @@ void CSSComputedStyleDeclaration::setCssText(const String&, ExceptionCode& ec)
 }
 
 // Display integers in integer format instead of "1.0".
-static DeprecatedString numberAsString(double n)
+static String numberAsString(double n)
 {
     long i = static_cast<long>(n);
-    return i == n ? DeprecatedString::number(i) : DeprecatedString::number(n);
+    return i == n ? String::number(i) : String::number(n);
 }
 
 PassRefPtr<CSSValue> CSSComputedStyleDeclaration::getPropertyCSSValue(int propertyID) const
@@ -391,10 +391,7 @@ PassRefPtr<CSSValue> CSSComputedStyleDeclaration::getPropertyCSSValue(int proper
             return new CSSPrimitiveValue(CSS_VAL_SEPARATE);
     case CSS_PROP_BORDER_SPACING:
     {
-        DeprecatedString string(numberAsString(style->horizontalBorderSpacing()) + 
-            "px " + 
-            numberAsString(style->verticalBorderSpacing()) +
-            "px");
+        String string(numberAsString(style->horizontalBorderSpacing()) + "px " + numberAsString(style->verticalBorderSpacing()) + "px");
         return new CSSPrimitiveValue(string, CSSPrimitiveValue::CSS_STRING);
     }
     case CSS_PROP__KHTML_BORDER_HORIZONTAL_SPACING:
index 3362ab7f4cf6c7e62de919d2ffea7e95bdedaf1b..e663bbeb91b3a7f5bb33be6f668b90947c3f9e77 100644 (file)
@@ -296,7 +296,6 @@ String CSSMutableStyleDeclaration::getPropertyValue(int propertyID) const
         return getShortHandValue( properties, 3 );
     }
     }
-    //kdDebug() << k_funcinfo << "property not found:" << propertyID << endl;
     return String();
 }
 
@@ -938,55 +937,55 @@ String CSSPrimitiveValue::cssText() const
             // ###
             break;
         case CSS_NUMBER:
-            text = DeprecatedString::number(m_value.num);
+            text = String::number(m_value.num);
             break;
         case CSS_PERCENTAGE:
-            text = DeprecatedString::number(m_value.num) + "%";
+            text = String::number(m_value.num) + "%";
             break;
         case CSS_EMS:
-            text = DeprecatedString::number(m_value.num) + "em";
+            text = String::number(m_value.num) + "em";
             break;
         case CSS_EXS:
-            text = DeprecatedString::number(m_value.num) + "ex";
+            text = String::number(m_value.num) + "ex";
             break;
         case CSS_PX:
-            text = DeprecatedString::number(m_value.num) + "px";
+            text = String::number(m_value.num) + "px";
             break;
         case CSS_CM:
-            text = DeprecatedString::number(m_value.num) + "cm";
+            text = String::number(m_value.num) + "cm";
             break;
         case CSS_MM:
-            text = DeprecatedString::number(m_value.num) + "mm";
+            text = String::number(m_value.num) + "mm";
             break;
         case CSS_IN:
-            text = DeprecatedString::number(m_value.num) + "in";
+            text = String::number(m_value.num) + "in";
             break;
         case CSS_PT:
-            text = DeprecatedString::number(m_value.num) + "pt";
+            text = String::number(m_value.num) + "pt";
             break;
         case CSS_PC:
-            text = DeprecatedString::number(m_value.num) + "pc";
+            text = String::number(m_value.num) + "pc";
             break;
         case CSS_DEG:
-            text = DeprecatedString::number(m_value.num) + "deg";
+            text = String::number(m_value.num) + "deg";
             break;
         case CSS_RAD:
-            text = DeprecatedString::number(m_value.num) + "rad";
+            text = String::number(m_value.num) + "rad";
             break;
         case CSS_GRAD:
-            text = DeprecatedString::number(m_value.num) + "grad";
+            text = String::number(m_value.num) + "grad";
             break;
         case CSS_MS:
-            text = DeprecatedString::number(m_value.num) + "ms";
+            text = String::number(m_value.num) + "ms";
             break;
         case CSS_S:
-            text = DeprecatedString::number(m_value.num) + "s";
+            text = String::number(m_value.num) + "s";
             break;
         case CSS_HZ:
-            text = DeprecatedString::number(m_value.num) + "hz";
+            text = String::number(m_value.num) + "hz";
             break;
         case CSS_KHZ:
-            text = DeprecatedString::number(m_value.num) + "khz";
+            text = String::number(m_value.num) + "khz";
             break;
         case CSS_DIMENSION:
             // ###
@@ -1021,11 +1020,11 @@ String CSSPrimitiveValue::cssText() const
                 text = "rgba(";
             else
                 text = "rgb(";
-            text += DeprecatedString::number(color.red()) + ", ";
-            text += DeprecatedString::number(color.green()) + ", ";
-            text += DeprecatedString::number(color.blue());
+            text += String::number(color.red()) + ", ";
+            text += String::number(color.green()) + ", ";
+            text += String::number(color.blue());
             if (color.alpha() < 0xFF)
-                text += ", " + DeprecatedString::number((float)color.alpha() / 0xFF);
+                text += ", " + String::number((float)color.alpha() / 0xFF);
             text += ")";
             break;
         }
index 6447974cd3edb491e70ad6bbe33b137456e87735..00e1a1a524ed6ce05446cd4fda0ed6cc02a39528 100644 (file)
@@ -689,12 +689,12 @@ void Position::formatForDebugger(char *buffer, unsigned length) const
 {
     String result;
     
-    if (isNull()) {
+    if (isNull())
         result = "<null>";
-    else {
+    else {
         char s[1024];
         result += "offset ";
-        result += DeprecatedString::number(m_offset);
+        result += String::number(m_offset);
         result += " of ";
         m_node->formatForDebugger(s, sizeof(s));
         result += s;
index 6050390357550382b8b5ee9fa0694d9892acf99c..dcc2750a43d9e4a90c511893735f6ad5d41225cc 100644 (file)
@@ -1302,18 +1302,17 @@ void Range::formatForDebugger(char *buffer, unsigned length) const
     String result;
     String s;
     
-    if (!m_startContainer || !m_endContainer) {
+    if (!m_startContainer || !m_endContainer)
         result = "<empty>";
-    }
     else {
         char s[FormatBufferSize];
         result += "from offset ";
-        result += DeprecatedString::number(m_startOffset);
+        result += String::number(m_startOffset);
         result += " of ";
         m_startContainer->formatForDebugger(s, FormatBufferSize);
         result += s;
         result += " to offset ";
-        result += DeprecatedString::number(m_endOffset);
+        result += String::number(m_endOffset);
         result += " of ";
         m_endContainer->formatForDebugger(s, FormatBufferSize);
         result += s;
index e2a0e643e0f2afe1cf97af00c1c9ee7aa72cf715..7257b47ad89b6f9d297942003659544bf8d50a26 100644 (file)
@@ -326,15 +326,16 @@ void StyledElement::addCSSLength(MappedAttribute* attr, int id, const String &va
 void StyledElement::addCSSColor(MappedAttribute* attr, int id, const String &c)
 {
     // this is the only case no color gets applied in IE.
-    if ( !c.length() )
+    if (!c.length())
         return;
 
-    if (!attr->decl()) createMappedDecl(attr);
+    if (!attr->decl())
+        createMappedDecl(attr);
     
-    if (attr->decl()->setProperty(id, c, false) )
+    if (attr->decl()->setProperty(id, c, false))
         return;
     
-    DeprecatedString color = c.deprecatedString();
+    String color = c.copy();
     // not something that fits the specs.
     
     // we're emulating IEs color parser here. It maps transparent to black, otherwise it tries to build a rgb value
@@ -352,30 +353,30 @@ void StyledElement::addCSSColor(MappedAttribute* attr, int id, const String &c)
     // used as a normalization point to normalize to values between 0
     // and 255.
     
-    if ( color.lower() != "transparent" ) {
-        if ( color[0] == '#' )
-            color.remove( 0,  1 );
+    if (color.lower() != "transparent") {
+        if (color[0] == '#')
+            color.remove(0, 1);
         int basicLength = (color.length() + 2) / 3;
-        if ( basicLength > 1 ) {
+        if (basicLength > 1) {
             // IE ignores colors with three digits or less
             int colors[3] = { 0, 0, 0 };
             int component = 0;
             int pos = 0;
             int maxDigit = basicLength-1;
-            while ( component < 3 ) {
+            while (component < 3) {
                 // search forward for digits in the string
                 int numDigits = 0;
-                while ( pos < (int)color.length() && numDigits < basicLength ) {
-                    int hex = toHex( color[pos] );
+                while (pos < (int)color.length() && numDigits < basicLength) {
+                    int hex = toHex(color[pos]);
                     colors[component] = (colors[component] << 4);
-                    if ( hex > 0 ) {
+                    if (hex > 0) {
                         colors[component] += hex;
-                        maxDigit = kMin( maxDigit, numDigits );
+                        maxDigit = kMin(maxDigit, numDigits);
                     }
                     numDigits++;
                     pos++;
                 }
-                while ( numDigits++ < basicLength )
+                while (numDigits++ < basicLength)
                     colors[component] <<= 4;
                 component++;
             }
@@ -386,10 +387,10 @@ void StyledElement::addCSSColor(MappedAttribute* attr, int id, const String &c)
             colors[0] >>= 4*maxDigit;
             colors[1] >>= 4*maxDigit;
             colors[2] >>= 4*maxDigit;
-            // assert( colors[0] < 0x100 && colors[1] < 0x100 && colors[2] < 0x100 );
+            // assert(colors[0] < 0x100 && colors[1] < 0x100 && colors[2] < 0x100);
             
-            color.sprintf("#%02x%02x%02x", colors[0], colors[1], colors[2] );
-            if ( attr->decl()->setProperty(id, String(color), false) )
+            color = String::sprintf("#%02x%02x%02x", colors[0], colors[1], colors[2]);
+            if (attr->decl()->setProperty(id, color, false))
                 return;
         }
     }
index 1996f5d0d63bd7d2d692555964d1d2dababfb451..0d5f451b53c2cc33dc9c9c9fbbf2aed130b358fc 100644 (file)
@@ -536,15 +536,15 @@ void XMLTokenizer::error(ErrorType type, const char *message, va_list args)
 
     if (type == fatal || (m_errorCount < maxErrors && m_lastErrorLine != lineNumber() && m_lastErrorColumn != columnNumber())) {
 
-        DeprecatedString format;
+        const char* format;
         switch (type) {
             case warning:
-                format = DeprecatedString("warning on line %2 at column %3: %1");
+                format = "warning on line %d at column %d: %s";
                 break;
             case fatal:
                 // fall through
             case nonFatal:
-                format = DeprecatedString("error on line %2 at column %3: %1");
+                format = "error on line %d at column %d: %s";
         }
 
 #if WIN32
@@ -554,7 +554,7 @@ void XMLTokenizer::error(ErrorType type, const char *message, va_list args)
         char *m;
         vasprintf(&m, message, args);
 #endif
-        m_errorMessages += format.arg(m).arg(lineNumber()).arg(columnNumber());
+        m_errorMessages += String::sprintf(format, lineNumber(), columnNumber(), m);
 #if !WIN32
         free(m);
 #endif
index 8ae53a2c4816b0b34c24dd5826b262aa3ded235e..f47773ef7f6e46d0ea27a534bf091b81a74bb705 100644 (file)
@@ -511,9 +511,7 @@ void ApplyStyleCommand::applyRelativeFontStyleChange(CSSMutableStyleDeclaration
             currentFontSize = computedFontSize(node);
         }
         if (currentFontSize != desiredFontSize) {
-            DeprecatedString desiredFontSizeString = DeprecatedString::number(desiredFontSize);
-            desiredFontSizeString += "px";
-            inlineStyleDecl->setProperty(CSS_PROP_FONT_SIZE, desiredFontSizeString, false, false);
+            inlineStyleDecl->setProperty(CSS_PROP_FONT_SIZE, String::number(desiredFontSize) + "px", false, false);
             setNodeAttribute(elem, styleAttr, inlineStyleDecl->cssText());
         }
         if (inlineStyleDecl->length() == 0) {
index 7979046f13543792d75f27250f23e47a26b57647..f31831808e773bfd790d51fc5612e9d790822639 100644 (file)
@@ -281,11 +281,10 @@ DeprecatedString HTMLGenericFormElement::findMatchingState(DeprecatedStringList
         assert(sep1 >= 0);
         assert(sep2 >= 0);
 
-        DeprecatedString nameAndType = state.left(sep2);
+        String nameAndType = state.left(sep2);
         if (encName.length() + typeStr.length() + 1 == (unsigned)sep2
             && nameAndType.startsWith(encName)
-            && nameAndType.endsWith(typeStr))
-        {
+            && nameAndType.endsWith(typeStr)) {
             states.remove(it);
             return state.mid(sep2+1);
         }
@@ -300,7 +299,7 @@ int HTMLGenericFormElement::tabIndex() const
 
 void HTMLGenericFormElement::setTabIndex(int value)
 {
-    setAttribute(tabindexAttr, DeprecatedString::number(value));
+    setAttribute(tabindexAttr, String::number(value));
 }
     
 } // namespace
index e6760cf49d9d56338778f9ac80fceba7049ebb0f..05d30f21afa5ae657982c4a2396453e57f100913 100644 (file)
@@ -1274,12 +1274,12 @@ void HTMLInputElement::setAlt(const String &value)
 
 void HTMLInputElement::setMaxLength(int _maxLength)
 {
-    setAttribute(maxlengthAttr, DeprecatedString::number(_maxLength));
+    setAttribute(maxlengthAttr, String::number(_maxLength));
 }
 
 void HTMLInputElement::setSize(unsigned _size)
 {
-    setAttribute(sizeAttr, DeprecatedString::number(_size));
+    setAttribute(sizeAttr, String::number(_size));
 }
 
 String HTMLInputElement::src() const
index 1ed8f33a51a97ed3c04c8c1febf9ddc8dc5a5514..a072fd62bf29dc303a7036e77cc2674a6c1b3aec 100644 (file)
@@ -465,7 +465,7 @@ void HTMLSelectElement::setMultiple(bool multiple)
 
 void HTMLSelectElement::setSize(int size)
 {
-    setAttribute(sizeAttr, DeprecatedString::number(size));
+    setAttribute(sizeAttr, String::number(size));
 }
     
 } // namespace
index 11cef511e26e70b9c6baf01e70f4a77a736de455..d5e4436c4c3a347173459501caee2028c4c47efd 100644 (file)
@@ -268,12 +268,12 @@ void HTMLTextAreaElement::setAccessKey(const String &value)
 
 void HTMLTextAreaElement::setCols(int cols)
 {
-    setAttribute(colsAttr, DeprecatedString::number(cols));
+    setAttribute(colsAttr, String::number(cols));
 }
 
 void HTMLTextAreaElement::setRows(int rows)
 {
-    setAttribute(rowsAttr, DeprecatedString::number(rows));
+    setAttribute(rowsAttr, String::number(rows));
 }
 
 } // namespace
index 6b0c8bf0a275eacea7122e394cc37d6c3afef2b5..6a737478acca6b81af67376ab6be818fabdc3e34 100644 (file)
@@ -166,16 +166,10 @@ void HTMLBodyElement::insertedIntoDocument()
     // FIXME: perhaps this code should be in attach() instead of here
 
     FrameView *w = getDocument()->view();
-    if (w && w->marginWidth() != -1) {
-        DeprecatedString s;
-        s.sprintf("%d", w->marginWidth());
-        setAttribute(marginwidthAttr, s);
-    }
-    if (w && w->marginHeight() != -1) {
-        DeprecatedString s;
-        s.sprintf("%d", w->marginHeight());
-        setAttribute(marginheightAttr, s);
-    }
+    if (w && w->marginWidth() != -1)
+        setAttribute(marginwidthAttr, String::number(w->marginWidth()));
+    if (w && w->marginHeight() != -1)
+        setAttribute(marginheightAttr, String::number(w->marginHeight()));
 
     if (w)
         w->scheduleRelayout();
index ed8e5e79981fc2eeb9f50917c6647788cae1a7d1..e2963ef320dd8e2121c49f5a94dc5f7bfc62eb71 100644 (file)
@@ -164,7 +164,7 @@ void HTMLHRElement::parseMappedAttribute(MappedAttribute *attr)
         if (size <= 1)
             addCSSProperty(attr, CSS_PROP_BORDER_BOTTOM_WIDTH, String("0"));
         else
-            addCSSLength(attr, CSS_PROP_HEIGHT, String(DeprecatedString::number(size-2)));
+            addCSSLength(attr, CSS_PROP_HEIGHT, String::number(size-2));
     } else
         HTMLElement::parseMappedAttribute(attr);
 }
@@ -320,7 +320,7 @@ int HTMLPreElement::width() const
 
 void HTMLPreElement::setWidth(int width)
 {
-    setAttribute(widthAttr, DeprecatedString::number(width));
+    setAttribute(widthAttr, String::number(width));
 }
 
 bool HTMLPreElement::wrap() const
index 7ee3db21733c9952c90b42e09064c96bedf56a1c..2076c53216655707c6da424f48f5f8a08da560d7 100644 (file)
@@ -260,7 +260,7 @@ int HTMLImageElement::width(bool ignorePendingStylesheets) const
     if (!renderer()) {
         // check the attribute first for an explicit pixel value
         bool ok;
-        int width = getAttribute(widthAttr).deprecatedString().toInt(&ok);
+        int width = getAttribute(widthAttr).toInt(&ok);
         if (ok)
             return width;
         
@@ -283,7 +283,7 @@ int HTMLImageElement::height(bool ignorePendingStylesheets) const
     if (!renderer()) {
         // check the attribute first for an explicit pixel value
         bool ok;
-        int height = getAttribute(heightAttr).deprecatedString().toInt(&ok);
+        int height = getAttribute(heightAttr).toInt(&ok);
         if (ok)
             return height;
         
@@ -344,12 +344,12 @@ int HTMLImageElement::border() const
 
 void HTMLImageElement::setBorder(int value)
 {
-    setAttribute(borderAttr, DeprecatedString::number(value));
+    setAttribute(borderAttr, String::number(value));
 }
 
 void HTMLImageElement::setHeight(int value)
 {
-    setAttribute(heightAttr, DeprecatedString::number(value));
+    setAttribute(heightAttr, String::number(value));
 }
 
 int HTMLImageElement::hspace() const
@@ -360,7 +360,7 @@ int HTMLImageElement::hspace() const
 
 void HTMLImageElement::setHspace(int value)
 {
-    setAttribute(hspaceAttr, DeprecatedString::number(value));
+    setAttribute(hspaceAttr, String::number(value));
 }
 
 bool HTMLImageElement::isMap() const
@@ -411,12 +411,12 @@ int HTMLImageElement::vspace() const
 
 void HTMLImageElement::setVspace(int value)
 {
-    setAttribute(vspaceAttr, DeprecatedString::number(value));
+    setAttribute(vspaceAttr, String::number(value));
 }
 
 void HTMLImageElement::setWidth(int value)
 {
-    setAttribute(widthAttr, DeprecatedString::number(value));
+    setAttribute(widthAttr, String::number(value));
 }
 
 int HTMLImageElement::x() const
@@ -693,7 +693,7 @@ int HTMLAreaElement::tabIndex() const
 
 void HTMLAreaElement::setTabIndex(int tabIndex)
 {
-    setAttribute(tabindexAttr, DeprecatedString::number(tabIndex));
+    setAttribute(tabindexAttr, String::number(tabIndex));
 }
 
 String HTMLAreaElement::target() const
index eccec786bde2cdb4daca71d51e2e419a9c096920..41961b46a476a167648d54e8719e99a69bba3487 100644 (file)
@@ -303,7 +303,7 @@ int HTMLAnchorElement::tabIndex() const
 
 void HTMLAnchorElement::setTabIndex(int tabIndex)
 {
-    setAttribute(tabindexAttr, DeprecatedString::number(tabIndex));
+    setAttribute(tabindexAttr, String::number(tabIndex));
 }
 
 String HTMLAnchorElement::target() const
index 003b37ff55e3be90138912192d18573f6893756b..c9dc96be5bb192dd7c9f719745427dee58f88696 100644 (file)
@@ -138,7 +138,7 @@ void HTMLOListElement::setCompact(bool b)
 
 void HTMLOListElement::setStart(int start)
 {
-    setAttribute(startAttr, DeprecatedString::number(start));
+    setAttribute(startAttr, String::number(start));
 }
 
 String HTMLOListElement::type() const
@@ -236,7 +236,7 @@ int HTMLLIElement::value() const
 
 void HTMLLIElement::setValue(int value)
 {
-    setAttribute(valueAttr, DeprecatedString::number(value));
+    setAttribute(valueAttr, String::number(value));
 }
 
 // -------------------------------------------------------------------------
index a79e8db2d27e9e696b00a0c8862e39ffcf53f239..39c3cb368e7795cafc32730fd88ebc79063ab9b1 100644 (file)
@@ -1005,7 +1005,7 @@ int HTMLObjectElement::tabIndex() const
 
 void HTMLObjectElement::setTabIndex(int tabIndex)
 {
-    setAttribute(tabindexAttr, DeprecatedString::number(tabIndex));
+    setAttribute(tabindexAttr, String::number(tabIndex));
 }
 
 String HTMLObjectElement::type() const
index 96510616e0e64888bca7dcfdaeb32e964546a36b..931e1c0beff617297bcb4130f49432efb76051a9 100644 (file)
@@ -397,8 +397,7 @@ void HTMLTableElement::parseMappedAttribute(MappedAttribute *attr)
             border=1;
 #endif
             m_noBorder = !border;
-            String v = DeprecatedString::number( border );
-            addCSSLength(attr, CSS_PROP_BORDER_WIDTH, v );
+            addCSSLength(attr, CSS_PROP_BORDER_WIDTH, String::number(border));
         }
     } else if (attr->name() == bgcolorAttr) {
         addCSSColor(attr, CSS_PROP_BACKGROUND_COLOR, attr->value());
@@ -1147,7 +1146,7 @@ void HTMLTableCellElement::setChOff(const String &value)
 
 void HTMLTableCellElement::setColSpan(int n)
 {
-    setAttribute(colspanAttr, DeprecatedString::number(n));
+    setAttribute(colspanAttr, String::number(n));
 }
 
 String HTMLTableCellElement::headers() const
@@ -1182,7 +1181,7 @@ void HTMLTableCellElement::setNoWrap(bool b)
 
 void HTMLTableCellElement::setRowSpan(int n)
 {
-    setAttribute(rowspanAttr, DeprecatedString::number(n));
+    setAttribute(rowspanAttr, String::number(n));
 }
 
 String HTMLTableCellElement::scope() const
@@ -1278,7 +1277,7 @@ void HTMLTableColElement::setChOff(const String &value)
 
 void HTMLTableColElement::setSpan(int n)
 {
-    setAttribute(spanAttr, DeprecatedString::number(n));
+    setAttribute(spanAttr, String::number(n));
 }
 
 String HTMLTableColElement::vAlign() const
index 97413180aa91978cd5746f0ed4fae33fcafce3a6..aeeb4265e8f005f52e0929cfb60340c7e65baa88 100644 (file)
@@ -1223,10 +1223,9 @@ bool Window::isSafeScript(const ScriptInterpreter *origin, const ScriptInterpret
         printf("Unsafe JavaScript attempt to access frame with URL %s from frame with URL %s. Domains must match.\n", 
              targetDocument->URL().latin1(), originDocument->URL().latin1());
     }
-    DeprecatedString message;
-    message.sprintf("Unsafe JavaScript attempt to access frame with URL %s from frame with URL %s. Domains must match.\n", 
+    String message = String::sprintf("Unsafe JavaScript attempt to access frame with URL %s from frame with URL %s. Domains must match.\n", 
                   targetDocument->URL().latin1(), originDocument->URL().latin1());
-    targetPart->addMessageToConsole(String(message), 1, String()); //fixme: provide a real line number and sourceurl
+    targetPart->addMessageToConsole(message, 1, String()); //fixme: provide a real line number and sourceurl
 
     return false;
 }
@@ -1285,10 +1284,9 @@ bool Window::isSafeScript(ExecState *exec) const
       printf("Unsafe JavaScript attempt to access frame with URL %s from frame with URL %s. Domains must match.\n", 
              thisDocument->URL().latin1(), actDocument->URL().latin1());
   }
-  DeprecatedString message;
-  message.sprintf("Unsafe JavaScript attempt to access frame with URL %s from frame with URL %s. Domains must match.\n", 
+  String message = String::sprintf("Unsafe JavaScript attempt to access frame with URL %s from frame with URL %s. Domains must match.\n", 
                   thisDocument->URL().latin1(), actDocument->URL().latin1());
-  m_frame->addMessageToConsole(String(message), 1, String());
+  m_frame->addMessageToConsole(message, 1, String());
   
   return false;
 }
@@ -2078,7 +2076,7 @@ JSValue *Location::getValueProperty(ExecState *exec, int token) const
     // Bleh.
     UString str = url.host();
     if (url.port())
-            str += ":" + DeprecatedString::number((int)url.port());
+        str += ":" + String::number((int)url.port());
     return jsString(str);
   }
   case Hostname:
@@ -2091,7 +2089,7 @@ JSValue *Location::getValueProperty(ExecState *exec, int token) const
   case Pathname:
     return jsString(url.path().isEmpty() ? "/" : url.path());
   case Port:
-    return jsString(url.port() ? DeprecatedString::number((int)url.port()) : "");
+    return jsString(url.port() ? String::number((int)url.port()) : "");
   case Protocol:
     return jsString(url.protocol() + ":");
   case Search:
index 4946316289f3795e403cb5508f4847c55e5b1b4a..a3911899cd336cf410b7e1f5a7b051f19e98dafc 100644 (file)
@@ -345,9 +345,8 @@ CSSValue *CSSParser::parseSVGPaint()
     KDOMCSSValue *value = valueList->current();
     if(!strict && value->unit == CSSPrimitiveValue::CSS_NUMBER &&
        value->fValue >= 0. && value->fValue < 1000000.) {
-        DeprecatedString str;
-        str.sprintf("%06d", (int)(value->fValue+.5));
-        return new SVGPaint(SVG_PAINTTYPE_RGBCOLOR, 0, new StringImpl(str));
+        String str = String::sprintf("%06d", (int)(value->fValue+.5));
+        return new SVGPaint(SVG_PAINTTYPE_RGBCOLOR, 0, str.impl());
     } else if (value->unit == CSSPrimitiveValue::CSS_RGBCOLOR) {
         String str = "#" + domString(value->string);
         return new SVGPaint(SVG_PAINTTYPE_RGBCOLOR, 0, str.impl());
@@ -379,9 +378,8 @@ CSSValue *CSSParser::parseSVGPaint()
         r = kMax(0, kMin(255, r));
         g = kMax(0, kMin(255, g));
         b = kMax(0, kMin(255, b));
-        DeprecatedString str;
-        str.sprintf("rgb(%d, %d, %d)", r, g, b);
-        return new SVGPaint(SVG_PAINTTYPE_RGBCOLOR, 0, new StringImpl(str));
+        
+        return new SVGPaint(SVG_PAINTTYPE_RGBCOLOR, 0, String::sprintf("rgb(%d, %d, %d)", r, g, b).impl());
     }
     else
         return 0;
@@ -392,12 +390,9 @@ CSSValue *CSSParser::parseSVGPaint()
 CSSValue *CSSParser::parseSVGColor()
 {
     KDOMCSSValue *value = valueList->current();
-    if(!strict && value->unit == CSSPrimitiveValue::CSS_NUMBER &&
-       value->fValue >= 0. && value->fValue < 1000000.) {
-        DeprecatedString str;
-        str.sprintf("%06d", (int)(value->fValue+.5));
-        return new SVGColor(new StringImpl(str));
-    } else if (value->unit == CSSPrimitiveValue::CSS_RGBCOLOR) {
+    if (!strict && value->unit == CSSPrimitiveValue::CSS_NUMBER && value->fValue >= 0. && value->fValue < 1000000.)
+        return new SVGColor(String::sprintf("%06d", (int)(value->fValue+.5)).impl());
+    else if (value->unit == CSSPrimitiveValue::CSS_RGBCOLOR) {
         String str = "#" + domString(value->string);
         return new SVGColor(str.impl());
     } else if (value->unit == CSSPrimitiveValue::CSS_IDENT || (!strict && value->unit == CSSPrimitiveValue::CSS_DIMENSION))
@@ -425,9 +420,8 @@ CSSValue *CSSParser::parseSVGColor()
         r = kMax(0, kMin(255, r));
         g = kMax(0, kMin(255, g));
         b = kMax(0, kMin(255, b));
-        DeprecatedString str;
-        str.sprintf("rgb(%d, %d, %d)", r, g, b);
-        return new SVGColor(new StringImpl(str));
+        
+        return new SVGColor(String::sprintf("rgb(%d, %d, %d)", r, g, b).impl());
     }
     else
         return 0;
index 91b8d60a2270ba3fb819e2859c5476a7be74c714..b1fba58ffcedb94d3b00189357b5f6cb0ebb3d21 100644 (file)
@@ -86,9 +86,9 @@ float SVGAngle::valueInSpecifiedUnits() const
     return m_valueInSpecifiedUnits;
 }
 
-void SVGAngle::setValueAsString(StringImpl *valueAsString)
+void SVGAngle::setValueAsString(const String& s)
 {
-    m_valueAsString = String(valueAsString);
+    m_valueAsString = s;
 
     bool bOK;
     m_valueInSpecifiedUnits = m_valueAsString.deprecatedString().toDouble(&bOK);
@@ -106,9 +106,9 @@ void SVGAngle::setValueAsString(StringImpl *valueAsString)
     calculate();
 }
 
-StringImpl *SVGAngle::valueAsString() const
+String SVGAngle::valueAsString() const
 {
-    m_valueAsString = DeprecatedString::number(m_valueInSpecifiedUnits);
+    m_valueAsString = String::number(m_valueInSpecifiedUnits);
 
     switch(m_unitType) {
         case SVG_ANGLETYPE_UNSPECIFIED:
@@ -123,7 +123,7 @@ StringImpl *SVGAngle::valueAsString() const
             break;
     }
     
-    return m_valueAsString.impl();
+    return m_valueAsString;
 }
 
 void SVGAngle::newValueSpecifiedUnits(unsigned short unitType, float valueInSpecifiedUnits)
index ee9c885875e670434a9db08264674712da1b5442..faeeee5bf2c479a689f057c790921fcaf35c6ee8 100644 (file)
@@ -45,8 +45,8 @@ namespace WebCore
         void setValueInSpecifiedUnits(float valueInSpecifiedUnits);
         float valueInSpecifiedUnits() const;
 
-        void setValueAsString(StringImpl *valueAsString);
-        StringImpl *valueAsString() const;
+        void setValueAsString(const String&);
+        String valueAsString() const;
 
         void newValueSpecifiedUnits(unsigned short unitType, float valueInSpecifiedUnits);
         void convertToSpecifiedUnits(unsigned short unitType);
index d8ba176de1e9d3d717100053a2443a4738899870..fbbdbddb8a7dec922801ffd3cda29843627617b6 100644 (file)
@@ -109,13 +109,12 @@ float SVGLength::valueInSpecifiedUnits() const
     return m_valueInSpecifiedUnits;
 }                                                
 
-void SVGLength::setValueAsString(StringImpl* s)
+void SVGLength::setValueAsString(const String& s)
 {
-    String valueAsString(s);
-    if(valueAsString.isEmpty())
+    if(s.isEmpty())
         return;
 
-    DeprecatedString valueAsQString = valueAsString.deprecatedString();
+    DeprecatedString valueAsQString = s.deprecatedString();
 
     double convertedNumber = 0;
     const char *start = valueAsQString.latin1();
@@ -123,27 +122,26 @@ void SVGLength::setValueAsString(StringImpl* s)
     m_valueInSpecifiedUnits = convertedNumber;
 
     unsigned int diff = end - start;
-    if(diff < valueAsQString.length())
-    {
-        if(valueAsQString.endsWith(UnitText[SVG_LENGTHTYPE_PX]))
+    if (diff < valueAsQString.length()) {
+        if (s.endsWith(UnitText[SVG_LENGTHTYPE_PX]))
             m_unitType = SVG_LENGTHTYPE_PX;
-        else if(valueAsQString.endsWith(UnitText[SVG_LENGTHTYPE_CM]))
+        else if (s.endsWith(UnitText[SVG_LENGTHTYPE_CM]))
             m_unitType = SVG_LENGTHTYPE_CM;
-        else if(valueAsQString.endsWith(UnitText[SVG_LENGTHTYPE_PC]))
+        else if (s.endsWith(UnitText[SVG_LENGTHTYPE_PC]))
             m_unitType = SVG_LENGTHTYPE_PC;
-        else if(valueAsQString.endsWith(UnitText[SVG_LENGTHTYPE_MM]))
+        else if (s.endsWith(UnitText[SVG_LENGTHTYPE_MM]))
             m_unitType = SVG_LENGTHTYPE_MM;
-        else if(valueAsQString.endsWith(UnitText[SVG_LENGTHTYPE_IN]))
+        else if (s.endsWith(UnitText[SVG_LENGTHTYPE_IN]))
             m_unitType = SVG_LENGTHTYPE_IN;
-        else if(valueAsQString.endsWith(UnitText[SVG_LENGTHTYPE_PT]))
+        else if (s.endsWith(UnitText[SVG_LENGTHTYPE_PT]))
             m_unitType = SVG_LENGTHTYPE_PT;
-         else if(valueAsQString.endsWith(UnitText[SVG_LENGTHTYPE_PERCENTAGE]))
+         else if (s.endsWith(UnitText[SVG_LENGTHTYPE_PERCENTAGE]))
             m_unitType = SVG_LENGTHTYPE_PERCENTAGE;
-        else if(valueAsQString.endsWith(UnitText[SVG_LENGTHTYPE_EMS]))
+        else if (s.endsWith(UnitText[SVG_LENGTHTYPE_EMS]))
             m_unitType = SVG_LENGTHTYPE_EMS;
-        else if(valueAsQString.endsWith(UnitText[SVG_LENGTHTYPE_EXS]))
+        else if (s.endsWith(UnitText[SVG_LENGTHTYPE_EXS]))
             m_unitType = SVG_LENGTHTYPE_EXS;
-        else if(valueAsQString.isEmpty())
+        else if (s.isEmpty())
             m_unitType = SVG_LENGTHTYPE_NUMBER;
         else
             m_unitType = SVG_LENGTHTYPE_UNKNOWN;
@@ -154,9 +152,9 @@ void SVGLength::setValueAsString(StringImpl* s)
     updateValue();
 }
 
-StringImpl *SVGLength::valueAsString() const
+String SVGLength::valueAsString() const
 {
-    return new StringImpl(DeprecatedString::number(m_valueInSpecifiedUnits) + UnitText[m_unitType]);
+    return String::number(m_valueInSpecifiedUnits) + UnitText[m_unitType];
 }
 
 void SVGLength::newValueSpecifiedUnits(unsigned short unitType, float valueInSpecifiedUnits)
index ea200294c8a5bc36c203d1504844d84a596143dd..876f7c1fbaf012561796c94084add125c377bd64 100644 (file)
@@ -50,8 +50,8 @@ namespace WebCore
         float valueInSpecifiedUnits() const;
         void setValueInSpecifiedUnits(float valueInSpecifiedUnits);
 
-        StringImpl *valueAsString() const;
-        void setValueAsString(StringImpl *valueAsString);
+        String valueAsString() const;
+        void setValueAsString(const String&);
 
         void newValueSpecifiedUnits(unsigned short unitType, float valueInSpecifiedUnits);
         void convertToSpecifiedUnits(unsigned short unitType);
index ce4106da5f28c52b9eeff99200bb4727eb76eeea..58d23c6a7824f0869ced9094f23ca46cce9c938c 100644 (file)
@@ -25,7 +25,7 @@
 #if SVG_SUPPORT
 
 #include "Shared.h"
-#include "StringImpl.h"
+#include "PlatformString.h"
 
 #include "ksvg.h"
 
@@ -40,8 +40,8 @@ namespace WebCore
         virtual ~SVGPathSeg();
 
         virtual unsigned short pathSegType() const { return PATHSEG_UNKNOWN; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl(""); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1(""); }
+        virtual String pathSegTypeAsLetter() const { return ""; }
+        virtual String toString() const { return ""; }
 
     protected:
         const SVGStyledElement *m_context;
index 43c0d0b4b5083c9c0bca9cd9cdd43758623bb127..ffcfca82e8a54843f9a92ff4da48921848df130a 100644 (file)
@@ -35,8 +35,8 @@ namespace WebCore
         virtual ~SVGPathSegArcAbs();
 
         virtual unsigned short pathSegType() const { return PATHSEG_ARC_ABS; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("A"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("A %1 %2 %3 %4 %5 %6 %7").arg(m_r1).arg(m_r2).arg(m_angle).arg(m_largeArcFlag).arg(m_sweepFlag).arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "A"; }
+        virtual String toString() const { return String::sprintf("A %.6lg %.6lg %.6lg %d %d %.6lg %.6lg", m_r1, m_r2, m_angle, m_largeArcFlag, m_sweepFlag, m_x, m_y); }
 
         void setX(double x);
         double x() const;
@@ -77,8 +77,8 @@ namespace WebCore
         virtual ~SVGPathSegArcRel();
 
         virtual unsigned short pathSegType() const { return PATHSEG_ARC_REL; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("a"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("a %1 %2 %3 %4 %5 %6 %7").arg(m_r1).arg(m_r2).arg(m_angle).arg(m_largeArcFlag).arg(m_sweepFlag).arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "a"; }
+        virtual String toString() const { return String::sprintf("a %.6lg %.6lg %.6lg %d %d %.6lg %.6lg", m_r1, m_r2, m_angle, m_largeArcFlag, m_sweepFlag, m_x, m_y); }
 
         void setX(double x);
         double x() const;
index 1def1020e95ca6f093edee338e94aa49917c01e4..9ae0ec79e1f546da83aebcbeff8a94a6a2bb1b86 100644 (file)
@@ -35,8 +35,8 @@ namespace WebCore
         virtual ~SVGPathSegClosePath();
 
         virtual unsigned short pathSegType() const { return PATHSEG_CLOSEPATH; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("Z"); }
-        virtual DeprecatedString toString() const { return "Z"; }
+        virtual String pathSegTypeAsLetter() const { return "Z"; }
+        virtual String toString() const { return "Z"; }
     };
 };
 
index 5d83616078be3f6b3507dfa95000c136904e6d5b..60f7924d67d7876307c2ec583306816dbf5a2c34 100644 (file)
@@ -35,8 +35,8 @@ namespace WebCore
         virtual ~SVGPathSegCurvetoCubicAbs();
 
         virtual unsigned short pathSegType() const { return PATHSEG_CURVETO_CUBIC_ABS; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("C"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("C %1 %2 %3 %4 %5 %6").arg(m_x1).arg(m_y1).arg(m_x2).arg(m_y2).arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "C"; }
+        virtual String toString() const { return String::sprintf("C %.6lg %.6lg %.6lg %.6lg %.6lg %.6lg", m_x1, m_y1, m_x2, m_y2, m_x, m_y); }
 
         void setX(double);
         double x() const;
@@ -72,8 +72,8 @@ namespace WebCore
         virtual ~SVGPathSegCurvetoCubicRel();
 
         virtual unsigned short pathSegType() const { return PATHSEG_CURVETO_CUBIC_REL; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("c"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("c %1 %2 %3 %4 %5 %6").arg(m_x1).arg(m_y1).arg(m_x2).arg(m_y2).arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "c"; }
+        virtual String toString() const { return String::sprintf("c %.6lg %.6lg %.6lg %.6lg %.6lg %.6lg", m_x1, m_y1, m_x2, m_y2, m_x, m_y); }
 
         void setX(double);
         double x() const;
index ed5f4f1a30c692fb3b4724283e5b9355d0971d12..12e81af2e79635d1a9dcb36ba4a3b126c563697f 100644 (file)
@@ -35,8 +35,8 @@ namespace WebCore
         virtual ~SVGPathSegCurvetoCubicSmoothAbs();
 
         virtual unsigned short pathSegType() const { return PATHSEG_CURVETO_CUBIC_SMOOTH_ABS; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("S"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("S %1 %2 %3 %4").arg(m_x2).arg(m_y2).arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "S"; }
+        virtual String toString() const { return String::sprintf("S %.6lg %.6lg %.6lg %.6lg", m_x2, m_y2, m_x, m_y); }
 
         void setX(double);
         double x() const;
@@ -64,8 +64,8 @@ namespace WebCore
         virtual ~SVGPathSegCurvetoCubicSmoothRel();
 
         virtual unsigned short pathSegType() const { return PATHSEG_CURVETO_CUBIC_SMOOTH_REL; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("s"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("s %1 %2 %3 %4").arg(m_x2).arg(m_y2).arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "s"; }
+        virtual String toString() const { return String::sprintf("s %.6lg %.6lg %.6lg %.6lg", m_x2, m_y2, m_x, m_y); }
 
         void setX(double);
         double x() const;
index fd68a538ec231dbfa2512de23a17ea1eb0225a1c..c9d2f4ce381481f369675b96473fef445de165a7 100644 (file)
@@ -35,8 +35,8 @@ namespace WebCore
         virtual ~SVGPathSegCurvetoQuadraticAbs();
 
         virtual unsigned short pathSegType() const { return PATHSEG_CURVETO_QUADRATIC_ABS; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("Q"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("Q %1 %2 %3 %4").arg(m_x1).arg(m_y1).arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "Q"; }
+        virtual String toString() const { return String::sprintf("Q %.6lg %.6lg %.6lg %.6lg", m_x1, m_y1, m_x, m_y); }
 
         void setX(double);
         double x() const;
@@ -64,8 +64,8 @@ namespace WebCore
         virtual ~SVGPathSegCurvetoQuadraticRel();
 
         virtual unsigned short pathSegType() const { return PATHSEG_CURVETO_QUADRATIC_REL; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("q"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("q %1 %2 %3 %4").arg(m_x1).arg(m_y1).arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "q"; }
+        virtual String toString() const { return String::sprintf("q %.6lg %.6lg %.6lg %.6lg", m_x1, m_y1, m_x, m_y); }
  
         void setX(double);
         double x() const;
index b390c2b061ac76299bf25d1c8b4201d22d1e713d..ce5522708cc96e0e8363bd77aad463ee0c2bf942 100644 (file)
@@ -35,8 +35,8 @@ namespace WebCore
         virtual ~SVGPathSegCurvetoQuadraticSmoothAbs();
 
         virtual unsigned short pathSegType() const { return PATHSEG_CURVETO_QUADRATIC_SMOOTH_ABS; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("T"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("T %1 %2").arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "T"; }
+        virtual String toString() const { return String::sprintf("T %.6lg %.6lg", m_x, m_y); }
 
         void setX(double);
         double x() const;
@@ -56,8 +56,8 @@ namespace WebCore
         virtual ~SVGPathSegCurvetoQuadraticSmoothRel();
 
         virtual unsigned short pathSegType() const { return PATHSEG_CURVETO_QUADRATIC_SMOOTH_REL; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("t"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("t %1 %2").arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "t"; }
+        virtual String toString() const { return String::sprintf("t %.6lg %.6lg", m_x, m_y); }
 
         void setX(double);
         double x() const;
index d02fca49c7d22f009cf80628c3b3c5617f2f912f..5ca4ce65bbfb17425d39de50a926ed357be76acd 100644 (file)
@@ -35,8 +35,8 @@ namespace WebCore
         virtual ~SVGPathSegLinetoAbs();
 
         virtual unsigned short pathSegType() const { return PATHSEG_LINETO_ABS; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("L"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("L %1 %2").arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "L"; }
+        virtual String toString() const { return String::sprintf("L %.6lg %.6lg", m_x, m_y); }
 
         void setX(double);
         double x() const;
@@ -56,8 +56,8 @@ namespace WebCore
         virtual ~SVGPathSegLinetoRel();
 
         virtual unsigned short pathSegType() const { return PATHSEG_LINETO_REL; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("l"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("l %1 %2").arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "l"; }
+        virtual String toString() const { return String::sprintf("l %.6lg %.6lg", m_x, m_y); }
 
         void setX(double);
         double x() const;
index b6dad857cd92ea14d3023f5ce9cc7e0c909903a5..e969317b6615d09893110ecb7b8773b077d61dab 100644 (file)
@@ -35,8 +35,8 @@ namespace WebCore
         virtual ~SVGPathSegLinetoHorizontalAbs();
 
         virtual unsigned short pathSegType() const { return PATHSEG_LINETO_HORIZONTAL_ABS; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("H"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("H %1").arg(m_x); }
+        virtual String pathSegTypeAsLetter() const { return "H"; }
+        virtual String toString() const { return String::sprintf("H %.6lg", m_x); }
 
         void setX(double);
         double x() const;
@@ -52,8 +52,8 @@ namespace WebCore
         virtual ~SVGPathSegLinetoHorizontalRel();
 
         virtual unsigned short pathSegType() const { return PATHSEG_LINETO_HORIZONTAL_REL; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("h"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("h %1").arg(m_x); }
+        virtual String pathSegTypeAsLetter() const { return "h"; }
+        virtual String toString() const { return String::sprintf("h %.6lg", m_x); }
 
         void setX(double);
         double x() const;
index 7fd6afd6a35b9e59b24630278d377064acd5d8fc..902cb6f48bee4f8b9ea1110aae48a93d0a27ab24 100644 (file)
@@ -35,8 +35,8 @@ namespace WebCore
         virtual~SVGPathSegLinetoVerticalAbs();
 
         virtual unsigned short pathSegType() const { return PATHSEG_LINETO_VERTICAL_ABS; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("V"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("V %1").arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "V"; }
+        virtual String toString() const { return String::sprintf("V %.6lg", m_y); }
 
         void setY(double);
         double y() const;
@@ -52,8 +52,8 @@ namespace WebCore
         virtual ~SVGPathSegLinetoVerticalRel();
 
         virtual unsigned short pathSegType() const { return PATHSEG_LINETO_VERTICAL_REL; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("v"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("v %1").arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "v"; }
+        virtual String toString() const { return String::sprintf("v %.6lg", m_y); }
 
         void setY(double);
         double y() const;
index 1873a9df90d4c985ffd73a17f91725ea403a8ed2..0336b15e298dc6329c431283e813b98800b67f91 100644 (file)
@@ -35,8 +35,8 @@ namespace WebCore
         virtual ~SVGPathSegMovetoAbs();
 
         virtual unsigned short pathSegType() const { return PATHSEG_MOVETO_ABS; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("M"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("M %1 %2").arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "M"; }
+        virtual String toString() const { return String::sprintf("M %.6lg %.6lg", m_x, m_y); }
 
         void setX(double);
         double x() const;
@@ -56,8 +56,8 @@ namespace WebCore
         virtual ~SVGPathSegMovetoRel();
 
         virtual unsigned short pathSegType() const { return PATHSEG_MOVETO_REL; }
-        virtual StringImpl *pathSegTypeAsLetter() const { return new StringImpl("m"); }
-        virtual DeprecatedString toString() const { return DeprecatedString::fromLatin1("m %1 %2").arg(m_x).arg(m_y); }
+        virtual String pathSegTypeAsLetter() const { return "m"; }
+        virtual String toString() const { return String::sprintf("m %.6lg %.6lg", m_x, m_y); }
 
         void setX(double);
         double x() const;
index da379f06a907fd7b3edc7d33c749b72d91251e20..ef4d590d285e92c3f02139d11f34906385a2e07d 100644 (file)
@@ -86,7 +86,7 @@ void SVGPolyElement::notifyAttributeChange() const
     int len = points()->numberOfItems();
     for (int i = 0; i < len; ++i) {
         SVGPoint *p = points()->getItem(i);
-        _points += DeprecatedString("%1 %2 ").arg(p->x()).arg(p->y());
+        _points += String::sprintf("%.6lg %.6lg ", p->x(), p->y());
     }
 
     String p("points");
index dba33f9b85975a9e0a37e9a80604f00d6a3d47ef..7470d640b738144d55f68efb92934117887d8048 100644 (file)
@@ -54,15 +54,14 @@ void SVGStopElement::parseMappedAttribute(MappedAttribute *attr)
 {
     const String& value = attr->value();
     if (attr->name() == SVGNames::offsetAttr) {
-        if(value.deprecatedString().endsWith("%"))
+        if(value.endsWith("%"))
             offset()->setBaseVal(value.deprecatedString().left(value.length() - 1).toDouble() / 100.);
         else
             offset()->setBaseVal(value.deprecatedString().toDouble());
     } else
         SVGStyledElement::parseMappedAttribute(attr);
 
-    if(!ownerDocument()->parsing() && attached())
-    {
+    if (!ownerDocument()->parsing() && attached()) {
         recalcStyle(Force);
         
         SVGStyledElement *parentStyled = static_cast<SVGStyledElement *>(parentNode());
index 175f0e4b2e74346586cbfac2c1a33fd92be7a658..aedb67dc94ae906cec1fb3d1279d86c7424f1635 100644 (file)
@@ -111,18 +111,17 @@ void SVGUseElement::closeRenderer()
     float _x = x()->baseVal()->value(), _y = y()->baseVal()->value();
     float _w = width()->baseVal()->value(), _h = height()->baseVal()->value();
     
-    String wString = DeprecatedString::number(_w);
-    String hString = DeprecatedString::number(_h);
+    String wString = String::number(_w);
+    String hString = String::number(_h);
     
     ExceptionCode ec;
-    DeprecatedString trans = DeprecatedString::fromLatin1("translate(%1, %2)").arg(_x).arg(_y);
-    if(target->hasTagName(SVGNames::symbolTag))
-    {
+    String trans = String::sprintf("translate(%f, %f)", _x, _y);
+    if(target->hasTagName(SVGNames::symbolTag)) {
         RefPtr<SVGElement> dummy = new SVGSVGElement(SVGNames::svgTag, getDocument());
-        if(_w > 0)
-            dummy->setAttribute(SVGNames::widthAttr, wString.impl());
-        if(_h > 0)
-            dummy->setAttribute(SVGNames::heightAttr, hString.impl());
+        if (_w > 0)
+            dummy->setAttribute(SVGNames::widthAttr, wString);
+        if (_h > 0)
+            dummy->setAttribute(SVGNames::heightAttr, hString);
         
         SVGSymbolElement *symbol = static_cast<SVGSymbolElement *>(target);
         if (symbol->hasAttribute(SVGNames::viewBoxAttr)) {
@@ -132,28 +131,24 @@ void SVGUseElement::closeRenderer()
         target->cloneChildNodes(dummy.get());
 
         RefPtr<SVGElement> dummy2 = new SVGDummyElement(SVGNames::gTag, getDocument());
-        dummy2->setAttribute(SVGNames::transformAttr, String(trans));
+        dummy2->setAttribute(SVGNames::transformAttr, trans);
         
         appendChild(dummy2, ec);
         dummy2->appendChild(dummy, ec);
-    }
-    else if(target->hasTagName(SVGNames::svgTag))
-    {
+    } else if(target->hasTagName(SVGNames::svgTag)) {
         RefPtr<SVGDummyElement> dummy = new SVGDummyElement(SVGNames::gTag, getDocument());
-        dummy->setAttribute(SVGNames::transformAttr, String(trans));
+        dummy->setAttribute(SVGNames::transformAttr, trans);
         
         RefPtr<SVGElement> root = static_pointer_cast<SVGElement>(target->cloneNode(true));
         if(hasAttribute(SVGNames::widthAttr))
-            root->setAttribute(SVGNames::widthAttr, wString.impl());
+            root->setAttribute(SVGNames::widthAttr, wString);
             
         if(hasAttribute(SVGNames::heightAttr))
-            root->setAttribute(SVGNames::heightAttr, hString.impl());
+            root->setAttribute(SVGNames::heightAttr, hString);
             
         appendChild(dummy, ec);
         dummy->appendChild(root.release(), ec);
-    }
-    else
-    {
+    } else {
         RefPtr<SVGDummyElement> dummy = new SVGDummyElement(SVGNames::gTag, getDocument());
         dummy->setAttribute(SVGNames::transformAttr, trans);
         
index 1d540cb55a0c7edab5252ec5836ab93931d2aa91..14f003c0ce5619bb4f5a176040a2011a6137c4aa 100644 (file)
@@ -1521,11 +1521,10 @@ void Frame::submitForm(const char *action, const String& url, const FormData& fo
          tmpbody.replace('+', ' ');
          tmpbody = KURL::decode_string(tmpbody);  // Decode the rest of it
          bodyEnc = KURL::encode_string(tmpbody);  // Recode for the URL
-      } else {
+      } else
          bodyEnc = KURL::encode_string(formData.flattenToString());
-      }
 
-      nvps.append(DeprecatedString("body=%1").arg(bodyEnc));
+      nvps.append(String::sprintf("body=%s", bodyEnc.latin1()).deprecatedString());
       q = nvps.join("&");
       u.setQuery(q);
   } 
index e7fd836194d683f71489fac2f06ecbd8789654d2..962b16281a6d7fdadeee15afce9f948c732c2d36 100644 (file)
@@ -71,7 +71,7 @@ public:
     bool endsWith(const AtomicString& s, bool caseSensitive = true) const
         { return m_string.endsWith(s.domString(), caseSensitive); }
     
-    int toInt() const { return m_string.toInt(); }
+    int toInt(bool* ok = 0) const { return m_string.toInt(ok); }
     bool percentage(int& p) const { return m_string.percentage(p); }
     Length* toLengthArray(int& len) const { return m_string.toLengthArray(len); }
     Length* toCoordsArray(int& len) const { return m_string.toCoordsArray(len); }
index c60d4caa1455be4604958c5a7b9601745ff263f3..cf966a95ae14ee76bc56d311f854205b8d9992d3 100644 (file)
@@ -27,6 +27,7 @@
 #include "Color.h"
 
 #include "DeprecatedString.h"
+#include "PlatformString.h"
 #include <kxmlcore/Assertions.h>
 
 // Turn off inlining to avoid warning with newer gcc.
@@ -106,13 +107,13 @@ Color::Color(const char *name)
     }
 }
 
-DeprecatedString Color::name() const
+String Color::name() const
 {
-    DeprecatedString name;
+    String name;
     if (alpha() < 0xFF)
-        name.sprintf("#%02X%02X%02X%02X", red(), green(), blue(), alpha());
+        name = String::sprintf("#%02X%02X%02X%02X", red(), green(), blue(), alpha());
     else
-        name.sprintf("#%02X%02X%02X", red(), green(), blue());
+        name = String::sprintf("#%02X%02X%02X", red(), green(), blue());
     return name;
 }
 
index b64925f9c3e96a473b5c000f6d19d2375bea9ee9..e99dcbc1b13fd6ddd9ee6f53287c1be90e95813a 100644 (file)
@@ -40,6 +40,8 @@ typedef struct CGColor *CGColorRef;
 
 namespace WebCore {
 
+class String;
+
 typedef unsigned int RGBA32;                      // RGBA quadruplet
 
 RGBA32 makeRGB(int r, int g, int b);
@@ -51,10 +53,10 @@ public:
     Color(RGBA32 col) : color(col), valid(true) { }
     Color(int r, int g, int b) : color(makeRGB(r, g, b)), valid(true) { }
     Color(int r, int g, int b, int a) : color(makeRGBA(r, g, b, a)), valid(true) { }
-    explicit Color(const DeprecatedString &);
+    explicit Color(const DeprecatedString&);
     explicit Color(const char *);
     
-    DeprecatedString name() const;
+    String name() const;
     void setNamedColor(const DeprecatedString&);
 
     bool isValid() const { return valid; }
index f982925037c2b54c73c2c87229c7cb89703a88ed..051a81c4e359385f7746659bbfc4e53a92912c86 100644 (file)
@@ -96,6 +96,18 @@ public:
 
     const QChar* unicode() const;
     DeprecatedString deprecatedString() const;
+    
+    static String number(int);
+    static String number(unsigned);
+    static String number(long);
+    static String number(unsigned long);
+    static String number(double);
+    
+    static String sprintf(const char *, ...)
+#if __GNUC__
+    __attribute__ ((format (printf, 1, 2)))
+#endif
+    ;
 
     int toInt(bool* ok = 0) const;
     Length* toLengthArray(int& len) const;
index 45c16c08307e89fc489632f89bf1c7fcb0d8d111..221cdb1dbc4970ceca7fc6162a7969d21a15d0d8 100644 (file)
@@ -24,6 +24,7 @@
 #include "PlatformString.h"
 
 #include <kjs/identifier.h>
+#include <kxmlcore/Vector.h>
 
 using namespace KJS;
 
@@ -209,6 +210,73 @@ DeprecatedString String::deprecatedString() const
     return DeprecatedString(m_impl->unicode(), m_impl->length());
 }
 
+String String::sprintf(const char *format, ...)
+{
+    va_list args;
+    va_start(args, format);
+
+    Vector<char, 256> buffer;
+#if PLATFORM(WIN_OS)
+    // Windows vnsprintf does not return the expected size on overflow
+    // So we just have to keep looping until our vsprintf call works!
+    int result = 0;
+    do {
+        if (result < 0)
+            buffer.resize(buffer.size() * 2);
+        result = vsnprintf(buffer.data(), buffer.size(), format, args);
+        // Windows vnsprintf returns -1 both for all errors.  There is no
+        // way to distinguish between "not enough room" and "invalid format"
+        // we just keep trying until we hit an arbitrary size and then stop.
+    } while (result < 0 && (buffer.size() * 2) < 2048);
+    if (result == 0)
+        return String("");
+    else if (result < 0)
+        return String();
+    unsigned len = result;
+#else
+    // Do the format once to get the length.
+    char ch;
+    int result = vsnprintf(&ch, 1, format, args);
+    if (result == 0)
+        return String("");
+    else if (result < 0)
+        return String();
+    unsigned len = result;
+    buffer.resize(len + 1);
+    
+    // Now do the formatting again, guaranteed to fit.
+    vsnprintf(buffer.data(), buffer.size(), format, args);
+#endif
+    va_end(args);
+    
+    return new StringImpl(buffer.data(), len);
+}
+
+String String::number(int n)
+{
+    return String::sprintf("%d", n);
+}
+
+String String::number(unsigned n)
+{
+    return String::sprintf("%u", n);
+}
+
+String String::number(long n)
+{
+    return String::sprintf("%ld", n);
+}
+
+String String::number(unsigned long n)
+{
+    return String::sprintf("%lu", n);
+}
+
+String String::number(double n)
+{
+    return String::sprintf("%.6lg", n);
+}
+
 int String::toInt(bool* ok) const
 {
     if (!m_impl) {
index 1a6748432b6dd84025d90f1e7a749a2aca5b46dd..0b0f824ca0aeb856a47268236c4250ab704b047d 100644 (file)
@@ -71,12 +71,12 @@ StringImpl::StringImpl(const char* str)
     initWithChar(str, strlen(str));
 }
 
-StringImpl::StringImpl(const char* str, unsigned int len)
+StringImpl::StringImpl(const char* str, unsigned len)
 {
     initWithChar(str, len);
 }
 
-void StringImpl::initWithChar(const char* str, unsigned int len)
+void StringImpl::initWithChar(const char* str, unsigned len)
 {
     m_hash = 0;
     m_inTable = false;
@@ -93,7 +93,7 @@ void StringImpl::initWithChar(const char* str, unsigned int len)
         *ptr++ = *str++;
 }
 
-void StringImpl::initWithQChar(const QChar* str, unsigned int len)
+void StringImpl::initWithQChar(const QChar* str, unsigned len)
 {
     m_hash = 0;
     m_inTable = false;
@@ -199,7 +199,7 @@ bool StringImpl::containsOnlyWhitespace() const
     return containsOnlyWhitespace(0, m_length);
 }
 
-bool StringImpl::containsOnlyWhitespace(unsigned int from, unsigned int len) const
+bool StringImpl::containsOnlyWhitespace(unsigned from, unsigned len) const
 {
     if (!m_data)
         return true;
index 72e61358b3957137ff358ff514886d5934829826..df1b1c317567685d9a3e6ef3d5136eec3e8dadad 100644 (file)
@@ -318,7 +318,7 @@ static String keyIdentifierForKeyEvent(NSEvent* event)
             // FIXME: We should use something other than the vendor-area Unicode values for the above keys.
             // For now, just fall through to the default.
         default:
-            return DeprecatedString().sprintf("U+%06X", toupper(c));
+            return String::sprintf("U+%06X", toupper(c));
     }
 }
 
index 288358a1ea6dfce94e4c58ddc288f2ac889dc15f..b4f758e69d0d7c1ccac2a5f4f0ad3b813263b37c 100644 (file)
@@ -1116,7 +1116,7 @@ void RenderSlider::updateFromElement()
     if (!equalIgnoringCase(precision, "float"))
         val = (int)(val + 0.5);
 
-    element()->setValue(DeprecatedString::number(val));
+    element()->setValue(String::number(val));
 
     QSlider* slider = (QSlider*)widget();
      
@@ -1138,7 +1138,7 @@ void RenderSlider::valueChanged(Widget*)
     if (!equalIgnoringCase(precision, "float"))
         val = (int)(val + 0.5);
 
-    element()->setValue(DeprecatedString::number(val));
+    element()->setValue(String::number(val));
     
     // Fire the "input" DOM event.
     element()->dispatchHTMLEvent(inputEvent, true, false);