[kdbus] sync with kdbus (kdbus.h - commit: 5ae1ecac44cb)
[platform/upstream/glib.git] / glib / garray.c
index 07149b0..8a94720 100644 (file)
@@ -12,9 +12,7 @@
  * Lesser General Public License for more details.
  *
  * You should have received a copy of the GNU Lesser General Public
- * License along with this library; if not, write to the
- * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
- * Boston, MA 02111-1307, USA.
+ * License along with this library; if not, see <http://www.gnu.org/licenses/>.
  */
 
 /*
 
 #include "garray.h"
 
+#include "gbytes.h"
+#include "gslice.h"
 #include "gmem.h"
+#include "gtestutils.h"
 #include "gthread.h"
 #include "gmessages.h"
 #include "gqsort.h"
@@ -45,7 +46,7 @@
  * SECTION:arrays
  * @title: Arrays
  * @short_description: arrays of arbitrary elements which grow
- *                     automatically as elements are added
+ *     automatically as elements are added
  *
  * Arrays are similar to standard C arrays, except that they grow
  * automatically as elements are added.
  *
  * To free an array, use g_array_free().
  *
- * <example>
- *  <title>Using a #GArray to store #gint values</title>
- *  <programlisting>
+ * Here is an example that stores integers in a #GArray:
+ * |[<!-- language="C" -->
  *   GArray *garray;
  *   gint i;
- *   /<!-- -->* We create a new array to store gint values.
- *      We don't want it zero-terminated or cleared to 0's. *<!-- -->/
+ *   // We create a new array to store gint values.
+ *   // We don't want it zero-terminated or cleared to 0's.
  *   garray = g_array_new (FALSE, FALSE, sizeof (gint));
- *   for (i = 0; i &lt; 10000; i++)
+ *   for (i = 0; i < 10000; i++)
  *     g_array_append_val (garray, i);
- *   for (i = 0; i &lt; 10000; i++)
+ *   for (i = 0; i < 10000; i++)
  *     if (g_array_index (garray, gint, i) != i)
- *       g_print ("ERROR: got &percnt;d instead of &percnt;d\n",
+ *       g_print ("ERROR: got %d instead of %d\n",
  *                g_array_index (garray, gint, i), i);
  *   g_array_free (garray, TRUE);
- *  </programlisting>
- * </example>
- **/
+ * ]|
+ */
 
 #define MIN_ARRAY_SIZE  16
 
@@ -92,12 +91,12 @@ typedef struct _GRealArray  GRealArray;
 /**
  * GArray:
  * @data: a pointer to the element data. The data may be moved as
- *        elements are added to the #GArray.
+ *     elements are added to the #GArray.
  * @len: the number of elements in the #GArray not including the
- *       possible terminating zero element.
+ *     possible terminating zero element.
  *
- * Contains the public fields of an <link linkend="glib-Arrays">Array</link>.
- **/
+ * Contains the public fields of a GArray.
+ */
 struct _GRealArray
 {
   guint8 *data;
@@ -107,82 +106,91 @@ struct _GRealArray
   guint   zero_terminated : 1;
   guint   clear : 1;
   gint    ref_count;
+  GDestroyNotify clear_func;
 };
 
 /**
  * g_array_index:
- * @a: a #GArray.
- * @t: the type of the elements.
- * @i: the index of the element to return.
- * @Returns: the element of the #GArray at the index given by @i.
+ * @a: a #GArray
+ * @t: the type of the elements
+ * @i: the index of the element to return
  *
  * Returns the element of a #GArray at the given index. The return
  * value is cast to the given type.
  *
- * <example>
- *  <title>Getting a pointer to an element in a #GArray</title>
- *  <programlisting>
+ * This example gets a pointer to an element in a #GArray:
+ * |[<!-- language="C" -->
  *   EDayViewEvent *event;
- *   /<!-- -->* This gets a pointer to the 4th element
- *      in the array of EDayViewEvent structs. *<!-- -->/
- *   event = &amp;g_array_index (events, EDayViewEvent, 3);
- *  </programlisting>
- * </example>
- **/
+ *   // This gets a pointer to the 4th element in the array of
+ *   // EDayViewEvent structs.
+ *   event = &g_array_index (events, EDayViewEvent, 3);
+ * ]|
+ *
+ * Returns: the element of the #GArray at the index given by @i
+ */
 
 #define g_array_elt_len(array,i) ((array)->elt_size * (i))
 #define g_array_elt_pos(array,i) ((array)->data + g_array_elt_len((array),(i)))
-#define g_array_elt_zero(array, pos, len)                              \
+#define g_array_elt_zero(array, pos, len)                               \
   (memset (g_array_elt_pos ((array), pos), 0,  g_array_elt_len ((array), len)))
-#define g_array_zero_terminate(array) G_STMT_START{                    \
-  if ((array)->zero_terminated)                                                \
-    g_array_elt_zero ((array), (array)->len, 1);                       \
+#define g_array_zero_terminate(array) G_STMT_START{                     \
+  if ((array)->zero_terminated)                                         \
+    g_array_elt_zero ((array), (array)->len, 1);                        \
 }G_STMT_END
 
 static guint g_nearest_pow        (gint        num) G_GNUC_CONST;
 static void  g_array_maybe_expand (GRealArray *array,
-                                  gint        len);
+                                   gint        len);
 
 /**
  * g_array_new:
  * @zero_terminated: %TRUE if the array should have an extra element at
- *                   the end which is set to 0.
+ *     the end which is set to 0
  * @clear_: %TRUE if #GArray elements should be automatically cleared
- *          to 0 when they are allocated.
- * @element_size: the size of each element in bytes.
- * @Returns: the new #GArray.
+ *     to 0 when they are allocated
+ * @element_size: the size of each element in bytes
  *
  * Creates a new #GArray with a reference count of 1.
- **/
+ *
+ * Returns: the new #GArray
+ */
 GArray*
 g_array_new (gboolean zero_terminated,
-            gboolean clear,
-            guint    elt_size)
+             gboolean clear,
+             guint    elt_size)
 {
-  return (GArray*) g_array_sized_new (zero_terminated, clear, elt_size, 0);
+  g_return_val_if_fail (elt_size > 0, NULL);
+
+  return g_array_sized_new (zero_terminated, clear, elt_size, 0);
 }
 
 /**
  * g_array_sized_new:
  * @zero_terminated: %TRUE if the array should have an extra element at
- *                   the end with all bits cleared.
+ *     the end with all bits cleared
  * @clear_: %TRUE if all bits in the array should be cleared to 0 on
- *          allocation.
- * @element_size: size of each element in the array.
- * @reserved_size: number of elements preallocated.
- * @Returns: the new #GArray.
+ *     allocation
+ * @element_size: size of each element in the array
+ * @reserved_size: number of elements preallocated
  *
  * Creates a new #GArray with @reserved_size elements preallocated and
  * a reference count of 1. This avoids frequent reallocation, if you
  * are going to add many elements to the array. Note however that the
  * size of the array is still 0.
- **/
-GArray* g_array_sized_new (gboolean zero_terminated,
-                          gboolean clear,
-                          guint    elt_size,
-                          guint    reserved_size)
+ *
+ * Returns: the new #GArray
+ */
+GArray*
+g_array_sized_new (gboolean zero_terminated,
+                   gboolean clear,
+                   guint    elt_size,
+                   guint    reserved_size)
 {
-  GRealArray *array = g_slice_new (GRealArray);
+  GRealArray *array;
+  
+  g_return_val_if_fail (elt_size > 0, NULL);
+
+  array = g_slice_new (GRealArray);
 
   array->data            = NULL;
   array->len             = 0;
@@ -191,6 +199,7 @@ GArray* g_array_sized_new (gboolean zero_terminated,
   array->clear           = (clear ? 1 : 0);
   array->elt_size        = elt_size;
   array->ref_count       = 1;
+  array->clear_func      = NULL;
 
   if (array->zero_terminated || reserved_size != 0)
     {
@@ -202,16 +211,44 @@ GArray* g_array_sized_new (gboolean zero_terminated,
 }
 
 /**
+ * g_array_set_clear_func:
+ * @array: A #GArray
+ * @clear_func: a function to clear an element of @array
+ *
+ * Sets a function to clear an element of @array.
+ *
+ * The @clear_func will be called when an element in the array
+ * data segment is removed and when the array is freed and data
+ * segment is deallocated as well.
+ *
+ * Note that in contrast with other uses of #GDestroyNotify
+ * functions, @clear_func is expected to clear the contents of
+ * the array element it is given, but not free the element itself.
+ *
+ * Since: 2.32
+ */
+void
+g_array_set_clear_func (GArray         *array,
+                        GDestroyNotify  clear_func)
+{
+  GRealArray *rarray = (GRealArray *) array;
+
+  g_return_if_fail (array != NULL);
+
+  rarray->clear_func = clear_func;
+}
+
+/**
  * g_array_ref:
- * @array: A #GArray.
+ * @array: A #GArray
  *
- * Atomically increments the reference count of @array by one. This
- * function is MT-safe and may be called from any thread.
+ * Atomically increments the reference count of @array by one.
+ * This function is MT-safe and may be called from any thread.
  *
- * Returns: The passed in #GArray.
+ * Returns: The passed in #GArray
  *
  * Since: 2.22
- **/
+ */
 GArray *
 g_array_ref (GArray *array)
 {
@@ -223,9 +260,17 @@ g_array_ref (GArray *array)
   return array;
 }
 
+typedef enum
+{
+  FREE_SEGMENT = 1 << 0,
+  PRESERVE_WRAPPER = 1 << 1
+} ArrayFreeFlags;
+
+static gchar *array_free (GRealArray *, ArrayFreeFlags);
+
 /**
  * g_array_unref:
- * @array: A #GArray.
+ * @array: A #GArray
  *
  * Atomically decrements the reference count of @array by one. If the
  * reference count drops to 0, all memory allocated by the array is
@@ -233,7 +278,7 @@ g_array_ref (GArray *array)
  * thread.
  *
  * Since: 2.22
- **/
+ */
 void
 g_array_unref (GArray *array)
 {
@@ -241,19 +286,19 @@ g_array_unref (GArray *array)
   g_return_if_fail (array);
 
   if (g_atomic_int_dec_and_test (&rarray->ref_count))
-    g_array_free (array, TRUE);
+    array_free (rarray, FREE_SEGMENT);
 }
 
 /**
  * g_array_get_element_size:
- * @array: A #GArray.
+ * @array: A #GArray
  *
  * Gets the size of the elements in @array.
  *
- * Returns: Size of each element, in bytes.
+ * Returns: Size of each element, in bytes
  *
  * Since: 2.22
- **/
+ */
 guint
 g_array_get_element_size (GArray *array)
 {
@@ -266,10 +311,8 @@ g_array_get_element_size (GArray *array)
 
 /**
  * g_array_free:
- * @array: a #GArray.
- * @free_segment: if %TRUE the actual element data is freed as well.
- * @Returns: the element data if @free_segment is %FALSE, otherwise
- *           %NULL.  The element data should be freed using g_free().
+ * @array: a #GArray
+ * @free_segment: if %TRUE the actual element data is freed as well
  *
  * Frees the memory allocated for the #GArray. If @free_segment is
  * %TRUE it frees the memory block holding the elements as well and
@@ -279,33 +322,53 @@ g_array_get_element_size (GArray *array)
  * is greater than one, the #GArray wrapper is preserved but the size
  * of @array will be set to zero.
  *
- * <note><para>If array elements contain dynamically-allocated memory,
- * they should be freed separately.</para></note>
- **/
+ * If array elements contain dynamically-allocated memory, they should
+ * be freed separately.
+ *
+ * Returns: the element data if @free_segment is %FALSE, otherwise
+ *     %NULL. The element data should be freed using g_free().
+ */
 gchar*
 g_array_free (GArray   *farray,
-             gboolean  free_segment)
+              gboolean  free_segment)
 {
   GRealArray *array = (GRealArray*) farray;
-  gchar* segment;
-  gboolean preserve_wrapper;
+  ArrayFreeFlags flags;
 
   g_return_val_if_fail (array, NULL);
 
+  flags = (free_segment ? FREE_SEGMENT : 0);
+
   /* if others are holding a reference, preserve the wrapper but do free/return the data */
-  preserve_wrapper = FALSE;
-  if (g_atomic_int_get (&array->ref_count) > 1)
-    preserve_wrapper = TRUE;
+  if (!g_atomic_int_dec_and_test (&array->ref_count))
+    flags |= PRESERVE_WRAPPER;
 
-  if (free_segment)
+  return array_free (array, flags);
+}
+
+static gchar *
+array_free (GRealArray     *array,
+            ArrayFreeFlags  flags)
+{
+  gchar *segment;
+
+  if (flags & FREE_SEGMENT)
     {
+      if (array->clear_func != NULL)
+        {
+          guint i;
+
+          for (i = 0; i < array->len; i++)
+            array->clear_func (g_array_elt_pos (array, i));
+        }
+
       g_free (array->data);
       segment = NULL;
     }
   else
     segment = (gchar*) array->data;
 
-  if (preserve_wrapper)
+  if (flags & PRESERVE_WRAPPER)
     {
       array->data            = NULL;
       array->len             = 0;
@@ -321,30 +384,32 @@ g_array_free (GArray   *farray,
 
 /**
  * g_array_append_vals:
- * @array: a #GArray.
- * @data: a pointer to the elements to append to the end of the array.
- * @len: the number of elements to append.
- * @Returns: the #GArray.
+ * @array: a #GArray
+ * @data: a pointer to the elements to append to the end of the array
+ * @len: the number of elements to append
  *
  * Adds @len elements onto the end of the array.
- **/
+ *
+ * Returns: the #GArray
+ */
 /**
  * g_array_append_val:
- * @a: a #GArray.
- * @v: the value to append to the #GArray.
- * @Returns: the #GArray.
+ * @a: a #GArray
+ * @v: the value to append to the #GArray
  *
  * Adds the value on to the end of the array. The array will grow in
  * size automatically if necessary.
  *
- * <note><para>g_array_append_val() is a macro which uses a reference
- * to the value parameter @v. This means that you cannot use it with
- * literal values such as "27". You must use variables.</para></note>
- **/
+ * g_array_append_val() is a macro which uses a reference to the value
+ * parameter @v. This means that you cannot use it with literal values
+ * such as "27". You must use variables.
+ *
+ * Returns: the #GArray
+ */
 GArray*
 g_array_append_vals (GArray       *farray,
-                    gconstpointer data,
-                    guint         len)
+                     gconstpointer data,
+                     guint         len)
 {
   GRealArray *array = (GRealArray*) farray;
 
@@ -353,7 +418,7 @@ g_array_append_vals (GArray       *farray,
   g_array_maybe_expand (array, len);
 
   memcpy (g_array_elt_pos (array, array->len), data, 
-         g_array_elt_len (array, len));
+          g_array_elt_len (array, len));
 
   array->len += len;
 
@@ -364,23 +429,22 @@ g_array_append_vals (GArray       *farray,
 
 /**
  * g_array_prepend_vals:
- * @array: a #GArray.
- * @data: a pointer to the elements to prepend to the start of the
- *        array.
- * @len: the number of elements to prepend.
- * @Returns: the #GArray.
+ * @array: a #GArray
+ * @data: a pointer to the elements to prepend to the start of the array
+ * @len: the number of elements to prepend
  *
  * Adds @len elements onto the start of the array.
  *
  * This operation is slower than g_array_append_vals() since the
  * existing elements in the array have to be moved to make space for
  * the new elements.
- **/
+ *
+ * Returns: the #GArray
+ */
 /**
  * g_array_prepend_val:
- * @a: a #GArray.
- * @v: the value to prepend to the #GArray.
- * @Returns: the #GArray.
+ * @a: a #GArray
+ * @v: the value to prepend to the #GArray
  *
  * Adds the value on to the start of the array. The array will grow in
  * size automatically if necessary.
@@ -389,14 +453,16 @@ g_array_append_vals (GArray       *farray,
  * existing elements in the array have to be moved to make space for
  * the new element.
  *
- * <note><para>g_array_prepend_val() is a macro which uses a reference
- * to the value parameter @v. This means that you cannot use it with
- * literal values such as "27". You must use variables.</para></note>
- **/
+ * g_array_prepend_val() is a macro which uses a reference to the value
+ * parameter @v. This means that you cannot use it with literal values
+ * such as "27". You must use variables.
+ *
+ * Returns: the #GArray
+ */
 GArray*
 g_array_prepend_vals (GArray        *farray,
-                     gconstpointer  data,
-                     guint          len)
+                      gconstpointer  data,
+                      guint          len)
 {
   GRealArray *array = (GRealArray*) farray;
 
@@ -404,8 +470,8 @@ g_array_prepend_vals (GArray        *farray,
 
   g_array_maybe_expand (array, len);
 
-  g_memmove (g_array_elt_pos (array, len), g_array_elt_pos (array, 0), 
-            g_array_elt_len (array, array->len));
+  memmove (g_array_elt_pos (array, len), g_array_elt_pos (array, 0),
+           g_array_elt_len (array, array->len));
 
   memcpy (g_array_elt_pos (array, 0), data, g_array_elt_len (array, len));
 
@@ -418,32 +484,34 @@ g_array_prepend_vals (GArray        *farray,
 
 /**
  * g_array_insert_vals:
- * @array: a #GArray.
- * @index_: the index to place the elements at.
- * @data: a pointer to the elements to insert.
- * @len: the number of elements to insert.
- * @Returns: the #GArray.
+ * @array: a #GArray
+ * @index_: the index to place the elements at
+ * @data: a pointer to the elements to insert
+ * @len: the number of elements to insert
  *
  * Inserts @len elements into a #GArray at the given index.
- **/
+ *
+ * Returns: the #GArray
+ */
 /**
  * g_array_insert_val:
- * @a: a #GArray.
- * @i: the index to place the element at.
- * @v: the value to insert into the array.
- * @Returns: the #GArray.
+ * @a: a #GArray
+ * @i: the index to place the element at
+ * @v: the value to insert into the array
  *
  * Inserts an element into an array at the given index.
  *
- * <note><para>g_array_insert_val() is a macro which uses a reference
- * to the value parameter @v. This means that you cannot use it with
- * literal values such as "27". You must use variables.</para></note>
- **/
+ * g_array_insert_val() is a macro which uses a reference to the value
+ * parameter @v. This means that you cannot use it with literal values
+ * such as "27". You must use variables.
+ *
+ * Returns: the #GArray
+ */
 GArray*
 g_array_insert_vals (GArray        *farray,
-                    guint          index_,
-                    gconstpointer  data,
-                    guint          len)
+                     guint          index_,
+                     gconstpointer  data,
+                     guint          len)
 {
   GRealArray *array = (GRealArray*) farray;
 
@@ -451,9 +519,9 @@ g_array_insert_vals (GArray        *farray,
 
   g_array_maybe_expand (array, len);
 
-  g_memmove (g_array_elt_pos (array, len + index_), 
-            g_array_elt_pos (array, index_), 
-            g_array_elt_len (array, array->len - index_));
+  memmove (g_array_elt_pos (array, len + index_),
+           g_array_elt_pos (array, index_),
+           g_array_elt_len (array, array->len - index_));
 
   memcpy (g_array_elt_pos (array, index_), data, g_array_elt_len (array, len));
 
@@ -466,16 +534,17 @@ g_array_insert_vals (GArray        *farray,
 
 /**
  * g_array_set_size:
- * @array: a #GArray.
- * @length: the new size of the #GArray.
- * @Returns: the #GArray.
+ * @array: a #GArray
+ * @length: the new size of the #GArray
  *
  * Sets the size of the array, expanding it if necessary. If the array
  * was created with @clear_ set to %TRUE, the new elements are set to 0.
- **/
+ *
+ * Returns: the #GArray
+ */
 GArray*
 g_array_set_size (GArray *farray,
-                 guint   length)
+                  guint   length)
 {
   GRealArray *array = (GRealArray*) farray;
 
@@ -486,10 +555,10 @@ g_array_set_size (GArray *farray,
       g_array_maybe_expand (array, length - array->len);
       
       if (array->clear)
-       g_array_elt_zero (array, array->len, length - array->len);
+        g_array_elt_zero (array, array->len, length - array->len);
     }
-  else if (G_UNLIKELY (g_mem_gc_friendly) && length < array->len)
-    g_array_elt_zero (array, length, array->len - length);
+  else if (length < array->len)
+    g_array_remove_range (farray, length, array->len - length);
   
   array->len = length;
   
@@ -500,16 +569,17 @@ g_array_set_size (GArray *farray,
 
 /**
  * g_array_remove_index:
- * @array: a #GArray.
- * @index_: the index of the element to remove.
- * @Returns: the #GArray.
+ * @array: a #GArray
+ * @index_: the index of the element to remove
  *
  * Removes the element at the given index from a #GArray. The following
  * elements are moved down one place.
- **/
+ *
+ * Returns: the #GArray
+ */
 GArray*
 g_array_remove_index (GArray *farray,
-                     guint   index_)
+                      guint   index_)
 {
   GRealArray* array = (GRealArray*) farray;
 
@@ -517,11 +587,14 @@ g_array_remove_index (GArray *farray,
 
   g_return_val_if_fail (index_ < array->len, NULL);
 
+  if (array->clear_func != NULL)
+    array->clear_func (g_array_elt_pos (array, index_));
+
   if (index_ != array->len - 1)
-    g_memmove (g_array_elt_pos (array, index_),
-              g_array_elt_pos (array, index_ + 1),
-              g_array_elt_len (array, array->len - index_ - 1));
-  
+    memmove (g_array_elt_pos (array, index_),
+             g_array_elt_pos (array, index_ + 1),
+             g_array_elt_len (array, array->len - index_ - 1));
+
   array->len -= 1;
 
   if (G_UNLIKELY (g_mem_gc_friendly))
@@ -534,18 +607,19 @@ g_array_remove_index (GArray *farray,
 
 /**
  * g_array_remove_index_fast:
- * @array: a @GArray.
- * @index_: the index of the element to remove.
- * @Returns: the #GArray.
+ * @array: a @GArray
+ * @index_: the index of the element to remove
  *
  * Removes the element at the given index from a #GArray. The last
  * element in the array is used to fill in the space, so this function
  * does not preserve the order of the #GArray. But it is faster than
  * g_array_remove_index().
- **/
+ *
+ * Returns: the #GArray
+ */
 GArray*
 g_array_remove_index_fast (GArray *farray,
-                          guint   index_)
+                           guint   index_)
 {
   GRealArray* array = (GRealArray*) farray;
 
@@ -553,10 +627,13 @@ g_array_remove_index_fast (GArray *farray,
 
   g_return_val_if_fail (index_ < array->len, NULL);
 
+  if (array->clear_func != NULL)
+    array->clear_func (g_array_elt_pos (array, index_));
+
   if (index_ != array->len - 1)
-    memcpy (g_array_elt_pos (array, index_), 
-           g_array_elt_pos (array, array->len - 1),
-           g_array_elt_len (array, 1));
+    memcpy (g_array_elt_pos (array, index_),
+            g_array_elt_pos (array, array->len - 1),
+            g_array_elt_len (array, 1));
   
   array->len -= 1;
 
@@ -570,16 +647,17 @@ g_array_remove_index_fast (GArray *farray,
 
 /**
  * g_array_remove_range:
- * @array: a @GArray.
- * @index_: the index of the first element to remove.
- * @length: the number of elements to remove.
- * @Returns: the #GArray.
+ * @array: a @GArray
+ * @index_: the index of the first element to remove
+ * @length: the number of elements to remove
  *
  * Removes the given number of elements starting at the given index
  * from a #GArray.  The following elements are moved to close the gap.
  *
+ * Returns: the #GArray
+ *
  * Since: 2.4
- **/
+ */
 GArray*
 g_array_remove_range (GArray *farray,
                       guint   index_,
@@ -591,10 +669,18 @@ g_array_remove_range (GArray *farray,
   g_return_val_if_fail (index_ < array->len, NULL);
   g_return_val_if_fail (index_ + length <= array->len, NULL);
 
+  if (array->clear_func != NULL)
+    {
+      guint i;
+
+      for (i = 0; i < length; i++)
+        array->clear_func (g_array_elt_pos (array, index_ + i));
+    }
+
   if (index_ + length != array->len)
-    g_memmove (g_array_elt_pos (array, index_), 
-               g_array_elt_pos (array, index_ + length), 
-               (array->len - (index_ + length)) * array->elt_size);
+    memmove (g_array_elt_pos (array, index_),
+             g_array_elt_pos (array, index_ + length),
+             (array->len - (index_ + length)) * array->elt_size);
 
   array->len -= length;
   if (G_UNLIKELY (g_mem_gc_friendly))
@@ -607,57 +693,61 @@ g_array_remove_range (GArray *farray,
 
 /**
  * g_array_sort:
- * @array: a #GArray.
- * @compare_func: comparison function.
+ * @array: a #GArray
+ * @compare_func: comparison function
  *
  * Sorts a #GArray using @compare_func which should be a qsort()-style
  * comparison function (returns less than zero for first arg is less
  * than second arg, zero for equal, greater zero if first arg is
  * greater than second arg).
  *
- * If two array elements compare equal, their order in the sorted array
- * is undefined. If you want equal elements to keep their order &#8211; i.e.
- * you want a stable sort &#8211; you can write a comparison function that,
- * if two elements would otherwise compare equal, compares them by
- * their addresses.
- **/
+ * This is guaranteed to be a stable sort since version 2.32.
+ */
 void
 g_array_sort (GArray       *farray,
-             GCompareFunc  compare_func)
+              GCompareFunc  compare_func)
 {
   GRealArray *array = (GRealArray*) farray;
 
   g_return_if_fail (array != NULL);
 
-  qsort (array->data,
-        array->len,
-        array->elt_size,
-        compare_func);
+  /* Don't use qsort as we want a guaranteed stable sort */
+  g_qsort_with_data (array->data,
+                     array->len,
+                     array->elt_size,
+                     (GCompareDataFunc)compare_func,
+                     NULL);
 }
 
 /**
  * g_array_sort_with_data:
- * @array: a #GArray.
- * @compare_func: comparison function.
- * @user_data: data to pass to @compare_func.
+ * @array: a #GArray
+ * @compare_func: comparison function
+ * @user_data: data to pass to @compare_func
  *
  * Like g_array_sort(), but the comparison function receives an extra
  * user data argument.
- **/
+ *
+ * This is guaranteed to be a stable sort since version 2.32.
+ *
+ * There used to be a comment here about making the sort stable by
+ * using the addresses of the elements in the comparison function.
+ * This did not actually work, so any such code should be removed.
+ */
 void
 g_array_sort_with_data (GArray           *farray,
-                       GCompareDataFunc  compare_func,
-                       gpointer          user_data)
+                        GCompareDataFunc  compare_func,
+                        gpointer          user_data)
 {
   GRealArray *array = (GRealArray*) farray;
 
   g_return_if_fail (array != NULL);
 
   g_qsort_with_data (array->data,
-                    array->len,
-                    array->elt_size,
-                    compare_func,
-                    user_data);
+                     array->len,
+                     array->elt_size,
+                     compare_func,
+                     user_data);
 }
 
 /* Returns the smallest power of 2 greater than n, or n if
@@ -676,10 +766,10 @@ g_nearest_pow (gint num)
 
 static void
 g_array_maybe_expand (GRealArray *array,
-                     gint        len)
+                      gint        len)
 {
   guint want_alloc = g_array_elt_len (array, array->len + len + 
-                                     array->zero_terminated);
+                                      array->zero_terminated);
 
   if (want_alloc > array->alloc)
     {
@@ -699,16 +789,16 @@ g_array_maybe_expand (GRealArray *array,
  * SECTION:arrays_pointer
  * @title: Pointer Arrays
  * @short_description: arrays of pointers to any type of data, which
- *                     grow automatically as new elements are added
+ *     grow automatically as new elements are added
  *
  * Pointer Arrays are similar to Arrays but are used only for storing
  * pointers.
  *
- * <note><para>If you remove elements from the array, elements at the
- * end of the array are moved into the space previously occupied by the
- * removed element. This means that you should not rely on the index of
- * particular elements remaining the same. You should also be careful
- * when deleting elements while iterating over the array.</para></note>
+ * If you remove elements from the array, elements at the end of the
+ * array are moved into the space previously occupied by the removed
+ * element. This means that you should not rely on the index of particular
+ * elements remaining the same. You should also be careful when deleting
+ * elements while iterating over the array.
  *
  * To create a pointer array, use g_ptr_array_new().
  *
@@ -723,63 +813,68 @@ g_array_maybe_expand (GRealArray *array,
  *
  * To free a pointer array, use g_ptr_array_free().
  *
- * <example>
- *  <title>Using a #GPtrArray</title>
- *  <programlisting>
- *   GPtrArray *gparray;
- *   gchar *string1 = "one", *string2 = "two", *string3 = "three";
+ * An example using a #GPtrArray:
+ * |[<!-- language="C" -->
+ *   GPtrArray *array;
+ *   gchar *string1 = "one";
+ *   gchar *string2 = "two";
+ *   gchar *string3 = "three";
  *
- *   gparray = g_ptr_array_new (<!-- -->);
- *   g_ptr_array_add (gparray, (gpointer) string1);
- *   g_ptr_array_add (gparray, (gpointer) string2);
- *   g_ptr_array_add (gparray, (gpointer) string3);
+ *   array = g_ptr_array_new ();
+ *   g_ptr_array_add (array, (gpointer) string1);
+ *   g_ptr_array_add (array, (gpointer) string2);
+ *   g_ptr_array_add (array, (gpointer) string3);
  *
- *   if (g_ptr_array_index (gparray, 0) != (gpointer) string1)
- *     g_print ("ERROR: got &percnt;p instead of &percnt;p\n",
- *              g_ptr_array_index (gparray, 0), string1);
+ *   if (g_ptr_array_index (array, 0) != (gpointer) string1)
+ *     g_print ("ERROR: got %p instead of %p\n",
+ *              g_ptr_array_index (array, 0), string1);
  *
- *   g_ptr_array_free (gparray, TRUE);
- *  </programlisting>
- * </example>
- **/
+ *   g_ptr_array_free (array, TRUE);
+ * ]|
+ */
 
 typedef struct _GRealPtrArray  GRealPtrArray;
 
 /**
  * GPtrArray:
  * @pdata: points to the array of pointers, which may be moved when the
- *         array grows.
- * @len: number of pointers in the array.
+ *     array grows
+ * @len: number of pointers in the array
  *
  * Contains the public fields of a pointer array.
- **/
+ */
 struct _GRealPtrArray
 {
-  gpointer     *pdata;
-  guint         len;
-  guint         alloc;
-  gint          ref_count;
-  GDestroyNotify element_free_func;
+  gpointer       *pdata;
+  guint           len;
+  guint           alloc;
+  gint            ref_count;
+  GDestroyNotify  element_free_func;
 };
 
 /**
  * g_ptr_array_index:
- * @array: a #GPtrArray.
- * @index_: the index of the pointer to return.
- * @Returns: the pointer at the given index.
+ * @array: a #GPtrArray
+ * @index_: the index of the pointer to return
  *
  * Returns the pointer at the given index of the pointer array.
- **/
+ *
+ * This does not perform bounds checking on the given @index_,
+ * so you are responsible for checking it against the array length.
+ *
+ * Returns: the pointer at the given index
+ */
 
 static void g_ptr_array_maybe_expand (GRealPtrArray *array,
-                                     gint           len);
+                                      gint           len);
 
 /**
  * g_ptr_array_new:
- * @Returns: the new #GPtrArray.
  *
  * Creates a new #GPtrArray with a reference count of 1.
- **/
+ *
+ * Returns: the new #GPtrArray
+ */
 GPtrArray*
 g_ptr_array_new (void)
 {
@@ -788,18 +883,21 @@ g_ptr_array_new (void)
 
 /**
  * g_ptr_array_sized_new:
- * @reserved_size: number of pointers preallocated.
- * @Returns: the new #GPtrArray.
+ * @reserved_size: number of pointers preallocated
  *
  * Creates a new #GPtrArray with @reserved_size pointers preallocated
  * and a reference count of 1. This avoids frequent reallocation, if
  * you are going to add many pointers to the array. Note however that
  * the size of the array is still 0.
- **/
+ *
+ * Returns: the new #GPtrArray
+ */
 GPtrArray*  
 g_ptr_array_sized_new (guint reserved_size)
 {
-  GRealPtrArray *array = g_slice_new (GRealPtrArray);
+  GRealPtrArray *array;
+
+  array = g_slice_new (GRealPtrArray);
 
   array->pdata = NULL;
   array->len = 0;
@@ -815,45 +913,48 @@ g_ptr_array_sized_new (guint reserved_size)
 
 /**
  * g_ptr_array_new_with_free_func:
- * @element_free_func: A function to free elements with destroy @array or %NULL.
+ * @element_free_func: (allow-none): A function to free elements with
+ *     destroy @array or %NULL
  *
- * Creates a new #GPtrArray with a reference count of 1 and use @element_free_func
- * for freeing each element when the array is destroyed either via
- * g_ptr_array_unref(), when g_ptr_array_free() is called with @free_segment
- * set to %TRUE or when removing elements.
+ * Creates a new #GPtrArray with a reference count of 1 and use
+ * @element_free_func for freeing each element when the array is destroyed
+ * either via g_ptr_array_unref(), when g_ptr_array_free() is called with
+ * @free_segment set to %TRUE or when removing elements.
  *
- * Returns: A new #GPtrArray.
+ * Returns: A new #GPtrArray
  *
  * Since: 2.22
- **/
-GPtrArray *
+ */
+GPtrArray*
 g_ptr_array_new_with_free_func (GDestroyNotify element_free_func)
 {
   GPtrArray *array;
 
   array = g_ptr_array_new ();
   g_ptr_array_set_free_func (array, element_free_func);
+
   return array;
 }
 
 /**
  * g_ptr_array_new_full:
- * @reserved_size: number of pointers preallocated.
- * @element_free_func: A function to free elements with destroy @array or %NULL.
+ * @reserved_size: number of pointers preallocated
+ * @element_free_func: (allow-none): A function to free elements with
+ *     destroy @array or %NULL
  *
  * Creates a new #GPtrArray with @reserved_size pointers preallocated
  * and a reference count of 1. This avoids frequent reallocation, if
  * you are going to add many pointers to the array. Note however that
  * the size of the array is still 0. It also set @element_free_func
  * for freeing each element when the array is destroyed either via
- * g_ptr_array_unref(), when g_ptr_array_free() is called with @free_segment
- * set to %TRUE or when removing elements.
+ * g_ptr_array_unref(), when g_ptr_array_free() is called with
+ * @free_segment set to %TRUE or when removing elements.
  *
- * Returns: A new #GPtrArray.
+ * Returns: A new #GPtrArray
  *
  * Since: 2.30
- **/
-GPtrArray *
+ */
+GPtrArray*
 g_ptr_array_new_full (guint          reserved_size,
                       GDestroyNotify element_free_func)
 {
@@ -861,25 +962,27 @@ g_ptr_array_new_full (guint          reserved_size,
 
   array = g_ptr_array_sized_new (reserved_size);
   g_ptr_array_set_free_func (array, element_free_func);
+
   return array;
 }
 
 /**
  * g_ptr_array_set_free_func:
- * @array: A #GPtrArray.
- * @element_free_func: A function to free elements with destroy @array or %NULL.
+ * @array: A #GPtrArray
+ * @element_free_func: (allow-none): A function to free elements with
+ *     destroy @array or %NULL
  *
  * Sets a function for freeing each element when @array is destroyed
  * either via g_ptr_array_unref(), when g_ptr_array_free() is called
  * with @free_segment set to %TRUE or when removing elements.
  *
  * Since: 2.22
- **/
+ */
 void
-g_ptr_array_set_free_func (GPtrArray        *array,
-                           GDestroyNotify    element_free_func)
+g_ptr_array_set_free_func (GPtrArray      *array,
+                           GDestroyNotify  element_free_func)
 {
-  GRealPtrArray* rarray = (GRealPtrArray*) array;
+  GRealPtrArray *rarray = (GRealPtrArray *)array;
 
   g_return_if_fail (array);
 
@@ -888,19 +991,19 @@ g_ptr_array_set_free_func (GPtrArray        *array,
 
 /**
  * g_ptr_array_ref:
- * @array: A #GArray.
+ * @array: a #GPtrArray
  *
- * Atomically increments the reference count of @array by one. This
- * function is MT-safe and may be called from any thread.
+ * Atomically increments the reference count of @array by one.
+ * This function is thread-safe and may be called from any thread.
  *
- * Returns: The passed in #GPtrArray.
+ * Returns: The passed in #GPtrArray
  *
  * Since: 2.22
- **/
-GPtrArray *
+ */
+GPtrArray*
 g_ptr_array_ref (GPtrArray *array)
 {
-  GRealPtrArray *rarray = (GRealPtrArray*) array;
+  GRealPtrArray *rarray = (GRealPtrArray *)array;
 
   g_return_val_if_fail (array, NULL);
 
@@ -909,9 +1012,11 @@ g_ptr_array_ref (GPtrArray *array)
   return array;
 }
 
+static gpointer *ptr_array_free (GPtrArray *, ArrayFreeFlags);
+
 /**
  * g_ptr_array_unref:
- * @array: A #GPtrArray.
+ * @array: A #GPtrArray
  *
  * Atomically decrements the reference count of @array by one. If the
  * reference count drops to 0, the effect is the same as calling
@@ -919,23 +1024,22 @@ g_ptr_array_ref (GPtrArray *array)
  * is MT-safe and may be called from any thread.
  *
  * Since: 2.22
- **/
+ */
 void
 g_ptr_array_unref (GPtrArray *array)
 {
-  GRealPtrArray *rarray = (GRealPtrArray*) array;
+  GRealPtrArray *rarray = (GRealPtrArray *)array;
+
   g_return_if_fail (array);
 
   if (g_atomic_int_dec_and_test (&rarray->ref_count))
-    g_ptr_array_free (array, TRUE);
+    ptr_array_free (array, FREE_SEGMENT);
 }
 
 /**
  * g_ptr_array_free:
- * @array: a #GPtrArray.
- * @free_seg: if %TRUE the actual pointer array is freed as well.
- * @Returns: the pointer array if @free_seg is %FALSE, otherwise %NULL.
- *           The pointer array should be freed using g_free().
+ * @array: a #GPtrArray
+ * @free_seg: if %TRUE the actual pointer array is freed as well
  *
  * Frees the memory allocated for the #GPtrArray. If @free_seg is %TRUE
  * it frees the memory block holding the elements as well. Pass %FALSE
@@ -944,44 +1048,59 @@ g_ptr_array_unref (GPtrArray *array)
  * is greater than one, the #GPtrArray wrapper is preserved but the
  * size of @array will be set to zero.
  *
- * <note><para>If array contents point to dynamically-allocated
- * memory, they should be freed separately if @free_seg is %TRUE and no
- * #GDestroyNotify function has been set for @array.</para></note>
- **/
+ * If array contents point to dynamically-allocated memory, they should
+ * be freed separately if @free_seg is %TRUE and no #GDestroyNotify
+ * function has been set for @array.
+ *
+ * Returns: the pointer array if @free_seg is %FALSE, otherwise %NULL.
+ *     The pointer array should be freed using g_free().
+ */
 gpointer*
-g_ptr_array_free (GPtrArray *farray,
-                 gboolean   free_segment)
+g_ptr_array_free (GPtrArray *array,
+                  gboolean   free_segment)
 {
-  GRealPtrArray *array = (GRealPtrArray*) farray;
-  gpointer* segment;
-  gboolean preserve_wrapper;
+  GRealPtrArray *rarray = (GRealPtrArray *)array;
+  ArrayFreeFlags flags;
 
-  g_return_val_if_fail (array, NULL);
+  g_return_val_if_fail (rarray, NULL);
 
-  /* if others are holding a reference, preserve the wrapper but do free/return the data */
-  preserve_wrapper = FALSE;
-  if (g_atomic_int_get (&array->ref_count) > 1)
-    preserve_wrapper = TRUE;
+  flags = (free_segment ? FREE_SEGMENT : 0);
 
-  if (free_segment)
+  /* if others are holding a reference, preserve the wrapper but
+   * do free/return the data
+   */
+  if (!g_atomic_int_dec_and_test (&rarray->ref_count))
+    flags |= PRESERVE_WRAPPER;
+
+  return ptr_array_free (array, flags);
+}
+
+static gpointer *
+ptr_array_free (GPtrArray      *array,
+                ArrayFreeFlags  flags)
+{
+  GRealPtrArray *rarray = (GRealPtrArray *)array;
+  gpointer *segment;
+
+  if (flags & FREE_SEGMENT)
     {
-      if (array->element_free_func != NULL)
-        g_ptr_array_foreach (farray, (GFunc) array->element_free_func, NULL);
-      g_free (array->pdata);
+      if (rarray->element_free_func != NULL)
+        g_ptr_array_foreach (array, (GFunc) rarray->element_free_func, NULL);
+      g_free (rarray->pdata);
       segment = NULL;
     }
   else
-    segment = array->pdata;
+    segment = rarray->pdata;
 
-  if (preserve_wrapper)
+  if (flags & PRESERVE_WRAPPER)
     {
-      array->pdata = NULL;
-      array->len = 0;
-      array->alloc = 0;
+      rarray->pdata = NULL;
+      rarray->len = 0;
+      rarray->alloc = 0;
     }
   else
     {
-      g_slice_free1 (sizeof (GRealPtrArray), array);
+      g_slice_free1 (sizeof (GRealPtrArray), rarray);
     }
 
   return segment;
@@ -989,7 +1108,7 @@ g_ptr_array_free (GPtrArray *farray,
 
 static void
 g_ptr_array_maybe_expand (GRealPtrArray *array,
-                         gint           len)
+                          gint           len)
 {
   if ((array->len + len) > array->alloc)
     {
@@ -1005,172 +1124,176 @@ g_ptr_array_maybe_expand (GRealPtrArray *array,
 
 /**
  * g_ptr_array_set_size:
- * @array: a #GPtrArray.
- * @length: the new length of the pointer array.
+ * @array: a #GPtrArray
+ * @length: the new length of the pointer array
  *
  * Sets the size of the array. When making the array larger,
  * newly-added elements will be set to %NULL. When making it smaller,
  * if @array has a non-%NULL #GDestroyNotify function then it will be
  * called for the removed elements.
- **/
+ */
 void
-g_ptr_array_set_size  (GPtrArray *farray,
-                      gint       length)
+g_ptr_array_set_size  (GPtrArray *array,
+                       gint       length)
 {
-  GRealPtrArray* array = (GRealPtrArray*) farray;
+  GRealPtrArray *rarray = (GRealPtrArray *)array;
 
-  g_return_if_fail (array);
+  g_return_if_fail (rarray);
 
-  if (length > array->len)
+  if (length > rarray->len)
     {
       int i;
-      g_ptr_array_maybe_expand (array, (length - array->len));
+      g_ptr_array_maybe_expand (rarray, (length - rarray->len));
       /* This is not 
        *     memset (array->pdata + array->len, 0,
        *            sizeof (gpointer) * (length - array->len));
        * to make it really portable. Remember (void*)NULL needn't be
        * bitwise zero. It of course is silly not to use memset (..,0,..).
        */
-      for (i = array->len; i < length; i++)
-       array->pdata[i] = NULL;
+      for (i = rarray->len; i < length; i++)
+        rarray->pdata[i] = NULL;
     }
-  else if (length < array->len)
-    g_ptr_array_remove_range (farray, length, array->len - length);
+  else if (length < rarray->len)
+    g_ptr_array_remove_range (array, length, rarray->len - length);
 
-  array->len = length;
+  rarray->len = length;
 }
 
 /**
  * g_ptr_array_remove_index:
- * @array: a #GPtrArray.
- * @index_: the index of the pointer to remove.
- * @Returns: the pointer which was removed.
+ * @array: a #GPtrArray
+ * @index_: the index of the pointer to remove
  *
- * Removes the pointer at the given index from the pointer array. The
- * following elements are moved down one place. If @array has a
- * non-%NULL #GDestroyNotify function it is called for the removed
+ * Removes the pointer at the given index from the pointer array.
+ * The following elements are moved down one place. If @array has
+ * non-%NULL #GDestroyNotify function it is called for the removed
  * element.
- **/
+ *
+ * Returns: the pointer which was removed
+ */
 gpointer
-g_ptr_array_remove_index (GPtrArray *farray,
-                         guint      index_)
+g_ptr_array_remove_index (GPtrArray *array,
+                          guint      index_)
 {
-  GRealPtrArray* array = (GRealPtrArray*) farray;
+  GRealPtrArray *rarray = (GRealPtrArray *)array;
   gpointer result;
 
-  g_return_val_if_fail (array, NULL);
+  g_return_val_if_fail (rarray, NULL);
 
-  g_return_val_if_fail (index_ < array->len, NULL);
+  g_return_val_if_fail (index_ < rarray->len, NULL);
 
-  result = array->pdata[index_];
+  result = rarray->pdata[index_];
   
-  if (array->element_free_func != NULL)
-    array->element_free_func (array->pdata[index_]);
+  if (rarray->element_free_func != NULL)
+    rarray->element_free_func (rarray->pdata[index_]);
 
-  if (index_ != array->len - 1)
-    g_memmove (array->pdata + index_, array->pdata + index_ + 1, 
-               sizeof (gpointer) * (array->len - index_ - 1));
+  if (index_ != rarray->len - 1)
+    memmove (rarray->pdata + index_, rarray->pdata + index_ + 1,
+             sizeof (gpointer) * (rarray->len - index_ - 1));
   
-  array->len -= 1;
+  rarray->len -= 1;
 
   if (G_UNLIKELY (g_mem_gc_friendly))
-    array->pdata[array->len] = NULL;
+    rarray->pdata[rarray->len] = NULL;
 
   return result;
 }
 
 /**
  * g_ptr_array_remove_index_fast:
- * @array: a #GPtrArray.
- * @index_: the index of the pointer to remove.
- * @Returns: the pointer which was removed.
- *
- * Removes the pointer at the given index from the pointer array. The
- * last element in the array is used to fill in the space, so this
- * function does not preserve the order of the array. But it is faster
- * than g_ptr_array_remove_index(). If @array has a non-%NULL
+ * @array: a #GPtrArray
+ * @index_: the index of the pointer to remove
+ *
+ * Removes the pointer at the given index from the pointer array.
+ * The last element in the array is used to fill in the space, so
+ * this function does not preserve the order of the array. But it
+ * is faster than g_ptr_array_remove_index(). If @array has a non-%NULL
  * #GDestroyNotify function it is called for the removed element.
- **/
+ *
+ * Returns: the pointer which was removed
+ */
 gpointer
-g_ptr_array_remove_index_fast (GPtrArray *farray,
-                              guint      index_)
+g_ptr_array_remove_index_fast (GPtrArray *array,
+                               guint      index_)
 {
-  GRealPtrArray* array = (GRealPtrArray*) farray;
+  GRealPtrArray *rarray = (GRealPtrArray *)array;
   gpointer result;
 
-  g_return_val_if_fail (array, NULL);
+  g_return_val_if_fail (rarray, NULL);
 
-  g_return_val_if_fail (index_ < array->len, NULL);
+  g_return_val_if_fail (index_ < rarray->len, NULL);
 
-  result = array->pdata[index_];
+  result = rarray->pdata[index_];
 
-  if (array->element_free_func != NULL)
-    array->element_free_func (array->pdata[index_]);
+  if (rarray->element_free_func != NULL)
+    rarray->element_free_func (rarray->pdata[index_]);
 
-  if (index_ != array->len - 1)
-    array->pdata[index_] = array->pdata[array->len - 1];
+  if (index_ != rarray->len - 1)
+    rarray->pdata[index_] = rarray->pdata[rarray->len - 1];
 
-  array->len -= 1;
+  rarray->len -= 1;
 
   if (G_UNLIKELY (g_mem_gc_friendly))
-    array->pdata[array->len] = NULL;
+    rarray->pdata[rarray->len] = NULL;
 
   return result;
 }
 
 /**
  * g_ptr_array_remove_range:
- * @array: a @GPtrArray.
- * @index_: the index of the first pointer to remove.
- * @length: the number of pointers to remove.
+ * @array: a @GPtrArray
+ * @index_: the index of the first pointer to remove
+ * @length: the number of pointers to remove
  *
  * Removes the given number of pointers starting at the given index
- * from a #GPtrArray.  The following elements are moved to close the
- * gap. If @array has a non-%NULL #GDestroyNotify function it is called
- * for the removed elements.
+ * from a #GPtrArray. The following elements are moved to close the
+ * gap. If @array has a non-%NULL #GDestroyNotify function it is
+ * called for the removed elements.
+ *
+ * Returns: the @array
  *
  * Since: 2.4
- **/
-void
-g_ptr_array_remove_range (GPtrArray *farray,
+ */
+GPtrArray*
+g_ptr_array_remove_range (GPtrArray *array,
                           guint      index_,
                           guint      length)
 {
-  GRealPtrArray* array = (GRealPtrArray*) farray;
+  GRealPtrArray *rarray = (GRealPtrArray *)array;
   guint n;
 
-  g_return_if_fail (array);
-  g_return_if_fail (index_ < array->len);
-  g_return_if_fail (index_ + length <= array->len);
+  g_return_val_if_fail (rarray != NULL, NULL);
+  g_return_val_if_fail (index_ < rarray->len, NULL);
+  g_return_val_if_fail (index_ + length <= rarray->len, NULL);
 
-  if (array->element_free_func != NULL)
+  if (rarray->element_free_func != NULL)
     {
       for (n = index_; n < index_ + length; n++)
-        array->element_free_func (array->pdata[n]);
+        rarray->element_free_func (rarray->pdata[n]);
     }
 
-  if (index_ + length != array->len)
+  if (index_ + length != rarray->len)
     {
-      g_memmove (&array->pdata[index_],
-                 &array->pdata[index_ + length], 
-                 (array->len - (index_ + length)) * sizeof (gpointer));
+      memmove (&rarray->pdata[index_],
+               &rarray->pdata[index_ + length],
+               (rarray->len - (index_ + length)) * sizeof (gpointer));
     }
 
-  array->len -= length;
+  rarray->len -= length;
   if (G_UNLIKELY (g_mem_gc_friendly))
     {
       guint i;
       for (i = 0; i < length; i++)
-        array->pdata[array->len + i] = NULL;
+        rarray->pdata[rarray->len + i] = NULL;
     }
+
+  return array;
 }
 
 /**
  * g_ptr_array_remove:
- * @array: a #GPtrArray.
- * @data: the pointer to remove.
- * @Returns: %TRUE if the pointer is removed. %FALSE if the pointer is
- *           not found in the array.
+ * @array: a #GPtrArray
+ * @data: the pointer to remove
  *
  * Removes the first occurrence of the given pointer from the pointer
  * array. The following elements are moved down one place. If @array
@@ -1179,12 +1302,14 @@ g_ptr_array_remove_range (GPtrArray *farray,
  *
  * It returns %TRUE if the pointer was removed, or %FALSE if the
  * pointer was not found.
- **/
+ *
+ * Returns: %TRUE if the pointer is removed, %FALSE if the pointer
+ *     is not found in the array
+ */
 gboolean
-g_ptr_array_remove (GPtrArray *farray,
-                   gpointer   data)
+g_ptr_array_remove (GPtrArray *array,
+                    gpointer   data)
 {
-  GRealPtrArray* array = (GRealPtrArray*) farray;
   guint i;
 
   g_return_val_if_fail (array, FALSE);
@@ -1192,10 +1317,10 @@ g_ptr_array_remove (GPtrArray *farray,
   for (i = 0; i < array->len; i += 1)
     {
       if (array->pdata[i] == data)
-       {
-         g_ptr_array_remove_index (farray, i);
-         return TRUE;
-       }
+        {
+          g_ptr_array_remove_index (array, i);
+          return TRUE;
+        }
     }
 
   return FALSE;
@@ -1203,35 +1328,36 @@ g_ptr_array_remove (GPtrArray *farray,
 
 /**
  * g_ptr_array_remove_fast:
- * @array: a #GPtrArray.
- * @data: the pointer to remove.
- * @Returns: %TRUE if the pointer was found in the array.
+ * @array: a #GPtrArray
+ * @data: the pointer to remove
  *
  * Removes the first occurrence of the given pointer from the pointer
  * array. The last element in the array is used to fill in the space,
- * so this function does not preserve the order of the array. But it is
- * faster than g_ptr_array_remove(). If @array has a non-%NULL
+ * so this function does not preserve the order of the array. But it
+ * is faster than g_ptr_array_remove(). If @array has a non-%NULL
  * #GDestroyNotify function it is called for the removed element.
  *
  * It returns %TRUE if the pointer was removed, or %FALSE if the
  * pointer was not found.
- **/
+ *
+ * Returns: %TRUE if the pointer was found in the array
+ */
 gboolean
-g_ptr_array_remove_fast (GPtrArray *farray,
-                        gpointer   data)
+g_ptr_array_remove_fast (GPtrArray *array,
+                         gpointer   data)
 {
-  GRealPtrArray* array = (GRealPtrArray*) farray;
+  GRealPtrArray *rarray = (GRealPtrArray *)array;
   guint i;
 
-  g_return_val_if_fail (array, FALSE);
+  g_return_val_if_fail (rarray, FALSE);
 
-  for (i = 0; i < array->len; i += 1)
+  for (i = 0; i < rarray->len; i += 1)
     {
-      if (array->pdata[i] == data)
-       {
-         g_ptr_array_remove_index_fast (farray, i);
-         return TRUE;
-       }
+      if (rarray->pdata[i] == data)
+        {
+          g_ptr_array_remove_index_fast (array, i);
+          return TRUE;
+        }
     }
 
   return FALSE;
@@ -1239,82 +1365,118 @@ g_ptr_array_remove_fast (GPtrArray *farray,
 
 /**
  * g_ptr_array_add:
- * @array: a #GPtrArray.
- * @data: the pointer to add.
+ * @array: a #GPtrArray
+ * @data: the pointer to add
  *
  * Adds a pointer to the end of the pointer array. The array will grow
  * in size automatically if necessary.
- **/
+ */
 void
-g_ptr_array_add (GPtrArray *farray,
-                gpointer   data)
+g_ptr_array_add (GPtrArray *array,
+                 gpointer   data)
 {
-  GRealPtrArray* array = (GRealPtrArray*) farray;
+  GRealPtrArray *rarray = (GRealPtrArray *)array;
 
-  g_return_if_fail (array);
+  g_return_if_fail (rarray);
 
-  g_ptr_array_maybe_expand (array, 1);
+  g_ptr_array_maybe_expand (rarray, 1);
 
-  array->pdata[array->len++] = data;
+  rarray->pdata[rarray->len++] = data;
+}
+
+/**
+ * g_ptr_array_insert:
+ * @array: a #GPtrArray
+ * @index_: the index to place the new element at, or -1 to append
+ * @data: the pointer to add.
+ *
+ * Inserts an element into the pointer array at the given index. The 
+ * array will grow in size automatically if necessary.
+ *
+ * Since: 2.40
+ */
+void
+g_ptr_array_insert (GPtrArray *array,
+                    gint       index_,
+                    gpointer   data)
+{
+  GRealPtrArray *rarray = (GRealPtrArray *)array;
+
+  g_return_if_fail (rarray);
+  g_return_if_fail (index_ >= -1);
+  g_return_if_fail (index_ <= (gint)rarray->len);
+
+  g_ptr_array_maybe_expand (rarray, 1);
+
+  if (index_ < 0)
+    index_ = rarray->len;
+
+  if (index_ < rarray->len)
+    memmove (&(rarray->pdata[index_ + 1]),
+             &(rarray->pdata[index_]),
+             (rarray->len - index_) * sizeof (gpointer));
+
+  rarray->len++;
+  rarray->pdata[index_] = data;
 }
 
 /**
  * g_ptr_array_sort:
- * @array: a #GPtrArray.
- * @compare_func: comparison function.
+ * @array: a #GPtrArray
+ * @compare_func: comparison function
  *
  * Sorts the array, using @compare_func which should be a qsort()-style
  * comparison function (returns less than zero for first arg is less
  * than second arg, zero for equal, greater than zero if irst arg is
  * greater than second arg).
  *
- * If two array elements compare equal, their order in the sorted array
- * is undefined. If you want equal elements to keep their order &#8211; i.e.
- * you want a stable sort &#8211; you can write a comparison function that,
- * if two elements would otherwise compare equal, compares them by
- * their addresses.
- *
- * <note><para>The comparison function for g_ptr_array_sort() doesn't
+ * Note that the comparison function for g_ptr_array_sort() doesn't
  * take the pointers from the array as arguments, it takes pointers to
- * the pointers in the array.</para></note>
- **/
+ * the pointers in the array.
+ *
+ * This is guaranteed to be a stable sort since version 2.32.
+ */
 void
 g_ptr_array_sort (GPtrArray    *array,
-                 GCompareFunc  compare_func)
+                  GCompareFunc  compare_func)
 {
   g_return_if_fail (array != NULL);
 
-  qsort (array->pdata,
-        array->len,
-        sizeof (gpointer),
-        compare_func);
+  /* Don't use qsort as we want a guaranteed stable sort */
+  g_qsort_with_data (array->pdata,
+                     array->len,
+                     sizeof (gpointer),
+                     (GCompareDataFunc)compare_func,
+                     NULL);
 }
 
 /**
  * g_ptr_array_sort_with_data:
- * @array: a #GPtrArray.
- * @compare_func: comparison function.
- * @user_data: data to pass to @compare_func.
+ * @array: a #GPtrArray
+ * @compare_func: comparison function
+ * @user_data: data to pass to @compare_func
  *
  * Like g_ptr_array_sort(), but the comparison function has an extra
  * user data argument.
  *
- * <note><para>The comparison function for g_ptr_array_sort_with_data()
+ * Note that the comparison function for g_ptr_array_sort_with_data()
  * doesn't take the pointers from the array as arguments, it takes
- * pointers to the pointers in the array.</para></note>
- **/
+ * pointers to the pointers in the array.
+ *
+ * This is guaranteed to be a stable sort since version 2.32.
+ */
 void
 g_ptr_array_sort_with_data (GPtrArray        *array,
-                           GCompareDataFunc  compare_func,
-                           gpointer          user_data)
+                            GCompareDataFunc  compare_func,
+                            gpointer          user_data)
 {
   g_return_if_fail (array != NULL);
 
   g_qsort_with_data (array->pdata,
-                    array->len,
-                    sizeof (gpointer),
-                    compare_func,
-                    user_data);
+                     array->len,
+                     sizeof (gpointer),
+                     compare_func,
+                     user_data);
 }
 
 /**
@@ -1326,7 +1488,7 @@ g_ptr_array_sort_with_data (GPtrArray        *array,
  * Calls a function for each element of a #GPtrArray.
  *
  * Since: 2.4
- **/
+ */
 void
 g_ptr_array_foreach (GPtrArray *array,
                      GFunc      func,
@@ -1343,32 +1505,28 @@ g_ptr_array_foreach (GPtrArray *array,
 /**
  * SECTION:arrays_byte
  * @title: Byte Arrays
- * @short_description: arrays of bytes, which grow automatically as
- *                     elements are added
- *
- * #GByteArray is based on #GArray, to provide arrays of bytes which
- * grow automatically as elements are added.
+ * @short_description: arrays of bytes
  *
- * To create a new #GByteArray use g_byte_array_new().
+ * #GByteArray is a mutable array of bytes based on #GArray, to provide arrays
+ * of bytes which grow automatically as elements are added.
  *
- * To add elements to a #GByteArray, use g_byte_array_append(), and
- * g_byte_array_prepend().
+ * To create a new #GByteArray use g_byte_array_new(). To add elements to a
+ * #GByteArray, use g_byte_array_append(), and g_byte_array_prepend().
  *
  * To set the size of a #GByteArray, use g_byte_array_set_size().
  *
  * To free a #GByteArray, use g_byte_array_free().
  *
- * <example>
- *  <title>Using a #GByteArray</title>
- *  <programlisting>
+ * An example for using a #GByteArray:
+ * |[<!-- language="C" -->
  *   GByteArray *gbarray;
  *   gint i;
  *
- *   gbarray = g_byte_array_new (<!-- -->);
- *   for (i = 0; i &lt; 10000; i++)
+ *   gbarray = g_byte_array_new ();
+ *   for (i = 0; i < 10000; i++)
  *     g_byte_array_append (gbarray, (guint8*) "abcd", 4);
  *
- *   for (i = 0; i &lt; 10000; i++)
+ *   for (i = 0; i < 10000; i++)
  *     {
  *       g_assert (gbarray->data[4*i] == 'a');
  *       g_assert (gbarray->data[4*i+1] == 'b');
@@ -1377,200 +1535,278 @@ g_ptr_array_foreach (GPtrArray *array,
  *     }
  *
  *   g_byte_array_free (gbarray, TRUE);
- *  </programlisting>
- * </example>
- **/
+ * ]|
+ *
+ * See #GBytes if you are interested in an immutable object representing a
+ * sequence of bytes.
+ */
 
 /**
  * GByteArray:
  * @data: a pointer to the element data. The data may be moved as
- *        elements are added to the #GByteArray.
- * @len: the number of elements in the #GByteArray.
+ *     elements are added to the #GByteArray
+ * @len: the number of elements in the #GByteArray
  *
- * The <structname>GByteArray</structname> struct allows access to the
- * public fields of a <structname>GByteArray</structname>.
- **/
+ * Contains the public fields of a GByteArray.
+ */
 
 /**
  * g_byte_array_new:
- * @Returns: the new #GByteArray.
  *
  * Creates a new #GByteArray with a reference count of 1.
- **/
-GByteArray* g_byte_array_new (void)
+ *
+ * Returns: (transfer full): the new #GByteArray
+ */
+GByteArray*
+g_byte_array_new (void)
+{
+  return (GByteArray *)g_array_sized_new (FALSE, FALSE, 1, 0);
+}
+
+/**
+ * g_byte_array_new_take:
+ * @data: (transfer full) (array length=len): byte data for the array
+ * @len: length of @data
+ *
+ * Create byte array containing the data. The data will be owned by the array
+ * and will be freed with g_free(), i.e. it could be allocated using g_strdup().
+ *
+ * Since: 2.32
+ *
+ * Returns: (transfer full): a new #GByteArray
+ */
+GByteArray*
+g_byte_array_new_take (guint8 *data,
+                       gsize   len)
 {
-  return (GByteArray*) g_array_sized_new (FALSE, FALSE, 1, 0);
+  GByteArray *array;
+  GRealArray *real;
+
+  array = g_byte_array_new ();
+  real = (GRealArray *)array;
+  g_assert (real->data == NULL);
+  g_assert (real->len == 0);
+
+  real->data = data;
+  real->len = len;
+  real->alloc = len;
+
+  return array;
 }
 
 /**
  * g_byte_array_sized_new:
- * @reserved_size: number of bytes preallocated.
- * @Returns: the new #GByteArray.
+ * @reserved_size: number of bytes preallocated
  *
  * Creates a new #GByteArray with @reserved_size bytes preallocated.
  * This avoids frequent reallocation, if you are going to add many
  * bytes to the array. Note however that the size of the array is still
  * 0.
- **/
-GByteArray* g_byte_array_sized_new (guint reserved_size)
+ *
+ * Returns: the new #GByteArray
+ */
+GByteArray*
+g_byte_array_sized_new (guint reserved_size)
 {
-  return (GByteArray*) g_array_sized_new (FALSE, FALSE, 1, reserved_size);
+  return (GByteArray *)g_array_sized_new (FALSE, FALSE, 1, reserved_size);
 }
 
 /**
  * g_byte_array_free:
- * @array: a #GByteArray.
- * @free_segment: if %TRUE the actual byte data is freed as well.
- * @Returns: the element data if @free_segment is %FALSE, otherwise
- *           %NULL.  The element data should be freed using g_free().
+ * @array: a #GByteArray
+ * @free_segment: if %TRUE the actual byte data is freed as well
  *
  * Frees the memory allocated by the #GByteArray. If @free_segment is
  * %TRUE it frees the actual byte data. If the reference count of
  * @array is greater than one, the #GByteArray wrapper is preserved but
  * the size of @array will be set to zero.
- **/
-guint8*            g_byte_array_free     (GByteArray *array,
-                                  gboolean    free_segment)
+ *
+ * Returns: the element data if @free_segment is %FALSE, otherwise
+ *          %NULL.  The element data should be freed using g_free().
+ */
+guint8*
+g_byte_array_free (GByteArray *array,
+                   gboolean    free_segment)
 {
-  return (guint8*) g_array_free ((GArray*) array, free_segment);
+  return (guint8 *)g_array_free ((GArray *)array, free_segment);
+}
+
+/**
+ * g_byte_array_free_to_bytes:
+ * @array: (transfer full): a #GByteArray
+ *
+ * Transfers the data from the #GByteArray into a new immutable #GBytes.
+ *
+ * The #GByteArray is freed unless the reference count of @array is greater
+ * than one, the #GByteArray wrapper is preserved but the size of @array
+ * will be set to zero.
+ *
+ * This is identical to using g_bytes_new_take() and g_byte_array_free()
+ * together.
+ *
+ * Since: 2.32
+ *
+ * Returns: (transfer full): a new immutable #GBytes representing same
+ *     byte data that was in the array
+ */
+GBytes*
+g_byte_array_free_to_bytes (GByteArray *array)
+{
+  gsize length;
+
+  g_return_val_if_fail (array != NULL, NULL);
+
+  length = array->len;
+  return g_bytes_new_take (g_byte_array_free (array, FALSE), length);
 }
 
 /**
  * g_byte_array_ref:
- * @array: A #GByteArray.
+ * @array: A #GByteArray
  *
- * Atomically increments the reference count of @array by one. This
- * function is MT-safe and may be called from any thread.
+ * Atomically increments the reference count of @array by one.
+ * This function is thread-safe and may be called from any thread.
  *
- * Returns: The passed in #GByteArray.
+ * Returns: The passed in #GByteArray
  *
  * Since: 2.22
- **/
-GByteArray *
+ */
+GByteArray*
 g_byte_array_ref (GByteArray *array)
 {
-  return (GByteArray *) g_array_ref ((GArray *) array);
+  return (GByteArray *)g_array_ref ((GArray *)array);
 }
 
 /**
  * g_byte_array_unref:
- * @array: A #GByteArray.
+ * @array: A #GByteArray
  *
  * Atomically decrements the reference count of @array by one. If the
  * reference count drops to 0, all memory allocated by the array is
- * released. This function is MT-safe and may be called from any
+ * released. This function is thread-safe and may be called from any
  * thread.
  *
  * Since: 2.22
- **/
+ */
 void
 g_byte_array_unref (GByteArray *array)
 {
-  g_array_unref ((GArray *) array);
+  g_array_unref ((GArray *)array);
 }
 
 /**
  * g_byte_array_append:
- * @array: a #GByteArray.
- * @data: the byte data to be added.
- * @len: the number of bytes to add.
- * @Returns: the #GByteArray.
+ * @array: a #GByteArray
+ * @data: the byte data to be added
+ * @len: the number of bytes to add
  *
- * Adds the given bytes to the end of the #GByteArray. The array will
- * grow in size automatically if necessary.
- **/
-GByteArray* g_byte_array_append   (GByteArray   *array,
-                                  const guint8 *data,
-                                  guint         len)
+ * Adds the given bytes to the end of the #GByteArray.
+ * The array will grow in size automatically if necessary.
+ *
+ * Returns: the #GByteArray
+ */
+GByteArray*
+g_byte_array_append (GByteArray   *array,
+                     const guint8 *data,
+                     guint         len)
 {
-  g_array_append_vals ((GArray*) array, (guint8*)data, len);
+  g_array_append_vals ((GArray *)array, (guint8 *)data, len);
 
   return array;
 }
 
 /**
  * g_byte_array_prepend:
- * @array: a #GByteArray.
- * @data: the byte data to be added.
- * @len: the number of bytes to add.
- * @Returns: the #GByteArray.
+ * @array: a #GByteArray
+ * @data: the byte data to be added
+ * @len: the number of bytes to add
  *
- * Adds the given data to the start of the #GByteArray. The array will
- * grow in size automatically if necessary.
- **/
-GByteArray* g_byte_array_prepend  (GByteArray   *array,
-                                  const guint8 *data,
-                                  guint         len)
+ * Adds the given data to the start of the #GByteArray.
+ * The array will grow in size automatically if necessary.
+ *
+ * Returns: the #GByteArray
+ */
+GByteArray*
+g_byte_array_prepend (GByteArray   *array,
+                      const guint8 *data,
+                      guint         len)
 {
-  g_array_prepend_vals ((GArray*) array, (guint8*)data, len);
+  g_array_prepend_vals ((GArray *)array, (guint8 *)data, len);
 
   return array;
 }
 
 /**
  * g_byte_array_set_size:
- * @array: a #GByteArray.
- * @length: the new size of the #GByteArray.
- * @Returns: the #GByteArray.
+ * @array: a #GByteArray
+ * @length: the new size of the #GByteArray
  *
  * Sets the size of the #GByteArray, expanding it if necessary.
- **/
-GByteArray* g_byte_array_set_size (GByteArray *array,
-                                  guint       length)
+ *
+ * Returns: the #GByteArray
+ */
+GByteArray*
+g_byte_array_set_size (GByteArray *array,
+                       guint       length)
 {
-  g_array_set_size ((GArray*) array, length);
+  g_array_set_size ((GArray *)array, length);
 
   return array;
 }
 
 /**
  * g_byte_array_remove_index:
- * @array: a #GByteArray.
- * @index_: the index of the byte to remove.
- * @Returns: the #GByteArray.
+ * @array: a #GByteArray
+ * @index_: the index of the byte to remove
  *
- * Removes the byte at the given index from a #GByteArray. The
- * following bytes are moved down one place.
+ * Removes the byte at the given index from a #GByteArray.
+ * The following bytes are moved down one place.
+ *
+ * Returns: the #GByteArray
  **/
-GByteArray* g_byte_array_remove_index (GByteArray *array,
-                                      guint       index_)
+GByteArray*
+g_byte_array_remove_index (GByteArray *array,
+                           guint       index_)
 {
-  g_array_remove_index ((GArray*) array, index_);
+  g_array_remove_index ((GArray *)array, index_);
 
   return array;
 }
 
 /**
  * g_byte_array_remove_index_fast:
- * @array: a #GByteArray.
- * @index_: the index of the byte to remove.
- * @Returns: the #GByteArray.
+ * @array: a #GByteArray
+ * @index_: the index of the byte to remove
  *
  * Removes the byte at the given index from a #GByteArray. The last
  * element in the array is used to fill in the space, so this function
  * does not preserve the order of the #GByteArray. But it is faster
  * than g_byte_array_remove_index().
- **/
-GByteArray* g_byte_array_remove_index_fast (GByteArray *array,
-                                           guint       index_)
+ *
+ * Returns: the #GByteArray
+ */
+GByteArray*
+g_byte_array_remove_index_fast (GByteArray *array,
+                                guint       index_)
 {
-  g_array_remove_index_fast ((GArray*) array, index_);
+  g_array_remove_index_fast ((GArray *)array, index_);
 
   return array;
 }
 
 /**
  * g_byte_array_remove_range:
- * @array: a @GByteArray.
- * @index_: the index of the first byte to remove.
- * @length: the number of bytes to remove.
- * @Returns: the #GByteArray.
+ * @array: a @GByteArray
+ * @index_: the index of the first byte to remove
+ * @length: the number of bytes to remove
  *
  * Removes the given number of bytes starting at the given index from a
  * #GByteArray.  The following elements are moved to close the gap.
  *
+ * Returns: the #GByteArray
+ *
  * Since: 2.4
- **/
+ */
 GByteArray*
 g_byte_array_remove_range (GByteArray *array,
                            guint       index_,
@@ -1580,13 +1816,13 @@ g_byte_array_remove_range (GByteArray *array,
   g_return_val_if_fail (index_ < array->len, NULL);
   g_return_val_if_fail (index_ + length <= array->len, NULL);
 
-  return (GByteArray *)g_array_remove_range ((GArray*) array, index_, length);
+  return (GByteArray *)g_array_remove_range ((GArray *)array, index_, length);
 }
 
 /**
  * g_byte_array_sort:
- * @array: a #GByteArray.
- * @compare_func: comparison function.
+ * @array: a #GByteArray
+ * @compare_func: comparison function
  *
  * Sorts a byte array, using @compare_func which should be a
  * qsort()-style comparison function (returns less than zero for first
@@ -1594,31 +1830,31 @@ g_byte_array_remove_range (GByteArray *array,
  * first arg is greater than second arg).
  *
  * If two array elements compare equal, their order in the sorted array
- * is undefined. If you want equal elements to keep their order &#8211; i.e.
- * you want a stable sort &#8211; you can write a comparison function that,
+ * is undefined. If you want equal elements to keep their order (i.e.
+ * you want a stable sort) you can write a comparison function that,
  * if two elements would otherwise compare equal, compares them by
  * their addresses.
- **/
+ */
 void
 g_byte_array_sort (GByteArray   *array,
-                  GCompareFunc  compare_func)
+                   GCompareFunc  compare_func)
 {
-  g_array_sort ((GArray *) array, compare_func);
+  g_array_sort ((GArray *)array, compare_func);
 }
 
 /**
  * g_byte_array_sort_with_data:
- * @array: a #GByteArray.
- * @compare_func: comparison function.
- * @user_data: data to pass to @compare_func.
+ * @array: a #GByteArray
+ * @compare_func: comparison function
+ * @user_data: data to pass to @compare_func
  *
  * Like g_byte_array_sort(), but the comparison function takes an extra
  * user data argument.
- **/
+ */
 void
 g_byte_array_sort_with_data (GByteArray       *array,
-                            GCompareDataFunc  compare_func,
-                            gpointer          user_data)
+                             GCompareDataFunc  compare_func,
+                             gpointer          user_data)
 {
-  g_array_sort_with_data ((GArray *) array, compare_func, user_data);
+  g_array_sort_with_data ((GArray *)array, compare_func, user_data);
 }