From d188ae1ca0a160970820549076c54fd808bb038c Mon Sep 17 00:00:00 2001
From: "brade%netscape.com" <brade%netscape.com>
Date: Mon, 30 Jun 2003 14:57:59 +0000
Subject: [PATCH] code cleanup; r=timeless, sr=kin, bug=209548

---
 editor/libeditor/base/SetDocTitleTxn.cpp     | 19 ++++++++-----------
 editor/libeditor/base/SplitElementTxn.cpp    |  7 +++----
 editor/libeditor/base/TransactionFactory.cpp |  3 +--
 editor/libeditor/base/nsEditorController.cpp |  2 --
 4 files changed, 12 insertions(+), 19 deletions(-)

diff --git a/editor/libeditor/base/SetDocTitleTxn.cpp b/editor/libeditor/base/SetDocTitleTxn.cpp
index 1db8b0eac23f..17d8141162e9 100644
--- a/editor/libeditor/base/SetDocTitleTxn.cpp
+++ b/editor/libeditor/base/SetDocTitleTxn.cpp
@@ -38,7 +38,6 @@
  * ***** END LICENSE BLOCK ***** */
 
 #include "SetDocTitleTxn.h"
-#include "nsEditor.h"
 #include "nsIDOMNode.h"
 #include "nsIDOMNodeList.h"
 #include "nsIDOMDocument.h"
@@ -93,12 +92,11 @@ nsresult SetDocTitleTxn::SetDocTitle(const nsAString& aTitle)
   NS_ASSERTION(mEditor, "bad state");
   if (!mEditor) return NS_ERROR_NOT_INITIALIZED;
 
-  nsCOMPtr<nsIDOMDocument>  domDoc;
   nsCOMPtr<nsIEditor> editor = do_QueryInterface(mEditor);
   if (!editor) return NS_ERROR_FAILURE;
+  nsCOMPtr<nsIDOMDocument> domDoc;
   nsresult rv = editor->GetDocument(getter_AddRefs(domDoc));
   if (NS_FAILED(rv)) return rv;
-  if (!domDoc) return NS_ERROR_FAILURE;
   nsCOMPtr<nsIDOMHTMLDocument> HTMLDoc = do_QueryInterface(domDoc);
   if (!HTMLDoc) return NS_ERROR_FAILURE;
 
@@ -107,17 +105,13 @@ nsresult SetDocTitleTxn::SetDocTitle(const nsAString& aTitle)
 
 nsresult SetDocTitleTxn::SetDomTitle(const nsAString& aTitle)
 {
-  nsCOMPtr<nsIDOMDocument>  domDoc;
   nsCOMPtr<nsIEditor> editor = do_QueryInterface(mEditor);
   if (!editor) return NS_ERROR_FAILURE;
+  nsCOMPtr<nsIDOMDocument> domDoc;
   nsresult res = editor->GetDocument(getter_AddRefs(domDoc));
-  
   if (!domDoc) return NS_ERROR_FAILURE;
 
   nsCOMPtr<nsIDOMNodeList> titleList;
-  nsCOMPtr<nsIDOMNode>titleNode;
-  nsCOMPtr<nsIDOMNode>headNode;
-  nsCOMPtr<nsIDOMNode> resultNode;
   res = domDoc->GetElementsByTagName(NS_LITERAL_STRING("title"), getter_AddRefs(titleList));
   if (NS_FAILED(res)) return res;
 
@@ -125,6 +119,7 @@ nsresult SetDocTitleTxn::SetDomTitle(const nsAString& aTitle)
   // (transaction will not be pushed on stack)
   mIsTransient = PR_TRUE;
 
+  nsCOMPtr<nsIDOMNode>titleNode;
   if(titleList)
   {
     res = titleList->Item(0, getter_AddRefs(titleNode));
@@ -164,6 +159,7 @@ nsresult SetDocTitleTxn::SetDomTitle(const nsAString& aTitle)
   if (NS_FAILED(res)) return res;
   if (!headList) return NS_ERROR_FAILURE;
   
+  nsCOMPtr<nsIDOMNode>headNode;
   headList->Item(0, getter_AddRefs(headNode));
   if (!headNode) return NS_ERROR_FAILURE;
 
@@ -197,13 +193,13 @@ nsresult SetDocTitleTxn::SetDomTitle(const nsAString& aTitle)
     nsCOMPtr<nsIDOMText> textNode;
     res = domDoc->CreateTextNode(aTitle, getter_AddRefs(textNode));
     if (NS_FAILED(res)) return res;
-    if (!textNode) return NS_ERROR_FAILURE;
     nsCOMPtr<nsIDOMNode> newNode = do_QueryInterface(textNode);
     if (!newNode) return NS_ERROR_FAILURE;
 
     if (newTitleNode)
     {
       // Not undoable: We will insert newTitleNode below
+      nsCOMPtr<nsIDOMNode> resultNode;
       res = titleNode->AppendChild(newNode, getter_AddRefs(resultNode));
     } 
     else 
@@ -224,8 +220,8 @@ nsresult SetDocTitleTxn::SetDomTitle(const nsAString& aTitle)
 
 NS_IMETHODIMP SetDocTitleTxn::Merge(nsITransaction *aTransaction, PRBool *aDidMerge)
 {
-  if (nsnull!=aDidMerge)
-    *aDidMerge=PR_FALSE;
+  if (aDidMerge)
+    *aDidMerge = PR_FALSE;
   return NS_OK;
 }
 
@@ -242,3 +238,4 @@ NS_IMETHODIMP SetDocTitleTxn::GetIsTransient(PRBool *aIsTransient)
   *aIsTransient = mIsTransient;
   return NS_OK;
 }
+
diff --git a/editor/libeditor/base/SplitElementTxn.cpp b/editor/libeditor/base/SplitElementTxn.cpp
index db3e31e034fc..40ab821fa85d 100644
--- a/editor/libeditor/base/SplitElementTxn.cpp
+++ b/editor/libeditor/base/SplitElementTxn.cpp
@@ -163,8 +163,7 @@ NS_IMETHODIMP SplitElementTxn::RedoTransaction(void)
   nsresult result;
   nsCOMPtr<nsIDOMNode>resultNode;
   // first, massage the existing node so it is in its post-split state
-  nsCOMPtr<nsIDOMCharacterData>rightNodeAsText;
-  rightNodeAsText = do_QueryInterface(mExistingRightNode);
+  nsCOMPtr<nsIDOMCharacterData>rightNodeAsText = do_QueryInterface(mExistingRightNode);
   if (rightNodeAsText)
   {
     result = rightNodeAsText->DeleteData(0, mOffset);
@@ -217,8 +216,8 @@ NS_IMETHODIMP SplitElementTxn::RedoTransaction(void)
 
 NS_IMETHODIMP SplitElementTxn::Merge(nsITransaction *aTransaction, PRBool *aDidMerge)
 {
-  if (nsnull!=aDidMerge)
-    *aDidMerge=PR_FALSE;
+  if (aDidMerge)
+    *aDidMerge = PR_FALSE;
   return NS_OK;
 }
 
diff --git a/editor/libeditor/base/TransactionFactory.cpp b/editor/libeditor/base/TransactionFactory.cpp
index d4e28d593f6b..6f1aa7e017b2 100644
--- a/editor/libeditor/base/TransactionFactory.cpp
+++ b/editor/libeditor/base/TransactionFactory.cpp
@@ -110,7 +110,7 @@ TransactionFactory::GetNewTransaction(REFNSIID aTxnType, EditTxn **aResult)
   else
     result = NS_ERROR_NO_INTERFACE;
   
-  if (NS_SUCCEEDED(result) && nsnull==*aResult)
+  if (NS_SUCCEEDED(result) && !*aResult)
     result = NS_ERROR_OUT_OF_MEMORY;
 
   if (NS_SUCCEEDED(result))
@@ -119,4 +119,3 @@ TransactionFactory::GetNewTransaction(REFNSIID aTxnType, EditTxn **aResult)
   return result;
 }
 
-
diff --git a/editor/libeditor/base/nsEditorController.cpp b/editor/libeditor/base/nsEditorController.cpp
index 58a37e383c1e..788ba8178d1b 100644
--- a/editor/libeditor/base/nsEditorController.cpp
+++ b/editor/libeditor/base/nsEditorController.cpp
@@ -41,7 +41,6 @@
 #include "nsString.h"
 #include "nsIComponentManager.h"
 #include "nsEditorController.h"
-#include "nsIEditor.h"
 #include "nsEditorCommands.h"
 #include "nsIControllerCommandTable.h"
 
@@ -141,4 +140,3 @@ nsresult nsEditorController::RegisterEditorCommands(nsIControllerCommandTable *i
   return NS_OK;
 }
 
-