check the item id validity in SetItem()

[wxWidgets.git] / include / wx / vector.h

diff --git a/include/wx/vector.h b/include/wx/vector.h
index 2034a9eaa8bcfaaeda7e355aa3a553d1c715d610..8ffc4a74f1389227550dc4dcc94a877438ab7c73 100644 (file)
--- a/include/wx/vector.h
+++ b/include/wx/vector.h
@@ -39,7 +39,7 @@ protected:
         m_objects = (void **) mem;
         m_capacity = sz;
         return true;
         m_objects = (void **) mem;
         m_capacity = sz;
         return true;

-    };
+    }

 
     // untyped destructor of elements - must be overriden
     virtual void Free(void *) = 0;
 
     // untyped destructor of elements - must be overriden
     virtual void Free(void *) = 0;


@@ -48,20 +48,20 @@ protected:
 
     const void *GetItem(size_type idx) const
     {
 
     const void *GetItem(size_type idx) const
     {

-        wxASSERT(idx >= 0 && idx < m_size);
+        wxASSERT(idx < m_size);

         return m_objects[idx];
         return m_objects[idx];

-    };
+    }

 
     void Append(void *obj)
     {
         wxASSERT(m_size < m_capacity);
         m_objects[m_size] = obj;
         m_size++;
 
     void Append(void *obj)
     {
         wxASSERT(m_size < m_capacity);
         m_objects[m_size] = obj;
         m_size++;

-    };
+    }

 
     void RemoveAt(size_type idx)
     {
 
     void RemoveAt(size_type idx)
     {

-        wxASSERT(idx >= 0 && idx < m_size);
+        wxASSERT(idx < m_size);

         Free(m_objects[idx]);
         if (idx < m_size - 1)
             memcpy(
         Free(m_objects[idx]);
         if (idx < m_size - 1)
             memcpy(


@@ -69,7 +69,7 @@ protected:
                 m_objects + idx + 1,
                 ( m_size - idx - 1 ) * sizeof(void*) );
         m_size--;
                 m_objects + idx + 1,
                 ( m_size - idx - 1 ) * sizeof(void*) );
         m_size--;

-    };
+    }

 
     bool copy(const wxVectorBase& vb)
     {
 
     bool copy(const wxVectorBase& vb)
     {


@@ -83,10 +83,10 @@ protected:
             if (! o)
                 return false;
             Append(o);
             if (! o)
                 return false;
             Append(o);

-        };
+        }

 
         return true;
 
         return true;

-    };
+    }

 
 public:
     wxVectorBase() : m_allocsize(16), m_size(0), m_capacity(0), m_objects(0) {}
 
 public:
     wxVectorBase() : m_allocsize(16), m_size(0), m_capacity(0), m_objects(0) {}


@@ -99,7 +99,7 @@ public:
         free(m_objects);
         m_objects = 0;
         m_size = m_capacity = 0;
         free(m_objects);
         m_objects = 0;
         m_size = m_capacity = 0;

-    };
+    }

 
     void reserve(size_type n)
     {
 
     void reserve(size_type n)
     {


@@ -107,7 +107,7 @@ public:
         {
             wxFAIL_MSG( _T("out of memory in wxVector::reserve()") );
         }
         {
             wxFAIL_MSG( _T("out of memory in wxVector::reserve()") );
         }

-    };
+    }

 
     size_type size() const
     {
 
     size_type size() const
     {


@@ -117,12 +117,12 @@ public:
     size_type capacity() const
     {
         return m_capacity;
     size_type capacity() const
     {
         return m_capacity;

-    };
+    }

 
     bool empty() const
     {
         return size() == 0;
 
     bool empty() const
     {
         return size() == 0;

-    };
+    }

 
     wxVectorBase& operator = (const wxVectorBase& vb)
     {
 
     wxVectorBase& operator = (const wxVectorBase& vb)
     {


@@ -132,18 +132,18 @@ public:
 };
 
 #define WX_DECLARE_VECTORBASE(obj, cls)\
 };
 
 #define WX_DECLARE_VECTORBASE(obj, cls)\

-private:\
+protected:\

     virtual void Free(void *o)\
     {\
         delete (obj *) o;\
     virtual void Free(void *o)\
     {\
         delete (obj *) o;\

-    };\
+    }\

     virtual void *Copy(const void *o) const\
     {\
         return new obj(*(obj *) o);\
     virtual void *Copy(const void *o) const\
     {\
         return new obj(*(obj *) o);\

-    };\
+    }\

 public:\
     cls() {}\
 public:\
     cls() {}\

-    cls(const cls& c)\
+    cls(const cls& c) : wxVectorBase()\

     {\
         wxCHECK2(copy(c), return);\
     }\
     {\
         wxCHECK2(copy(c), return);\
     }\


@@ -161,48 +161,48 @@ public:\
     {\
         wxCHECK2(Alloc(size() + 1), return);\
         Append(new obj(o));\
     {\
         wxCHECK2(Alloc(size() + 1), return);\
         Append(new obj(o));\

-    };\
+    }\

     void pop_back()\
     {\
         RemoveAt(size() - 1);\
     void pop_back()\
     {\
         RemoveAt(size() - 1);\

-    };\
+    }\

     const obj& at(size_type idx) const\
     {\
         return *(obj *) GetItem(idx);\
     const obj& at(size_type idx) const\
     {\
         return *(obj *) GetItem(idx);\

-    };\
+    }\

     obj& at(size_type idx)\
     {\
         return *(obj *) GetItem(idx);\
     obj& at(size_type idx)\
     {\
         return *(obj *) GetItem(idx);\

-    };\
+    }\

     const obj& operator[](size_type idx) const\
     {\
         return at(idx);\
     const obj& operator[](size_type idx) const\
     {\
         return at(idx);\

-    };\
+    }\

     obj& operator[](size_type idx)\
     {\
         return at(idx);\
     obj& operator[](size_type idx)\
     {\
         return at(idx);\

-    };\
+    }\

     const obj& front() const\
     {\
         return at(0);\
     const obj& front() const\
     {\
         return at(0);\

-    };\
+    }\

     obj& front()\
     {\
         return at(0);\
     obj& front()\
     {\
         return at(0);\

-    };\
+    }\

     const obj& back() const\
     {\
         return at(size() - 1);\
     const obj& back() const\
     {\
         return at(size() - 1);\

-    };\
+    }\

     obj& back()\
     {\
         return at(size() - 1);\
     obj& back()\
     {\
         return at(size() - 1);\

-    };\
+    }\

     size_type erase(size_type idx)\
     {\
         RemoveAt(idx);\
         return idx;\
     size_type erase(size_type idx)\
     {\
         RemoveAt(idx);\
         return idx;\

-    };\
+    }\

 }
 
 #define WX_DECLARE_VECTOR(obj, cls) \
 }
 
 #define WX_DECLARE_VECTOR(obj, cls) \