continuing work in progress on nsCOMArray<T> - take darin's suggestion and move all nsISupports-related code into a non-template base class nsCOMArray_base. This allows the generated template to be paper thin (or really, non-existent!)

Then, we move some of the non-trivial methods of this class into a .cpp file
This commit is contained in:
alecf%netscape.com 2005-11-02 16:04:34 +00:00
Родитель c7d124ba58
Коммит 6f3992e110
2 изменённых файлов: 175 добавлений и 54 удалений

95
xpcom/glue/nsCOMArray.cpp Normal file
Просмотреть файл

@ -0,0 +1,95 @@
/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
/* ***** BEGIN LICENSE BLOCK *****
* Version: NPL 1.1/GPL 2.0/LGPL 2.1
*
* The contents of this file are subject to the Netscape Public License
* Version 1.1 (the "License"); you may not use this file except in
* compliance with the License. You may obtain a copy of the License at
* http://www.mozilla.org/NPL/
*
* Software distributed under the License is distributed on an "AS IS" basis,
* WITHOUT WARRANTY OF ANY KIND, either express or implied. See the License
* for the specific language governing rights and limitations under the
* License.
*
* The Original Code is mozilla.org code.
*
* The Initial Developer of the Original Code is
* Netscape Communications Corporation.
* Portions created by the Initial Developer are Copyright (C) 1998
* the Initial Developer. All Rights Reserved.
*
* Contributor(s):
*
* Alternatively, the contents of this file may be used under the terms of
* either the GNU General Public License Version 2 or later (the "GPL"), or
* the GNU Lesser General Public License Version 2.1 or later (the "LGPL"),
* in which case the provisions of the GPL or the LGPL are applicable instead
* of those above. If you wish to allow use of your version of this file only
* under the terms of either the GPL or the LGPL, and not to allow others to
* use your version of this file under the terms of the NPL, indicate your
* decision by deleting the provisions above and replace them with the notice
* and other provisions required by the GPL or the LGPL. If you do not delete
* the provisions above, a recipient may use your version of this file under
* the terms of any one of the NPL, the GPL or the LGPL.
*
* ***** END LICENSE BLOCK ***** */
#include "nsCOMArray.h"
// implementations of non-trivial methods in nsCOMArray_base
PRBool
nsCOMArray_base::InsertObjectAt(nsISupports* aObject, PRInt32 aIndex) {
PRBool result = mArray.InsertElementAt(aObject, aIndex);
if (result)
NS_ADDREF(aObject);
return result;
}
PRBool
nsCOMArray_base::ReplaceObjectAt(nsISupports* aObject, PRInt32 aIndex) {
nsISupports *oldObject = ObjectAt(aIndex);
NS_IF_RELEASE(oldObject);
PRBool result = mArray.ReplaceElementAt(aObject, aIndex);
if (result)
NS_ADDREF(aObject);
return result;
}
PRBool
nsCOMArray_base::AppendObject(nsISupports *aObject) {
PRBool result = InsertObjectAt(aObject, Count());
if (result)
NS_ADDREF(aObject);
return result;
}
PRBool
nsCOMArray_base::RemoveObject(nsISupports *aObject) {
PRBool result = mArray.RemoveElement(aObject);
if (result)
NS_RELEASE(aObject);
return result;
}
PRBool
nsCOMArray_base::RemoveObjectAt(PRInt32 aIndex) {
nsISupports* element = ObjectAt(aIndex);
if (element) {
PRBool result = mArray.RemoveElementAt(aIndex);
if (result)
NS_RELEASE(element);
return result;
}
return PR_FALSE;
}
PRBool
nsCOMArray_base::ClearObjectsCallback(void* aElement, void*)
{
nsISupports* element = NS_STATIC_CAST(nsISupports*, aElement);
NS_RELEASE(element);
return PR_TRUE;
}

Просмотреть файл

@ -41,46 +41,25 @@
#include "nsVoidArray.h"
#include "nsISupports.h"
// a non-XPCOM, refcounting array of XPCOM objects
// used as a member variable or stack variable - this object is NOT
// refcounted, but the objects that it holds are
template <class T>
class NS_COM nsCOMArray
// a class that's nsISupports-specific, so that we can contain the
// work of this class in the XPCOM dll
class NS_COM nsCOMArray_base
{
public:
nsCOMArray() {}
nsCOMArray(PRInt32 aCount) : mArray(aCount) {}
protected:
nsCOMArray_base() {}
nsCOMArray_base(PRInt32 aCount) : mArray(aCount) {}
~nsCOMArray() {}
// these do NOT refcount on the way out, for speed
T* ObjectAt(PRInt32 aIndex) const {
return NS_STATIC_CAST(T*,mArray.ElementAt(aIndex));
nsISupports* ObjectAt(PRInt32 aIndex) const {
return NS_STATIC_CAST(nsISupports*, mArray.ElementAt(aIndex));
}
T* operator[](PRInt32 aIndex) const {
nsISupports* operator[](PRInt32 aIndex) const {
return ObjectAt(aIndex);
}
PRInt32 IndexOf(T* aObject) {
PRInt32 IndexOf(nsISupports* aObject) {
return mArray.IndexOf(aObject);
}
PRBool InsertObjectAt(T* aObject, PRInt32 aIndex) {
PRBool result = mArray.InsertElementAt(aObject, aIndex);
if (result)
NS_ADDREF(aObject);
return result;
}
PRBool ReplaceObjectAt(T* aObject, PRInt32 aIndex) {
T *oldObject = ObjectAt(aIndex);
NS_IF_RELEASE(oldObject);
PRBool result = mArray.ReplaceElementAt(aObject, aIndex);
if (result)
NS_ADDREF(aObject);
return result;
}
// override nsVoidArray stuff so that they can be accessed by
// other methods
@ -97,43 +76,90 @@ class NS_COM nsCOMArray
return mArray.EnumerateForwards(aFunc, aData);
}
// any method which is not a direct forward to mArray should
// avoid inline bodies, so that the compiler doesn't inline them
// all over the place
PRBool InsertObjectAt(nsISupports* aObject, PRInt32 aIndex);
PRBool ReplaceObjectAt(nsISupports* aObject, PRInt32 aIndex);
PRBool AppendObject(nsISupports *aObject);
PRBool RemoveObject(nsISupports *aObject);
PRBool RemoveObjectAt(PRInt32 aIndex);
private:
static PRBool ClearObjectsCallback(void *aElement, void*);
// the actual storage
nsVoidArray mArray;
nsCOMArray_base(const nsCOMArray_base& other);
nsCOMArray_base& operator=(const nsCOMArray_base& other);
};
// a non-XPCOM, refcounting array of XPCOM objects
// used as a member variable or stack variable - this object is NOT
// refcounted, but the objects that it holds are
template <class T>
class nsCOMArray : protected nsCOMArray_base
{
public:
nsCOMArray() {}
nsCOMArray(PRInt32 aCount) : nsCOMArray_base(aCount) {}
~nsCOMArray() {}
// these do NOT refcount on the way out, for speed
T* ObjectAt(PRInt32 aIndex) const {
return NS_STATIC_CAST(T*,nsCOMArray_base::ObjectAt(aIndex));
}
T* operator[](PRInt32 aIndex) const {
return ObjectAt(aIndex);
}
PRInt32 IndexOf(T* aObject) {
return nsCOMArray_base::IndexOf(aObject);
}
PRBool InsertObjectAt(T* aObject, PRInt32 aIndex) {
return nsCOMArray_base::InsertObjectAt(aObject, aIndex);
}
PRBool ReplaceObjectAt(T* aObject, PRInt32 aIndex) {
return nsCOMArray_base::ReplaceObjectAt(aObject, aIndex);
}
// override nsVoidArray stuff so that they can be accessed by
// other methods
PRInt32 Count() const {
return nsCOMArray_base::Count();
}
void Clear() {
nsCOMArray_base::Clear();
}
PRBool EnumerateForwards(nsVoidArrayEnumFunc aFunc, void* aData) {
return nsCOMArray_base::EnumerateForwards(aFunc, aData);
}
PRBool AppendObject(T *aObject) {
PRBool result = InsertObjectAt(aObject, Count());
if (result)
NS_ADDREF(aObject);
return result;
return nsCOMArray_base::AppendObject(aObject);
}
PRBool RemoveObject(T *aObject) {
PRBool result = mArray.RemoveElement(aObject);
if (result)
NS_RELEASE(aObject);
return result;
return nsCOMArray_base::RemoveObject(aObject);
}
PRBool RemoveObjectAt(PRInt32 aIndex) {
T* element = ObjectAt(aIndex);
if (element) {
PRBool result = mArray.RemoveElementAt(aIndex);
if (result)
NS_RELEASE(element);
return result;
}
return PR_FALSE;
return nsCOMArray_base::RemoveObjectAt(aIndex);
}
private:
static PRBool ClearObjectsCallback(void *aElement, void*) {
T* element = NS_STATIC_CAST(nsISupports*, aElement);
NS_RELEASE(element);
return PR_TRUE;
}
nsCOMArray(const nsCOMArray& other);
nsCOMArray& operator=(const nsCOMArray& other);
nsVoidArray mArray;
};