Fix slot warnings when no partManager is available

This resolves the remainder of Bug 1088
(cherry picked from commit ecaf622512)

khtml/ecma/kjs_dom.cpp

@@ -609,17 +609,23 @@ Value DOMNodeProtoFunc::tryCall(ExecState *exec, Object &thisObj, const List &ar
 
       DOMString where = args[0].toString(exec).string();
 
-      if (where == "beforeBegin" || where == "BeforeBegin")
+      if (where == "beforeBegin" || where == "BeforeBegin") {
         node.parentNode().insertBefore(docFrag, node);
-      else if (where == "afterBegin" || where == "AfterBegin")
+      }
+      else if (where == "afterBegin" || where == "AfterBegin") {
         node.insertBefore(docFrag, node.firstChild());
-      else if (where == "beforeEnd" || where == "BeforeEnd")
+      }
+      else if (where == "beforeEnd" || where == "BeforeEnd") {
         return getDOMNode(exec, node.appendChild(docFrag));
-      else if (where == "afterEnd" || where == "AfterEnd")
+      }
-        if (!node.nextSibling().isNull())
+      else if (where == "afterEnd" || where == "AfterEnd") {
+        if (!node.nextSibling().isNull()) {
 	  node.parentNode().insertBefore(docFrag, node.nextSibling());
-	else
+	}
+	else {
 	  node.parentNode().appendChild(docFrag);
+	}
+      }
 
       return Undefined();
     }

khtml/khtml_part.cpp

@@ -1429,8 +1429,10 @@ void KHTMLPart::clear()
   d->m_decoder = 0;
 
   // We don't want to change between parts if we are going to delete all of them anyway
+  if (partManager()) {
     disconnect( partManager(), TQT_SIGNAL( activePartChanged( KParts::Part * ) ),
                this, TQT_SLOT( slotActiveFrameChanged( KParts::Part * ) ) );
+  }
 
   if (d->m_frames.count())
   {
@@ -1462,8 +1464,10 @@ void KHTMLPart::clear()
   }
 
   // Listen to part changes again
+  if (partManager()) {
     connect( partManager(), TQT_SIGNAL( activePartChanged( KParts::Part * ) ),
              this, TQT_SLOT( slotActiveFrameChanged( KParts::Part * ) ) );
+  }
 
   d->m_delayRedirect = 0;
   d->m_redirectURL = TQString();
@@ -5593,16 +5597,18 @@ void KHTMLPart::restoreState( TQDataStream &stream )
 
       if ( (*childFrame)->m_part )
       {
-        if ( (*childFrame)->m_extension )
+        if ( (*childFrame)->m_extension ) {
           if ( (*childFrame)->m_extension && !(*fBufferIt).isEmpty() )
           {
             TQDataStream frameStream( *fBufferIt, IO_ReadOnly );
             (*childFrame)->m_extension->restoreState( frameStream );
           }
-        else
+          else {
             (*childFrame)->m_part->openURL( *fURLIt );
           }
         }
+      }
+    }
 
     KParts::URLArgs args( d->m_extension->urlArgs() );
     args.xOffset = xOffset;

khtml/xml/dom_elementimpl.h

@@ -357,11 +357,11 @@ inline bool checkQualifiedName(const DOMString &qualifiedName, const DOMString &
         }
     }
 
-    if (!qualifiedName.isNull() && Element::khtmlMalformedQualifiedName(qualifiedName) ||
+    if (((!qualifiedName.isNull()) && Element::khtmlMalformedQualifiedName(qualifiedName)) ||
         (colonpos >= 0 && namespaceURI.isNull()) ||
         (qualifiedName.isNull() && !namespaceURI.isNull()) ||
-        (colonpos == 3 && qualifiedName[0] == 'x' && qualifiedName[1] == 'm' && qualifiedName[2] == 'l' &&
+        ((colonpos == 3) && (qualifiedName[0] == 'x') && (qualifiedName[1] == 'm') && (qualifiedName[2] == 'l') &&
-         namespaceURI != "http://www.w3.org/XML/1998/namespace")) {
+         (namespaceURI != "http://www.w3.org/XML/1998/namespace"))) {
         if (pExceptioncode)
             *pExceptioncode = DOMException::NAMESPACE_ERR;
         return false;