Bug 472348 - Some unused variables found by combing the warning database, rs=bz

content/xml/document/src/nsXMLFragmentContentSink.cpp

@@ -663,7 +663,6 @@ nsXHTMLParanoidFragmentSink::HandleStartElement(const PRUnichar *aName,
                                                 PRInt32 aIndex,
                                                 PRUint32 aLineNumber)
 {
-  nsresult rv;
   PRInt32 nameSpaceID;
   nsCOMPtr<nsIAtom> prefix, localName;
   nsContentUtils::SplitExpatName(aName, getter_AddRefs(prefix),

editor/libeditor/base/DeleteElementTxn.cpp

@@ -113,7 +113,7 @@ NS_IMETHODIMP DeleteElementTxn::DoTransaction(void)
 #endif
 
   // remember which child mElement was (by remembering which child was next)
-  nsresult result = mElement->GetNextSibling(getter_AddRefs(mRefNode));  // can return null mRefNode
+  mElement->GetNextSibling(getter_AddRefs(mRefNode));  // can return null mRefNode
 
   // give range updater a chance.  SelAdjDeleteNode() needs to be called *before*
   // we do the action, unlike some of the other nsRangeStore update methods.

editor/libeditor/base/nsEditor.cpp

@@ -938,7 +938,7 @@ nsEditor::EndPlaceHolderTransaction()
   if (mPlaceHolderBatch == 1)
   {
     nsCOMPtr<nsISelection>selection;
-    nsresult rv = GetSelection(getter_AddRefs(selection));
+    GetSelection(getter_AddRefs(selection));
 
     nsCOMPtr<nsISelectionPrivate>selPrivate(do_QueryInterface(selection));
 

editor/libeditor/html/nsHTMLAnonymousUtils.cpp

@@ -253,7 +253,7 @@ nsHTMLEditor::DeleteRefToAnonymousNode(nsIDOMElement* aElement,
           // Call BeginUpdate() so that the nsCSSFrameConstructor/PresShell
           // knows we're messing with the frame tree.
           nsCOMPtr<nsIDOMDocument> domDocument;
-          nsresult res = GetDocument(getter_AddRefs(domDocument));
+          GetDocument(getter_AddRefs(domDocument));
           nsCOMPtr<nsIDocument> document = do_QueryInterface(domDocument);
           if (document)
             docObserver->BeginUpdate(document, UPDATE_CONTENT_MODEL);

editor/libeditor/html/nsWSRunObject.cpp

@@ -1710,7 +1710,6 @@ nsWSRunObject::GetCharBefore(WSPoint &aPoint, WSPoint *outPoint)
   outPoint->mOffset = 0;
   outPoint->mChar = 0;
   
-  nsresult res = NS_OK;
   nsCOMPtr<nsIDOMNode> pointTextNode(do_QueryInterface(aPoint.mTextNode));
   PRInt32 idx = mNodeArray.IndexOf(pointTextNode);
   if (idx == -1) return NS_OK;  // can't find point, but it's not an error

intl/uconv/tests/TestUConv.cpp

@@ -218,8 +218,6 @@ nsresult nsTestUConv::TestEncoders()
   res = ccMan->GetEncoderList(getter_AddRefs(encoders));
   if (NS_FAILED(res)) return res;
 
-  PRUint32 encoderCount=0;
-
   PRBool hasMore;
   encoders->HasMore(&hasMore);
   

layout/generic/nsFrame.cpp

@@ -5346,7 +5346,6 @@ nsFrame::GetLineNumber(nsIFrame *aFrame, PRBool aLockScroll, nsIFrame** aContain
   nsFrameManager* frameManager = aFrame->PresContext()->FrameManager();
   nsIFrame *blockFrame = aFrame;
   nsIFrame *thisBlock;
-  PRInt32   thisLine;
   nsAutoLineIterator it;
   nsresult result = NS_ERROR_FAILURE;
   while (NS_FAILED(result) && blockFrame)

rdf/base/src/nsRDFXMLDataSource.cpp

@@ -501,7 +501,6 @@ RDFXMLDataSourceImpl::BlockingParse(nsIURI* aURL, nsIStreamListener* aConsumer)
     rv = NS_NewChannel(getter_AddRefs(channel), aURL, nsnull);
     if (NS_FAILED(rv)) return rv;
     nsCOMPtr<nsIInputStream> in;
-    PRUint32 sourceOffset = 0;
     rv = channel->Open(getter_AddRefs(in));
 
     // Report success if the file doesn't exist, but propagate other errors.

xpcom/tests/TestCRT.cpp

@@ -63,8 +63,6 @@ static void Check(const char* s1, const char* s2, PRIntn n)
 {
   PRIntn clib = PL_strcmp(s1, s2);
   PRIntn clib_n = PL_strncmp(s1, s2, n);
-  PRIntn clib_case = PL_strcasecmp(s1, s2);
-  PRIntn clib_case_n = PL_strncasecmp(s1, s2, n);
 
   nsAutoString t1,t2; 
   t1.AssignWithConversion(s1);