From 58d1bf611616d708f45af98e0ca0f8eed68eec9f Mon Sep 17 00:00:00 2001
From: "brade%netscape.com"
 <brade%netscape.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Mon, 30 Jun 2003 14:20:49 +0000
Subject: [PATCH] code cleanup; r=timeless, sr=kin, bug=209548

git-svn-id: svn://10.0.0.236/trunk@144263 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../libeditor/base/EditAggregateTxn.cpp       | 26 +++++--------------
 1 file changed, 7 insertions(+), 19 deletions(-)

diff --git a/mozilla/editor/libeditor/base/EditAggregateTxn.cpp b/mozilla/editor/libeditor/base/EditAggregateTxn.cpp
index 1feccf55535..c6b0d6e9f1b 100644
--- a/mozilla/editor/libeditor/base/EditAggregateTxn.cpp
+++ b/mozilla/editor/libeditor/base/EditAggregateTxn.cpp
@@ -38,8 +38,6 @@
 
 #include "EditAggregateTxn.h"
 #include "nsCOMPtr.h"
-#include "nsIDOMNode.h"
-#include "nsVoidArray.h"
 
 EditAggregateTxn::EditAggregateTxn()
   : EditTxn()
@@ -116,7 +114,7 @@ NS_IMETHODIMP EditAggregateTxn::RedoTransaction(void)
 
 NS_IMETHODIMP EditAggregateTxn::GetIsTransient(PRBool *aIsTransient)
 {
-  if (nsnull!=aIsTransient)
+  if (aIsTransient)
     *aIsTransient = PR_FALSE;
   return NS_OK;
 }
@@ -124,8 +122,8 @@ NS_IMETHODIMP EditAggregateTxn::GetIsTransient(PRBool *aIsTransient)
 NS_IMETHODIMP EditAggregateTxn::Merge(nsITransaction *aTransaction, PRBool *aDidMerge)
 {
   nsresult result=NS_OK;  // it's legal (but not very useful) to have an empty child list
-  if (nsnull!=aDidMerge)
-    *aDidMerge=PR_FALSE;
+  if (aDidMerge)
+    *aDidMerge = PR_FALSE;
   if (mChildren)
   {
     PRInt32 i=0;
@@ -140,7 +138,6 @@ NS_IMETHODIMP EditAggregateTxn::Merge(nsITransaction *aTransaction, PRBool *aDid
     }
   }
   return result;
-
 }
 
 NS_IMETHODIMP EditAggregateTxn::GetTxnDescription(nsAString& aString)
@@ -218,14 +215,11 @@ NS_IMETHODIMP EditAggregateTxn::SetName(nsIAtom *aName)
 
 NS_IMETHODIMP EditAggregateTxn::GetName(nsIAtom **aName)
 {
-  if (aName)
+  if (aName && mName)
   {
-    if (mName)
-    {
-      *aName = mName;
-      NS_ADDREF(*aName);
-      return NS_OK;
-    }
+    *aName = mName;
+    NS_ADDREF(*aName);
+    return NS_OK;
   }
   return NS_ERROR_NULL_POINTER;
 }
@@ -248,15 +242,9 @@ NS_IMETHODIMP EditAggregateTxn::QueryInterface(REFNSIID aIID, void** aInstancePt
  
   if (aIID.Equals(EditAggregateTxn::GetCID())) {
     *aInstancePtr = NS_STATIC_CAST(EditAggregateTxn*, this);
-//    *aInstancePtr = (nsISupports*)(EditAggregateTxn*)(this);
     NS_ADDREF_THIS();
     return NS_OK;
   }
   return EditTxn::QueryInterface(aIID, aInstancePtr);
 }
 
-
-
-
-
-