From 6f34f01f3da9b8f25c63c9183bcb03e2d782bcd9 Mon Sep 17 00:00:00 2001
From: "bzbarsky%mit.edu"
 <bzbarsky%mit.edu@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Wed, 6 Nov 2002 02:00:50 +0000
Subject: [PATCH] nsCOMArray_base copy constructor should not assert.  Bug
 178534, r=dougt, sr=alecf

git-svn-id: svn://10.0.0.236/trunk@133068 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/xpcom/ds/nsCOMArray.cpp | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/mozilla/xpcom/ds/nsCOMArray.cpp b/mozilla/xpcom/ds/nsCOMArray.cpp
index 9c28266c36c..bb80e7061f9 100644
--- a/mozilla/xpcom/ds/nsCOMArray.cpp
+++ b/mozilla/xpcom/ds/nsCOMArray.cpp
@@ -47,15 +47,9 @@ PR_STATIC_CALLBACK(PRBool) ReleaseObjects(void* aElement, void*);
 // object gets another AddRef()
 nsCOMArray_base::nsCOMArray_base(const nsCOMArray_base& aOther)
 {
-    PRInt32 count = aOther.Count();
     // make sure we do only one allocation
-    mArray.SizeTo(count);
-    
-    PRInt32 i;
-    for (i=0; i<count; i++) {
-        // ReplaceObjectAt will handle existing null entries for us
-        ReplaceObjectAt(aOther[i], i);
-    }
+    mArray.SizeTo(aOther.Count());
+    AppendObjects(aOther);
 }
 
 PRBool