Reviewed by Adele.
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 16 May 2005 19:19:38 +0000 (19:19 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 16 May 2005 19:19:38 +0000 (19:19 +0000)
        - fixed issues preventing us from compiling with newer versions of gcc 4.0

        * khtml/dom/dom2_events.cpp: Add definitions of some static data member constants, as required
        by the C++ standard and the gcc 4.0 compiler.

        * khtml/editing/selection.h: Specified KHTMLPart friend class as ::KHTMLPart, since if it's
        not explicitly qualified, it means DOM::KHTMLPart.

        * khtml/dom/dom_string.cpp:
        (DOM::strcasecmp): Removed redundant and illegal DOM:: prefix.
        (DOM::operator==): Ditto.
        * khtml/ecma/kjs_binding.cpp:
        (KJS::getStringOrNull): Removed redundant and illegal KJS:: prefix.
        (KJS::ValueToVariant): Ditto.
        * khtml/ecma/kjs_css.cpp:
        (KJS::getCSSRuleConstructor): Ditto.
        (KJS::getCSSValueConstructor): Ditto.
        (KJS::getCSSPrimitiveValueConstructor): Ditto.
        * khtml/ecma/kjs_events.cpp:
        (KJS::getEventConstructor): Ditto.
        (KJS::getEventExceptionConstructor): Ditto.
        (KJS::getMutationEventConstructor): Ditto.
        * khtml/ecma/kjs_traversal.cpp:
        (KJS::getNodeFilterConstructor): Ditto.

        * khtml/misc/loader_client.h: Added an empty virtual destructor to CachedObjectClient
        to quiet the compiler. This doesn't really do any good, but also does no harm.
        * khtml/misc/loader.cpp: (CachedObjectClient::~CachedObjectClient): Added.

        * khtml/rendering/render_block.cpp: (khtml::RenderBlock::fillBlockSelectionGaps):
        Initialize a couple of variables that should have been initialized to 0.
        Not just about making the compiler happy -- warning found a real bug!

        * kwq/KWQTextArea.mm: (RangeOfParagraph): Change else structure to work around compiler
        warning bug.

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

14 files changed:
WebCore/ChangeLog-2005-08-23
WebCore/khtml/dom/dom2_events.cpp
WebCore/khtml/dom/dom_string.cpp
WebCore/khtml/ecma/kjs_binding.cpp
WebCore/khtml/ecma/kjs_css.cpp
WebCore/khtml/ecma/kjs_events.cpp
WebCore/khtml/ecma/kjs_traversal.cpp
WebCore/khtml/editing/SelectionController.h
WebCore/khtml/editing/selection.h
WebCore/khtml/misc/loader.cpp
WebCore/khtml/misc/loader_client.h
WebCore/khtml/rendering/render_block.cpp
WebCore/khtml/xml/dom_nodeimpl.cpp
WebCore/kwq/KWQTextArea.mm

index a885c686b01b4806b8afbc3a9d13f038b9bf118f..a2a0528bcebdf52b7a655180ba4d364a03197039 100644 (file)
@@ -1,3 +1,43 @@
+2005-05-16  Darin Adler  <darin@apple.com>
+
+        Reviewed by Adele.
+
+        - fixed issues preventing us from compiling with newer versions of gcc 4.0
+
+        * khtml/dom/dom2_events.cpp: Add definitions of some static data member constants, as required
+        by the C++ standard and the gcc 4.0 compiler.
+
+        * khtml/editing/selection.h: Specified KHTMLPart friend class as ::KHTMLPart, since if it's
+        not explicitly qualified, it means DOM::KHTMLPart.
+
+        * khtml/dom/dom_string.cpp:
+        (DOM::strcasecmp): Removed redundant and illegal DOM:: prefix.
+        (DOM::operator==): Ditto.
+        * khtml/ecma/kjs_binding.cpp:
+        (KJS::getStringOrNull): Removed redundant and illegal KJS:: prefix.
+        (KJS::ValueToVariant): Ditto.
+        * khtml/ecma/kjs_css.cpp:
+        (KJS::getCSSRuleConstructor): Ditto.
+        (KJS::getCSSValueConstructor): Ditto.
+        (KJS::getCSSPrimitiveValueConstructor): Ditto.
+        * khtml/ecma/kjs_events.cpp:
+        (KJS::getEventConstructor): Ditto.
+        (KJS::getEventExceptionConstructor): Ditto.
+        (KJS::getMutationEventConstructor): Ditto.
+        * khtml/ecma/kjs_traversal.cpp:
+        (KJS::getNodeFilterConstructor): Ditto.
+
+        * khtml/misc/loader_client.h: Added an empty virtual destructor to CachedObjectClient
+        to quiet the compiler. This doesn't really do any good, but also does no harm.
+        * khtml/misc/loader.cpp: (CachedObjectClient::~CachedObjectClient): Added.
+
+        * khtml/rendering/render_block.cpp: (khtml::RenderBlock::fillBlockSelectionGaps):
+        Initialize a couple of variables that should have been initialized to 0.
+        Not just about making the compiler happy -- warning found a real bug!
+
+        * kwq/KWQTextArea.mm: (RangeOfParagraph): Change else structure to work around compiler
+        warning bug.
+
 2005-05-13  John Sullivan  <sullivan@apple.com>
 
         * kwq/KWQTextArea.mm:
index 120d78dd435857a19d95060bafd61aab404fd88e..4e5e5fb0c483f511daa89c01471d2e7dd8ed7123 100644 (file)
@@ -642,6 +642,12 @@ void MutationEvent::initMutationEvent(const DOMString &typeArg,
 
 // -----------------------------------------------------------------------------
 
+const unsigned long KeyboardEvent::DOM_KEY_LOCATION_STANDARD;
+const unsigned long KeyboardEvent::DOM_KEY_LOCATION_LEFT;
+const unsigned long KeyboardEvent::DOM_KEY_LOCATION_RIGHT;
+const unsigned long KeyboardEvent::DOM_KEY_LOCATION_NUMPAD;
+const unsigned long KeyboardEvent::DOM_KEY_LOCATION_UNKNOWN;
+
 KeyboardEvent::KeyboardEvent()
 {
 }
index 44457aaffc081fbdb194f356feb25099f8cf350f..d4faf92be5d8a40d6d9133b304737a9545f2d0f2 100644 (file)
@@ -236,7 +236,7 @@ DOMString DOMString::copy() const
 
 // ------------------------------------------------------------------------
 
-bool DOM::strcasecmp( const DOMString &as, const DOMString &bs )
+bool strcasecmp( const DOMString &as, const DOMString &bs )
 {
     if ( as.length() != bs.length() ) return true;
 
@@ -252,7 +252,7 @@ bool DOM::strcasecmp( const DOMString &as, const DOMString &bs )
     return false;
 }
 
-bool DOM::strcasecmp( const DOMString &as, const char* bs )
+bool strcasecmp( const DOMString &as, const char* bs )
 {
     const QChar *a = as.unicode();
     int l = as.length();
@@ -286,7 +286,7 @@ const char *DOMString::ascii() const
 
 //-----------------------------------------------------------------------------
 
-bool DOM::operator==( const DOMString &a, const DOMString &b )
+bool operator==( const DOMString &a, const DOMString &b )
 {
     if (a.impl == b.impl)
         return true;
@@ -300,7 +300,7 @@ bool DOM::operator==( const DOMString &a, const DOMString &b )
     return false;
 }
 
-bool DOM::operator==( const DOMString &a, const QString &b )
+bool operator==( const DOMString &a, const QString &b )
 {
     unsigned int l = a.length();
 
@@ -311,7 +311,7 @@ bool DOM::operator==( const DOMString &a, const QString &b )
     return false;
 }
 
-bool DOM::operator==( const DOMString &a, const char *b )
+bool operator==( const DOMString &a, const char *b )
 {
     DOMStringImpl *aimpl = a.impl;
     
index e27a5dc7835f2e2ba2dbd505391b64d86a99b86f..666a2ab302228379d1bb4e42e2c22e141a5104c1 100644 (file)
@@ -380,7 +380,7 @@ QString Identifier::qstring() const
   return QString((QChar*) data(), size());
 }
 
-Value KJS::getStringOrNull(DOMString s)
+Value getStringOrNull(DOMString s)
 {
   if (s.isNull())
     return Null();
@@ -388,7 +388,7 @@ Value KJS::getStringOrNull(DOMString s)
     return String(s);
 }
 
-QVariant KJS::ValueToVariant(ExecState* exec, const Value &val) {
+QVariant ValueToVariant(ExecState* exec, const Value &val) {
   QVariant res;
   switch (val.type()) {
   case BooleanType:
index 56e8daddce719a301e7d34182a4d87a59fee4a56..65a828ccdf564839793c52057e9bd0883ecfcf2e 100644 (file)
@@ -882,7 +882,7 @@ Value CSSRuleConstructor::getValueProperty(ExecState *, int token) const
   return Value();
 }
 
-Value KJS::getCSSRuleConstructor(ExecState *exec)
+Value getCSSRuleConstructor(ExecState *exec)
 {
   return cacheGlobalObject<CSSRuleConstructor>( exec, "[[cssRule.constructor]]" );
 }
@@ -972,7 +972,7 @@ Value CSSValueConstructor::getValueProperty(ExecState *, int token) const
   return Value();
 }
 
-Value KJS::getCSSValueConstructor(ExecState *exec)
+Value getCSSValueConstructor(ExecState *exec)
 {
   return cacheGlobalObject<CSSValueConstructor>( exec, "[[cssValue.constructor]]" );
 }
@@ -1088,7 +1088,7 @@ Value CSSPrimitiveValueConstructor::getValueProperty(ExecState *, int token) con
   return Number(token);
 }
 
-Value KJS::getCSSPrimitiveValueConstructor(ExecState *exec)
+Value getCSSPrimitiveValueConstructor(ExecState *exec)
 {
   return cacheGlobalObject<CSSPrimitiveValueConstructor>( exec, "[[cssPrimitiveValue.constructor]]" );
 }
index 689f67e75b863daf52e4902920916b0a969e3b3c..80af9d3550ed281e6d85265555cf2edbc7053562 100644 (file)
@@ -362,7 +362,7 @@ Value EventConstructor::getValueProperty(ExecState *, int token) const
   return Number(token);
 }
 
-Value KJS::getEventConstructor(ExecState *exec)
+Value getEventConstructor(ExecState *exec)
 {
   return cacheGlobalObject<EventConstructor>(exec, "[[event.constructor]]");
 }
@@ -576,7 +576,7 @@ Value EventExceptionConstructor::getValueProperty(ExecState *, int token) const
   return Number(token);
 }
 
-Value KJS::getEventExceptionConstructor(ExecState *exec)
+Value getEventExceptionConstructor(ExecState *exec)
 {
   return cacheGlobalObject<EventExceptionConstructor>(exec, "[[eventException.constructor]]");
 }
@@ -935,7 +935,7 @@ Value MutationEventConstructor::getValueProperty(ExecState *, int token) const
   return Number(token);
 }
 
-Value KJS::getMutationEventConstructor(ExecState *exec)
+Value getMutationEventConstructor(ExecState *exec)
 {
   return cacheGlobalObject<MutationEventConstructor>(exec, "[[mutationEvent.constructor]]");
 }
index 56cdb77e4e58f6219517ddb7d0abe9a942a130de..27d82124479c2069ea850410d477d7d2353c5e78 100644 (file)
@@ -156,7 +156,7 @@ Value NodeFilterConstructor::getValueProperty(ExecState *, int token) const
   return Number(token);
 }
 
-Value KJS::getNodeFilterConstructor(ExecState *exec)
+Value getNodeFilterConstructor(ExecState *exec)
 {
   return cacheGlobalObject<NodeFilterConstructor>(exec, "[[nodeFilter.constructor]]");
 }
index 72c5e764f963b8264831b6ce0d246ad3c917327c..e4cd164436d5fad13d42eda536d37d3ad8a2151a 100644 (file)
@@ -115,7 +115,7 @@ public:
     void debugPosition() const;
     void debugRenderer(khtml::RenderObject *r, bool selected) const;
 
-    friend class KHTMLPart;
+    friend class ::KHTMLPart;
 
 #ifndef NDEBUG
     void formatForDebugger(char *buffer, unsigned length) const;
index 72c5e764f963b8264831b6ce0d246ad3c917327c..e4cd164436d5fad13d42eda536d37d3ad8a2151a 100644 (file)
@@ -115,7 +115,7 @@ public:
     void debugPosition() const;
     void debugRenderer(khtml::RenderObject *r, bool selected) const;
 
-    friend class KHTMLPart;
+    friend class ::KHTMLPart;
 
 #ifndef NDEBUG
     void formatForDebugger(char *buffer, unsigned length) const;
index 0a6cbf1569a9bae6f7818ac5c80fabaa01fdd548..552f87c72def7112b037dee350724cc942bfaa12 100644 (file)
@@ -2539,6 +2539,7 @@ CachedObjectClient *CachedObjectClientWalker::next()
 
 // --------------------------------------
 
+CachedObjectClient::~CachedObjectClient() { }
 void CachedObjectClient::setPixmap(const QPixmap &, const QRect&, CachedImage *) {}
 void CachedObjectClient::setStyleSheet(const DOM::DOMString &/*url*/, const DOM::DOMString &/*sheet*/) {}
 #ifndef KHTML_NO_XBL
index 400c63af3d3d2733b7670d76de4fc43364aa30f2..ff274eb3c50940940db92dae80ed26e8983a5242 100644 (file)
@@ -25,6 +25,8 @@ namespace khtml {
     class CachedObjectClient
     {
     public:
+        virtual ~CachedObjectClient();
+
         // clipped pixmap (if it is not yet completely loaded,
         // size of the complete (finished loading) pixmap
         // rectangle of the part that has been loaded very recently
index 8784098b94c7b0c1653091708adb8cae57ca6626..f8596ef9d3fd9c5be8e5d8cf28ec7a254f605520 100644 (file)
@@ -1543,7 +1543,8 @@ GapRects RenderBlock::fillBlockSelectionGaps(RenderBlock* rootBlock, int blockX,
         if (curr->isRelPositioned() && curr->layer()) {
             // If the relposition offset is anything other than 0, then treat this just like an absolute positioned element.
             // Just disregard it completely.
-            int x, y;
+            int x = 0;
+            int y = 0;
             curr->layer()->relativePositionOffset(x, y);
             if (x || y)
                 continue;
index 76cb7a4090ed389d07a69064ced88b0b9f99917e..ca25db0633d7acb1c8abde24e1d4abf12443e981 100644 (file)
@@ -2041,6 +2041,7 @@ bool ContainerNodeImpl::checkSameDocument( NodeImpl *newChild, int &exceptioncod
 {
     exceptioncode = 0;
     DocumentImpl *ownerDocThis = getDocument();
+    // FIXME: Doh! This next line isn't getting newChild, so it's never going to work!
     DocumentImpl *ownerDocNew = getDocument();
     if(ownerDocThis != ownerDocNew) {
         kdDebug(6010)<< "not same document, newChild = " << newChild << "document = " << getDocument() << endl;
index 564b594f6e61b143f3788fd4fbe8b036e33dabf5..c582bb150382e432df970f031b38ac62562784d3 100644 (file)
@@ -360,12 +360,12 @@ static NSRange RangeOfParagraph(NSString *text, int paragraph)
     }
 
     if (paragraphSoFar < paragraph) {
-       return NSMakeRange(NSNotFound, 0);
-    } else if (searchRange.location == NSNotFound || newlineRange.location == NSNotFound) {
-       return searchRange;
-    } else {
-       return NSMakeRange(searchRange.location, newlineRange.location - searchRange.location);
+        return NSMakeRange(NSNotFound, 0);
+    }
+    if (searchRange.location == NSNotFound || newlineRange.location == NSNotFound) {
+        return searchRange;
     }
+    return NSMakeRange(searchRange.location, newlineRange.location - searchRange.location);
 }
 
 - (void)setCursorPositionToIndex:(int)index inParagraph:(int)paragraph