Add internal tag to VectorAlgorithms
[platform/core/uifw/dali-core.git] / dali / public-api / common / dali-vector.h
old mode 100644 (file)
new mode 100755 (executable)
index 359e6dc..98fae38
@@ -1,8 +1,8 @@
-#ifndef __DALI_VECTOR_H__
-#define __DALI_VECTOR_H__
+#ifndef DALI_VECTOR_H
+#define DALI_VECTOR_H
 
 /*
- * Copyright (c) 2015 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2019 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -21,6 +21,7 @@
 // EXTERNAL INCLUDES
 #include <cstddef>
 #include <algorithm>
+#include <cstdint> // uint32_t
 
 // INTERNAL INCLUDES
 #include <dali/public-api/common/dali-common.h>
 #include <dali/public-api/math/math-utils.h>
 
 /**
- * @brief For DALi internal use asserts are enabled in debug builds.
+ * @brief For DALi internal use, asserts are enabled in debug builds.
  *
- * For Application use asserts can be enabled manually.
+ * For Application use, asserts can be enabled manually.
+ * @SINCE_1_0.0
  */
 #if defined( DEBUG_ENABLED )
 #define ENABLE_VECTOR_ASSERTS
 
 namespace Dali
 {
+/**
+ * @addtogroup dali_core_common
+ * @{
+ */
 
 /**
  * @brief Base class to handle the memory of simple vector.
  *
  * Memory layout is such that it has two std::size_t to hold the count
- * and capacity of the vector. mData is adjusted so that it points to the
+ * and capacity of the vector. VectorBase::mData is adjusted so that it points to the
  * beginning of the first real item so that iterating the items is quick.
+ * @SINCE_1_0.0
  */
-class DALI_IMPORT_API VectorBase
+class DALI_CORE_API VectorBase
 {
 public: // Typedefs
 
@@ -62,6 +69,7 @@ protected: // Construction
 
   /**
    * @brief Default constructor. Does not allocate space.
+   * @SINCE_1_0.0
    */
   VectorBase();
 
@@ -69,17 +77,19 @@ protected: // Construction
    * @brief Destructor.
    *
    * Does not release the space. Derived class needs to call Release.
-   * Not virtual as should not be called directly and we do not want
+   * Not virtual as this should not be called directly and we do not want
    * a vtable for this class as it would unnecessarily increase size.
+   * @SINCE_1_0.0
    */
   ~VectorBase();
 
 public: // API
 
   /**
-   * @brief This method is inlined as its needed frequently for End() iterator.
+   * @brief This method is inlined as it's needed frequently for Vector::End() iterator.
    *
-   * @return The count of elements in this vector.
+   * @SINCE_1_0.0
+   * @return The count of elements in this vector
    */
   SizeType Count() const
   {
@@ -93,7 +103,9 @@ public: // API
   }
 
   /**
-   * @return The count of elements in this vector.
+   * @brief Gets the count of elements in this vector.
+   * @SINCE_1_0.0
+   * @return The count of elements in this vector
    */
   SizeType Size() const
   {
@@ -101,14 +113,27 @@ public: // API
   }
 
   /**
-   * @return The capacity of this vector.
+   * @brief @ return if the vector is empty.
+   * @SINCE_1_0.0
+   * @return True if the count of elements is empty
+   */
+  bool Empty() const
+  {
+    return Count() == 0u;
+  }
+
+  /**
+   * @brief Gets the capacity of this vector.
+   * @SINCE_1_0.0
+   * @return The capacity of this vector
    */
   SizeType Capacity() const;
 
   /**
-   * @brief Release the data.
+   * @brief Releases the data.
    *
    * Does not call destructors on objects held.
+   * @SINCE_1_0.0
    */
   void Release();
 
@@ -117,70 +142,77 @@ protected: // for Derived classes
   /**
    * @brief Helper to set the count.
    *
-   * @param count Number of elements in the vector.
+   * @SINCE_1_0.0
+   * @param[in] count Number of elements in the vector
    */
   void SetCount( SizeType count );
 
   /**
-   * @brief Reserve space in the vector.
+   * @brief Reserves space in the vector.
    *
-   * @param count of elements to reserve.
-   * @param elementSize of a single element.
+   * @SINCE_1_0.0
+   * @param[in] count Count of elements to reserve
+   * @param[in] elementSize Size of a single element
    */
   void Reserve( SizeType count, SizeType elementSize );
 
   /**
    * @brief Copy a vector.
    *
-   * @param vector Vector to copy from.
-   * @param elementSize of a single element.
+   * @SINCE_1_0.0
+   * @param[in] vector Vector to copy from
+   * @param[in] elementSize Size of a single element
    */
   void Copy( const VectorBase& vector, SizeType elementSize );
 
   /**
-   * @brief Swap the contents of two vectors.
+   * @brief Swaps the contents of two vectors.
    *
-   * @param vector Vector to swap with.
+   * @SINCE_1_0.0
+   * @param[in] vector Vector to swap with
    */
   void Swap( VectorBase& vector );
 
   /**
-   * @brief Erase an element.
+   * @brief Erases an element.
    *
    * Does not change capacity.
-   * @pre last element cannot be erased as there is nothing to move.
-   * @param address to erase from.
-   * @param elementSize to erase.
+   * @SINCE_1_0.0
+   * @param[in] address Address to erase from
+   * @param[in] elementSize Size to erase
+   * @pre Last element cannot be erased as there is nothing to move.
    */
   void Erase( char* address, SizeType elementSize );
 
   /**
-   * @brief Erase a range of elements.
+   * @brief Erases a range of elements.
    *
    * Does not change capacity.
-   * @param[in] first Address to the first element to be erased.
-   * @param[in] last Address to the last element to be erased.
-   * @param[in] elementSize Size of one of the elements to be erased.
-   * @return address pointing to the next element of the last one.
+   * @SINCE_1_0.0
+   * @param[in] first Address to the first element to be erased
+   * @param[in] last Address to the last element to be erased
+   * @param[in] elementSize Size of one of the elements to be erased
+   * @return Address pointing to the next element of the last one
    */
   char* Erase( char* first, char* last, SizeType elementSize );
 
   /**
-   * Copies a number of bytes from \e source to \e destination.
+   * @brief Copies a number of bytes from \e source to \e destination.
    *
    * \e source and \e destination must not overlap.
    *
-   * @param[in] destination Pointer to the destination address.
-   * @param[in] source Pointer to the source address.
-   * @param[in] numberOfBytes The number of bytes to be copied.
+   * @SINCE_1_0.0
+   * @param[in] destination Pointer to the destination address
+   * @param[in] source Pointer to the source address
+   * @param[in] numberOfBytes The number of bytes to be copied
    */
   void CopyMemory( char* destination, const char* source, size_t numberOfBytes );
 
 private:
 
-  // not copiable as it does not know the size of elements
-  VectorBase( const VectorBase& ); ///< Undefined
-  VectorBase& operator=( const VectorBase& ); ///< Undefined
+  // not copyable as it does not know the size of elements
+  VectorBase( const VectorBase& ); ///< Undefined @SINCE_1_0.0
+  VectorBase& operator=( const VectorBase& ); ///< Undefined @SINCE_1_0.0
 
 protected: // Data
 
@@ -188,10 +220,12 @@ protected: // Data
 
 };
 
+/// @cond internal
 /**
  * @brief Vector algorithm variant for trivial types.
  *
  * Trivial types do not need destructor or copy constructor called.
+ * @SINCE_1_0.0
  */
 template< bool IsTrivial >
 class VectorAlgorithms : public VectorBase
@@ -202,12 +236,14 @@ protected: // API for deriving classes
 
   /**
    * @brief Empty constructor.
+   * @SINCE_1_0.0
    */
   VectorAlgorithms()
   { }
 
   /**
    * @brief Empty destructor.
+   * @SINCE_1_0.0
    */
   ~VectorAlgorithms()
   { }
@@ -215,8 +251,9 @@ protected: // API for deriving classes
   /**
    * @brief Copy vector contents.
    *
-   * @param rhs to copy from.
-   * @param elementSize of the content.
+   * @SINCE_1_0.0
+   * @param[in] rhs VectorBase object to copy from
+   * @param[in] elementSize Size of the content
    */
   void Copy( const VectorBase& rhs, SizeType elementSize )
   {
@@ -231,10 +268,11 @@ protected: // API for deriving classes
   }
 
   /**
-   * @brief Reserve space in the vector.
+   * @brief Reserves space in the vector.
    *
-   * @param count of elements to reserve.
-   * @param elementSize of a single element.
+   * @SINCE_1_0.0
+   * @param[in] count Count of elements to reserve
+   * @param[in] elementSize Size of a single element
    */
   void Reserve( SizeType count, SizeType elementSize )
   {
@@ -242,10 +280,11 @@ protected: // API for deriving classes
   }
 
   /**
-   * @brief Resize the vector. Does not change capacity.
+   * @brief Resizes the vector. Does not change capacity.
    *
-   * @param count to resize to.
-   * @param elementSize of a single element.
+   * @SINCE_1_0.0
+   * @param[in] count Count to resize to
+   * @param[in] elementSize Size of a single element
    */
   void Resize( SizeType count, SizeType elementSize )
   {
@@ -254,9 +293,10 @@ protected: // API for deriving classes
   }
 
   /**
-   * @brief Clear the contents.
+   * @brief Clears the contents.
    *
-   * For simple types nothing to do.
+   * For simple types, nothing to do.
+   * @SINCE_1_0.0
    */
   void Clear()
   {
@@ -267,7 +307,8 @@ protected: // API for deriving classes
   }
 
   /**
-   * @brief Release the vector.
+   * @brief Releases the vector.
+   * @SINCE_1_0.0
    */
   void Release()
   {
@@ -275,38 +316,41 @@ protected: // API for deriving classes
   }
 
   /**
-   * @brief Erase an element. Does not change capacity.
+   * @brief Erases an element. Does not change capacity.
    *
-   * @param address to erase from.
-   * @param elementSize to erase.
+   * @SINCE_1_0.0
+   * @param[in] address Address to erase from
+   * @param[in] elementSize Size to erase
    */
-  void Erase( char* address, SizeType elementSize )
+  void Erase( uint8_t* address, SizeType elementSize )
   {
-    VectorBase::Erase( address, elementSize );
+    VectorBase::Erase( reinterpret_cast< char* >( address ), elementSize );
   }
 
   /**
-   * @brief Erase a range of elements. Does not change capacity.
+   * @brief Erases a range of elements. Does not change capacity.
    *
-   * @param[in] first Address to the first element to be erased.
-   * @param[in] last Address to the last element to be erased.
-   * @param[in] elementSize Size of one of the elements to be erased.
-   * @return address pointing to the next element of the last one.
+   * @SINCE_1_0.0
+   * @param[in] first Address to the first element to be erased
+   * @param[in] last Address to the last element to be erased
+   * @param[in] elementSize Size of one of the elements to be erased
+   * @return Address pointing to the next element of the last one
    */
-  char* Erase( char* first, char* last, SizeType elementSize )
+  uint8_t* Erase( uint8_t* first, uint8_t* last, SizeType elementSize )
   {
-    return VectorBase::Erase( first, last, elementSize );
+    return reinterpret_cast< uint8_t* >( VectorBase::Erase( reinterpret_cast< char* >( first ), reinterpret_cast< char *>( last ), elementSize ) );
   }
 
   /**
    * @brief Inserts the given elements into the vector.
    *
-   * @param[in] at Address where to insert the elements into the vector.
-   * @param[in] from Address to the first element to be inserted.
-   * @param[in] to Address to the last element to be inserted.
-   * @param[in] elementSize Size of one of the elements to be inserted.
+   * @SINCE_1_0.0
+   * @param[in] at Address where to insert the elements into the vector
+   * @param[in] from Address to the first element to be inserted
+   * @param[in] to Address to the last element to be inserted
+   * @param[in] elementSize Size of one of the elements to be inserted
    */
-  void Insert( char* at, char* from, char* to, SizeType elementSize )
+  void Insert( uint8_t* at, uint8_t* from, uint8_t* to, SizeType elementSize )
   {
     const SizeType size = to - from;
     const SizeType count = Count();
@@ -315,33 +359,36 @@ protected: // API for deriving classes
     if( newCount > Capacity() )
     {
       // Calculate the at offset as the pointer is invalid after the Reserve() call.
-      std::size_t offset = at - reinterpret_cast<char*>( mData );
+      std::size_t offset = at - reinterpret_cast<uint8_t*>( mData );
 
       // need more space
-      Reserve( NextPowerOfTwo( newCount ), elementSize ); // reserve enough space to store at least the next power of two elements of the new required size.
+      Reserve( NextPowerOfTwo( static_cast<uint32_t>( newCount ) ), elementSize ); // reserve enough space to store at least the next power of two elements of the new required size.
 
       // Set the new at pointer.
-      at = reinterpret_cast<char*>( mData ) + offset;
+      at = reinterpret_cast<uint8_t*>( mData ) + offset;
     }
     // set new count first as otherwise the debug assert will hit us
     SetCount( newCount );
 
     // Move current items to a new position inside the vector.
-    CopyMemory( at + size,
-                at,
-                ( reinterpret_cast<char*>( mData ) + count * elementSize ) - at );
+    CopyMemory( reinterpret_cast< char* >( at + size ),
+                reinterpret_cast< const char* >( at ),
+                ( reinterpret_cast<uint8_t*>( mData ) + count * elementSize ) - at );
 
     // Copy the given items.
-    CopyMemory( at, from, size );
+    CopyMemory( reinterpret_cast< char* >( at ), reinterpret_cast< const char* >( from ), size );
   }
 };
+/// @endcond
 
+/// @cond internal
 /**
  * @brief Vector algorithm variant for complex types.
  *
  * Not yet supported so will lead to compile error
  * as constructor and destructor are private.
  * TODO add support for this variant.
+ * @SINCE_1_0.0
  */
 template<>
 class VectorAlgorithms< false > : public VectorBase
@@ -352,11 +399,13 @@ private:
   ~VectorAlgorithms()
   { }
 };
+/// @endcond
 
 /**
- * @brief Vector class with minimum space allocation when its empty.
+ * @brief Vector class with minimum space allocation when it's empty.
  *
- * @param type of the data that the vector holds.
+ * @SINCE_1_0.0
+ * @param type The type of the data that the vector holds
  */
 template< class T, bool IsTrivialType = TypeTraits<T>::IS_TRIVIAL_TYPE == true >
 class Vector : public VectorAlgorithms< IsTrivialType >
@@ -365,25 +414,32 @@ public: // API
 
   /**
    * @brief Type definitions.
+   * @SINCE_1_0.0
    */
-  typedef VectorBase::SizeType SizeType;
-  typedef T* Iterator;  ///< Most simple Iterator is a pointer
-  typedef const T* ConstIterator;
-  typedef T  ItemType;
+  typedef VectorBase::SizeType SizeType; ///< Size type @SINCE_1_0.0
+  typedef T* Iterator;  ///< Most simple Iterator is a pointer @SINCE_1_0.0
+  typedef const T* ConstIterator; ///< Const iterator @SINCE_1_0.0
+  typedef T  ItemType; ///< Item type @SINCE_1_0.0
 
+  /**
+   * @brief Enumeration for BaseType.
+   * @SINCE_1_0.0
+   */
   enum
   {
-    BaseType = IsTrivialType
+    BaseType = IsTrivialType ///< @SINCE_1_0.0
   };
 
   /**
    * @brief Default constructor. Does not allocate any space.
+   * @SINCE_1_0.0
    */
   Vector()
   { }
 
   /**
    * @brief Destructor. Releases the allocated space.
+   * @SINCE_1_0.0
    */
   ~Vector()
   {
@@ -393,7 +449,8 @@ public: // API
   /**
    * @brief Copy constructor.
    *
-   * @param vector Vector to copy from.
+   * @SINCE_1_0.0
+   * @param[in] vector Vector to copy from
    */
   Vector( const Vector& vector )
   {
@@ -404,8 +461,9 @@ public: // API
   /**
    * @brief Assignment operator.
    *
-   * @param  vector Vector to assign from.
-   * @return reference to self for chaining.
+   * @SINCE_1_0.0
+   * @param[in] vector Vector to assign from
+   * @return Reference to self for chaining
    */
   Vector& operator=( const Vector& vector )
   {
@@ -417,7 +475,9 @@ public: // API
   }
 
   /**
-   * @return Iterator to the beginning of the data.
+   * @brief Iterator to the beginning of the data.
+   * @SINCE_1_0.0
+   * @return Iterator to the beginning of the data
    */
   Iterator Begin() const
   {
@@ -426,7 +486,9 @@ public: // API
   }
 
   /**
-   * @return Iterator to the end of the data (one past last element).
+   * @brief Iterator to the end of the data (one past last element).
+   * @SINCE_1_0.0
+   * @return Iterator to the end of the data (one past last element)
    */
   Iterator End() const
   {
@@ -436,9 +498,31 @@ public: // API
   }
 
   /**
-   * @pre index must be in the vector's range.
-   * @param  index of the element.
-   * @return reference to the element for given index.
+   * Support for C++11 Range-based for loop: for( item : container ).
+   * @SINCE_1_2.60
+   * @return The start iterator
+   */
+  Iterator begin() const
+  {
+    return Begin();
+  }
+
+  /**
+   * Support for C++11 Range-based for loop: for( item : container ).
+   * @SINCE_1_2.60
+   * @return The end iterator
+   */
+  Iterator end() const
+  {
+    return End();
+  }
+
+  /**
+   * @brief Subscript operator.
+   * @SINCE_1_0.0
+   * @param[in] index Index of the element
+   * @return Reference to the element for given index
+   * @pre Index must be in the vector's range.
    */
   ItemType& operator[]( SizeType index )
   {
@@ -447,9 +531,11 @@ public: // API
   }
 
   /**
-   * @pre index must be in the vector's range.
-   * @param  index of the element.
-   * @return reference to the element for given index.
+   * @brief Subscript operator.
+   * @SINCE_1_0.0
+   * @param[in] index Index of the element
+   * @return Reference to the element for given index
+   * @pre Index must be in the vector's range.
    */
   const ItemType& operator[]( SizeType index ) const
   {
@@ -461,13 +547,14 @@ public: // API
   }
 
   /**
-   * @brief Push back an element to the vector.
+   * @brief Pushes back an element to the vector.
    *
    * The underlying storage may be reallocated to provide space.
    * If this occurs, all pre-existing pointers into the vector will
    * become invalid.
    *
-   * @param[in] element to be added.
+   * @SINCE_1_0.0
+   * @param[in] element Element to be added
    */
   void PushBack( const ItemType& element )
   {
@@ -485,7 +572,7 @@ public: // API
   }
 
   /**
-   *@brief Insert an element to the vector.
+   * @brief Inserts an element to the vector.
    *
    * Elements after \e at are moved one position to the right.
    *
@@ -493,17 +580,17 @@ public: // API
    * If this occurs, all pre-existing pointers into the vector will
    * become invalid.
    *
+   * @SINCE_1_0.0
+   * @param[in] at Iterator where to insert the elements into the vector
+   * @param[in] element An element to be added
    * @pre Iterator at must be in the vector's range ( Vector::Begin(), Vector::End() ).
-   *
-   * @param[in] at Iterator where to insert the elements into the vector.
-   * @param[in] element to be added.
    */
   void Insert( Iterator at, const ItemType& element )
   {
     DALI_ASSERT_VECTOR( ( at <= End() ) && ( at >= Begin() ) && "Iterator not inside vector" );
     const SizeType size = sizeof( ItemType );
-    char* address = const_cast<char*>( reinterpret_cast<const char*>( &element ) );
-    VectorAlgorithms<BaseType>::Insert( reinterpret_cast< char* >( at ),
+    uint8_t* address = const_cast<uint8_t*>( reinterpret_cast<const uint8_t*>( &element ) );
+    VectorAlgorithms<BaseType>::Insert( reinterpret_cast< uint8_t* >( at ),
                                         address,
                                         address + size,
                                         size );
@@ -518,13 +605,14 @@ public: // API
    * If this occurs, all pre-existing pointers into the vector will
    * become invalid.
    *
+   * @SINCE_1_0.0
+   * @param[in] at Iterator where to insert the elements into the vector
+   * @param[in] from Iterator to the first element to be inserted
+   * @param[in] to Iterator to the last element to be inserted
    * @pre Iterator \e at must be in the vector's range ( Vector::Begin(), Vector::End() ).
    * @pre Iterators \e from and \e to must be valid iterators.
    * @pre Iterator \e from must not be grater than Iterator \e to.
    *
-   * @param[in] at Iterator where to insert the elements into the vector.
-   * @param[in] from Iterator to the first element to be inserted.
-   * @param[in] to Iterator to the last element to be inserted.
    */
   void Insert( Iterator at, Iterator from, Iterator to )
   {
@@ -537,17 +625,18 @@ public: // API
       return;
     }
 
-    VectorAlgorithms<BaseType>::Insert( reinterpret_cast< char* >( at ),
-                                        reinterpret_cast< char* >( from ),
-                                        reinterpret_cast< char* >( to ),
+    VectorAlgorithms<BaseType>::Insert( reinterpret_cast< uint8_t* >( at ),
+                                        reinterpret_cast< uint8_t* >( from ),
+                                        reinterpret_cast< uint8_t* >( to ),
                                         sizeof( ItemType ) );
   }
 
   /**
-   * @brief Reserve space in the vector.
+   * @brief Reserves space in the vector.
    *
    * Reserving less than current Capacity is a no-op.
-   * @param count of elements to reserve.
+   * @SINCE_1_0.0
+   * @param[in] count Count of elements to reserve
    */
   void Reserve( SizeType count )
   {
@@ -555,12 +644,25 @@ public: // API
   }
 
   /**
-   * @brief Resize the vector. Does not change capacity.
+   * @brief Resizes the vector. Does not change capacity.
    *
-   * @param count to resize to.
-   * @param item to insert to the new indices.
+   * @SINCE_1_0.0
+   * @param[in] count Count to resize to
    */
-  void Resize( SizeType count, ItemType item = ItemType() )
+  void Resize( SizeType count )
+  {
+    ItemType item = ItemType();
+    Resize(count, item);
+  }
+
+  /**
+   * @brief Resizes the vector. Does not change capacity.
+   *
+   * @SINCE_1_0.0
+   * @param[in] count Count to resize to
+   * @param[in] item An item to insert to the new indices
+   */
+  void Resize( SizeType count, const ItemType& item )
   {
     const SizeType oldCount = VectorBase::Count();
     if( count <= oldCount )
@@ -581,21 +683,22 @@ public: // API
   }
 
   /**
-   * @brief Erase an element.
+   * @brief Erases an element.
    *
    * Does not change capacity. Other elements get moved.
    *
+   * @SINCE_1_0.0
+   * @param[in] iterator Iterator pointing to the item to remove
+   * @return Iterator pointing to next element
    * @pre Iterator \e iterator must be within the vector's range ( Vector::Begin(), Vector::End() - 1 ).
    *
-   * @param iterator Iterator pointing to item to remove.
-   * @return Iterator pointing to next element.
    */
   Iterator Erase( Iterator iterator )
   {
     DALI_ASSERT_VECTOR( (iterator < End()) && (iterator >= Begin()) && "Iterator not inside vector" );
     if( iterator < ( End() - 1u ) )
     {
-      VectorAlgorithms<BaseType>::Erase( reinterpret_cast< char* >( iterator ), sizeof( ItemType ) );
+      VectorAlgorithms<BaseType>::Erase( reinterpret_cast< uint8_t* >( iterator ), sizeof( ItemType ) );
     }
     else
     {
@@ -606,18 +709,19 @@ public: // API
   }
 
   /**
-   * @brief Erase a range of elements.
+   * @brief Erases a range of elements.
    *
    * Does not change capacity. Other elements get moved.
    *
+   * @SINCE_1_0.0
+   * @param[in] first Iterator to the first element to be erased
+   * @param[in] last Iterator to the last element to be erased
+   *
+   * @return Iterator pointing to the next element of the last one
    * @pre Iterator \e first must be in the vector's range ( Vector::Begin(), Vector::End() ).
    * @pre Iterator \e last must be in the vector's range ( Vector::Begin(), Vector::End() ).
    * @pre Iterator \e first must not be grater than Iterator \e last.
    *
-   * @param[in] first Iterator to the first element to be erased.
-   * @param[in] last Iterator to the last element to be erased.
-   *
-   * @return Iterator pointing to the next element of the last one.
    */
   Iterator Erase( Iterator first, Iterator last )
   {
@@ -635,8 +739,8 @@ public: // API
     }
     else
     {
-      nextElement = reinterpret_cast<Iterator>( VectorAlgorithms<BaseType>::Erase( reinterpret_cast< char* >( first ),
-                                                                                   reinterpret_cast< char* >( last ),
+      nextElement = reinterpret_cast<Iterator>( VectorAlgorithms<BaseType>::Erase( reinterpret_cast< uint8_t* >( first ),
+                                                                                   reinterpret_cast< uint8_t* >( last ),
                                                                                    sizeof( ItemType ) ) );
     }
 
@@ -646,13 +750,14 @@ public: // API
   /**
    * @brief Removes an element.
    *
-   * Does not maintain order.  Swaps the element with end and
+   * Does not maintain order. Swaps the element with end and
    * decreases size by one.  This is much faster than Erase so use
    * this in case order does not matter. Does not change capacity.
    *
+   * @SINCE_1_0.0
+   * @param[in] iterator Iterator pointing to the item to remove
    * @pre Iterator \e iterator must be in the vector's range ( Vector::Begin(), Vector::End() - 1 ).
    *
-   * @param iterator Iterator pointing to item to remove.
    */
   void Remove( Iterator iterator )
   {
@@ -667,9 +772,10 @@ public: // API
   }
 
   /**
-   * @brief Swap the contents of two vectors.
+   * @brief Swaps the contents of two vectors.
    *
-   * @param vector Vector to swap with.
+   * @SINCE_1_0.0
+   * @param[in] vector Vector to swap with
    */
   void Swap( Vector& vector )
   {
@@ -677,7 +783,8 @@ public: // API
   }
 
   /**
-   * @brief Clear the contents of the vector. Keeps its capacity.
+   * @brief Clears the contents of the vector. Keeps its capacity.
+   * @SINCE_1_0.0
    */
   void Clear()
   {
@@ -685,7 +792,8 @@ public: // API
   }
 
   /**
-   * @brief Release the memory that the vector holds.
+   * @brief Releases the memory that the vector holds.
+   * @SINCE_1_0.0
    */
   void Release()
   {
@@ -693,6 +801,9 @@ public: // API
   }
 };
 
+/**
+ * @}
+ */
 } // namespace Dali
 
-#endif /* __DALI_VECTOR_H__ */
+#endif // DALI_VECTOR_H