Revert "Revert "Merge remote-tracking branch 'origin/sandbox/mniesluchow/upstream_2_1...
[platform/upstream/atk.git] / atk / atktext.c
index 85f376d..f18e8cb 100755 (executable)
  * Boston, MA 02111-1307, USA.
  */
 
-#include "atktext.h"
+#include "config.h"
+
+#include "atk.h"
 #include "atkmarshal.h"
-#include "atk-enum-types.h"
 
 #include <string.h>
 
-GPtrArray *extra_attributes = NULL;
+/**
+ * SECTION:atktext
+ * @Short_description: The ATK interface implemented by components
+ *  with text content.
+ * @Title:AtkText
+ *
+ * #AtkText should be implemented by #AtkObjects on behalf of widgets
+ * that have text content which is either attributed or otherwise
+ * non-trivial.  #AtkObjects whose text content is simple,
+ * unattributed, and very brief may expose that content via
+ * #atk_object_get_name instead; however if the text is editable,
+ * multi-line, typically longer than three or four words, attributed,
+ * selectable, or if the object already uses the 'name' ATK property
+ * for other information, the #AtkText interface should be used to
+ * expose the text content.  In the case of editable text content,
+ * #AtkEditableText (a subtype of the #AtkText interface) should be
+ * implemented instead.
+ *
+ *  #AtkText provides not only traversal facilities and change
+ * notification for text content, but also caret tracking and glyph
+ * bounding box calculations.  Note that the text strings are exposed
+ * as UTF-8, and are therefore potentially multi-byte, and
+ * caret-to-byte offset mapping makes no assumptions about the
+ * character length; also bounding box glyph-to-offset mapping may be
+ * complex for languages which use ligatures.
+ */
+
+static GPtrArray *extra_attributes = NULL;
 
 enum {
   TEXT_CHANGED,
   TEXT_CARET_MOVED,
   TEXT_SELECTION_CHANGED,
   TEXT_ATTRIBUTES_CHANGED,
+  TEXT_INSERT,
+  TEXT_REMOVE,
   LAST_SIGNAL
 };
 
-static const gchar *bool[] = {"false",
-                              "true"};
-static const gchar *style[] = {"normal",
-                               "oblique",
-                               "italic"};
-static const gchar *variant[] = {"normal",
-                                 "small_caps"};
-static const gchar *stretch[] = {"ultra_condensed",
-                                 "extra_condensed",
-                                 "condensed",
-                                 "semi_condensed",
-                                 "normal",
-                                 "semi_expanded",
-                                 "expanded",
-                                 "extra_expanded",
-                                 "ultra_expanded"};
-static const gchar *justification[] = {"left",
-                                       "right",
-                                       "center",
-                                       "fill"};
-static const gchar *direction[] = {"none",
-                                   "ltr",
-                                   "rtl"};
-static const gchar *wrap_mode[] = {"none",
-                                   "char",
-                                   "word"};
-static const gchar *underline[] = {"none",
-                                   "single",
-                                   "double",
-                                   "low"};
+static const char boolean[] =
+  "false\0"
+  "true";
+static const guint8 boolean_offsets[] = {
+  0, 6
+};
+
+static const char style[] =
+  "normal\0"
+  "oblique\0"
+  "italic";
+static const guint8 style_offsets[] = {
+  0, 7, 15
+};
+
+static const char variant[] =
+  "normal\0"
+  "small_caps";
+static const guint8 variant_offsets[] = {
+  0, 7
+};
+
+static const char stretch[] =
+  "ultra_condensed\0"
+  "extra_condensed\0"
+  "condensed\0"
+  "semi_condensed\0"
+  "normal\0"
+  "semi_expanded\0"
+  "expanded\0"
+  "extra_expanded\0"
+  "ultra_expanded";
+static const guint8 stretch_offsets[] = {
+  0, 16, 32, 42, 57, 64, 78, 87, 102
+};
+
+static const char justification[] =
+  "left\0"
+  "right\0"
+  "center\0"
+  "fill";
+static const guint8 justification_offsets[] = {
+  0, 5, 11, 18
+};
+
+static const char direction[] =
+  "none\0"
+  "ltr\0"
+  "rtl";
+static const guint8 direction_offsets[] = {
+  0, 5, 9
+};
+
+static const char wrap_mode[] =
+  "none\0"
+  "char\0"
+  "word\0"
+  "word_char";
+static const guint8 wrap_mode_offsets[] = {
+  0, 5, 10, 15
+};
+
+static const char underline[] =
+  "none\0"
+  "single\0"
+  "double\0"
+  "low\0"
+  "error";
+static const guint8 underline_offsets[] = {
+  0, 5, 12, 19, 23
+};
 
 static void atk_text_base_init (AtkTextIface *class);
 
@@ -81,7 +152,7 @@ static AtkTextRange** atk_text_real_get_bounded_ranges (AtkText          *text,
 static guint atk_text_signals[LAST_SIGNAL] = { 0 };
 
 GType
-atk_text_get_type ()
+atk_text_get_type (void)
 {
   static GType type = 0;
 
@@ -118,6 +189,21 @@ atk_text_base_init (AtkTextIface *class)
       class->get_range_extents = atk_text_real_get_range_extents; 
       class->get_bounded_ranges = atk_text_real_get_bounded_ranges; 
 
+      /**
+       * AtkText::text-changed:
+       * @atktext: the object which received the signal.
+       * @arg1: The position (character offset) of the insertion or deletion.
+       * @arg2: The length (in characters) of text inserted or deleted.
+       *
+       * The "text-changed" signal is emitted when the text of the
+       * object which implements the AtkText interface changes, This
+       * signal will have a detail which is either "insert" or
+       * "delete" which identifies whether the text change was an
+       * insertion or a deletion.
+       *
+       * Deprecated: 2.9.4: Use #AtkObject::text-insert or
+       * #AtkObject::text-remove instead.
+       */
       atk_text_signals[TEXT_CHANGED] =
        g_signal_new ("text_changed",
                      ATK_TYPE_TEXT,
@@ -127,7 +213,60 @@ atk_text_base_init (AtkTextIface *class)
                      atk_marshal_VOID__INT_INT,
                      G_TYPE_NONE,
                      2, G_TYPE_INT, G_TYPE_INT);
-      
+
+      /**
+       * AtkText::text-insert:
+       * @atktext: the object which received the signal.
+       * @arg1: The position (character offset) of the insertion.
+       * @arg2: The length (in characters) of text inserted.
+       * @arg3: The new text inserted
+       *
+       * The "text-insert" signal is emitted when a new text is
+       * inserted. If the signal was not triggered by the user
+       * (e.g. typing or pasting text), the "system" detail should be
+       * included.
+       */
+      atk_text_signals[TEXT_INSERT] =
+       g_signal_new ("text_insert",
+                     ATK_TYPE_TEXT,
+                     G_SIGNAL_RUN_LAST | G_SIGNAL_DETAILED,
+                     0,
+                     (GSignalAccumulator) NULL, NULL,
+                     atk_marshal_VOID__INT_INT_STRING,
+                     G_TYPE_NONE,
+                     3, G_TYPE_INT, G_TYPE_INT, G_TYPE_STRING);
+
+      /**
+       * AtkText::text-remove:
+       * @atktext: the object which received the signal.
+       * @arg1: The position (character offset) of the removal.
+       * @arg2: The length (in characters) of text removed.
+       * @arg3: The old text removed
+       *
+       * The "text-remove" signal is emitted when a new text is
+       * removed. If the signal was not triggered by the user
+       * (e.g. typing or pasting text), the "system" detail should be
+       * included.
+       */
+      atk_text_signals[TEXT_REMOVE] =
+       g_signal_new ("text_remove",
+                     ATK_TYPE_TEXT,
+                     G_SIGNAL_RUN_LAST | G_SIGNAL_DETAILED,
+                     0,
+                     (GSignalAccumulator) NULL, NULL,
+                     atk_marshal_VOID__INT_INT_STRING,
+                     G_TYPE_NONE,
+                     3, G_TYPE_INT, G_TYPE_INT, G_TYPE_STRING);
+
+      /**
+       * AtkText::text-caret-moved:
+       * @atktext: the object which received the signal.
+       * @arg1: The new position of the text caret.
+       *
+       * The "text-caret-moved" signal is emitted when the caret
+       * position of the text of an object which implements AtkText
+       * changes.
+       */
       atk_text_signals[TEXT_CARET_MOVED] =
        g_signal_new ("text_caret_moved",
                      ATK_TYPE_TEXT,
@@ -137,6 +276,14 @@ atk_text_base_init (AtkTextIface *class)
                      g_cclosure_marshal_VOID__INT,
                      G_TYPE_NONE,
                      1, G_TYPE_INT);
+
+      /**
+       * AtkText::text-selection-changed:
+       * @atktext: the object which received the signal.
+       *
+       * The "text-selection-changed" signal is emitted when the
+       * selected text of an object which implements AtkText changes.
+       */
       atk_text_signals[TEXT_SELECTION_CHANGED] =
         g_signal_new ("text_selection_changed",
                       ATK_TYPE_TEXT,
@@ -145,6 +292,14 @@ atk_text_base_init (AtkTextIface *class)
                       (GSignalAccumulator) NULL, NULL,
                       g_cclosure_marshal_VOID__VOID,
                       G_TYPE_NONE, 0);
+      /**
+       * AtkText::text-attributes-changed:
+       * @atktext: the object which received the signal.
+       *
+       * The "text-attributes-changed" signal is emitted when the text
+       * attributes of the text of an object which implements AtkText
+       * changes.
+       */
       atk_text_signals[TEXT_ATTRIBUTES_CHANGED] =
         g_signal_new ("text_attributes_changed",
                       ATK_TYPE_TEXT,
@@ -163,11 +318,12 @@ atk_text_base_init (AtkTextIface *class)
  * atk_text_get_text:
  * @text: an #AtkText
  * @start_offset: start position
- * @end_offset: end position
+ * @end_offset: end position, or -1 for the end of the string.
  *
  * Gets the specified text.
  *
- * Returns: the text from @start_offset up to, but not including @end_offset.
+ * Returns: a newly allocated string containing the text from @start_offset up
+ *   to, but not including @end_offset. Use g_free() to free the returned string.
  **/
 gchar*
 atk_text_get_text (AtkText      *text,
@@ -180,7 +336,8 @@ atk_text_get_text (AtkText      *text,
 
   iface = ATK_TEXT_GET_IFACE (text);
 
-  if (start_offset < 0 || end_offset < -1)
+  if (start_offset < 0 || end_offset < -1 ||
+      (end_offset != -1 && end_offset < start_offset))
     return NULL;
 
   if (iface->get_text)
@@ -206,9 +363,6 @@ atk_text_get_character_at_offset (AtkText      *text,
 
   g_return_val_if_fail (ATK_IS_TEXT (text), (gunichar) 0);
 
-  if (offset < 0)
-    return (gunichar) 0;
-
   iface = ATK_TEXT_GET_IFACE (text);
 
   if (iface->get_character_at_offset)
@@ -222,48 +376,16 @@ atk_text_get_character_at_offset (AtkText      *text,
  * @text: an #AtkText
  * @offset: position
  * @boundary_type: An #AtkTextBoundary
- * @start_offset: the start offset of the returned string.
- * @end_offset: the end offset of the returned string.
+ * @start_offset: (out): the start offset of the returned string
+ * @end_offset: (out): the offset of the first character after the
+ *              returned substring
  *
  * Gets the specified text.
  *
- * If the boundary_type if ATK_TEXT_BOUNDARY_CHAR the character after the 
- * offset is returned.
- *
- * If the boundary_type is ATK_TEXT_BOUNDARY_WORD_START the returned string
- * is from the word start after the offset to the next word start.
- *
- * The returned string will contain the word after the offset if the offset 
- * is inside a word or if the offset is not inside a word.
- *
- * If the boundary_type is ATK_TEXT_BOUNDARY_WORD_END the returned string
- * is from the word end at or after the offset to the next work end.
- *
- * The returned string will contain the word after the offset if the offset
- * is inside a word and will contain the word after the word after the offset
- * if the offset is not inside a word.
- *
- * If the boundary type is ATK_TEXT_BOUNDARY_SENTENCE_START the returned
- * string is from the sentence start after the offset to the next sentence
- * start.
- *
- * The returned string will contain the sentence after the offset if the offset
- * is inside a sentence or if the offset is not inside a sentence.
- *
- * If the boundary_type is ATK_TEXT_BOUNDARY_SENTENCE_END the returned string
- * is from the sentence end at or after the offset to the next sentence end.
+ * Deprecated: 2.9.3: Please use atk_text_get_string_at_offset() instead.
  *
- * The returned string will contain the sentence after the offset if the offset
- * is inside a sentence and will contain the sentence after the sentence
- * after the offset if the offset is not inside a sentence.
- *
- * If the boundary type is ATK_TEXT_BOUNDARY_LINE_START the returned
- * string is from the line start after the offset to the next line start.
- *
- * If the boundary_type is ATK_TEXT_BOUNDARY_LINE_END the returned string
- * is from the line end at or after the offset to the next line start.
- *
- * Returns: the text after @offset bounded by the specified @boundary_type.
+ * Returns: a newly allocated string containing the text after @offset bounded
+ *   by the specified @boundary_type. Use g_free() to free the returned string.
  **/
 gchar*
 atk_text_get_text_after_offset (AtkText          *text,
@@ -303,8 +425,9 @@ atk_text_get_text_after_offset (AtkText          *text,
  * @text: an #AtkText
  * @offset: position
  * @boundary_type: An #AtkTextBoundary
- * @start_offset: the start offset of the returned string.
- * @end_offset: the end offset of the returned string.
+ * @start_offset: (out): the start offset of the returned string
+ * @end_offset: (out): the offset of the first character after the
+ *              returned substring
  *
  * Gets the specified text.
  *
@@ -312,19 +435,11 @@ atk_text_get_text_after_offset (AtkText          *text,
  * offset is returned.
  *
  * If the boundary_type is ATK_TEXT_BOUNDARY_WORD_START the returned string
- * is from the word start at or before the offset to the word start after 
+ * is from the word start at or before the offset to the word start after
  * the offset.
  *
  * The returned string will contain the word at the offset if the offset
- * is inside a word and will contain the word before the offset if the 
- * offset is not inside a word.
- *
- * If the boundary_type is ATK_TEXT_BOUNDARY_WORD_END the returned string
- * is from the word end before the offset to the word end at or after the
- * offset.
- *
- * The returned string will contain the word at the offset if the offset
- * is inside a word and will contain the word after to the offset if the 
+ * is inside a word and will contain the word before the offset if the
  * offset is not inside a word.
  *
  * If the boundary type is ATK_TEXT_BOUNDARY_SENTENCE_START the returned
@@ -332,26 +447,18 @@ atk_text_get_text_after_offset (AtkText          *text,
  * start after the offset.
  *
  * The returned string will contain the sentence at the offset if the offset
- * is inside a sentence and will contain the sentence before the offset 
- * if the offset is not inside a sentence.
- *
- * If the boundary_type is ATK_TEXT_BOUNDARY_SENTENCE_END the returned string
- * is from the sentence end before the offset to the sentence end at or
- * after the offset.
- *
- * The returned string will contain the sentence at the offset if the offset
- * is inside a sentence and will contain the sentence after the offset 
+ * is inside a sentence and will contain the sentence before the offset
  * if the offset is not inside a sentence.
  *
  * If the boundary type is ATK_TEXT_BOUNDARY_LINE_START the returned
  * string is from the line start at or before the offset to the line
  * start after the offset.
  *
- * If the boundary_type is ATK_TEXT_BOUNDARY_LINE_END the returned string
- * is from the line end before the offset to the line end at or after
- * the offset.
+ * Deprecated: This method is deprecated since ATK version
+ * 2.9.4. Please use atk_text_get_string_at_offset() instead.
  *
- * Returns: the text at @offset bounded by the specified @boundary_type.
+ * Returns: a newly allocated string containing the text at @offset bounded by
+ *   the specified @boundary_type. Use g_free() to free the returned string.
  **/
 gchar*
 atk_text_get_text_at_offset (AtkText          *text,
@@ -375,9 +482,6 @@ atk_text_get_text_at_offset (AtkText          *text,
   else
     real_end_offset = &local_end_offset;
 
-  if (offset < 0)
-    return NULL;
-
   iface = ATK_TEXT_GET_IFACE (text);
 
   if (iface->get_text_at_offset)
@@ -391,53 +495,16 @@ atk_text_get_text_at_offset (AtkText          *text,
  * @text: an #AtkText
  * @offset: position
  * @boundary_type: An #AtkTextBoundary
- * @start_offset: the start offset of the returned string.
- * @end_offset: the end offset of the returned string.
+ * @start_offset: (out): the start offset of the returned string
+ * @end_offset: (out): the offset of the first character after the
+ *              returned substring
  *
  * Gets the specified text.
  *
- * If the boundary_type if ATK_TEXT_BOUNDARY_CHAR the character before the
- * offset is returned.
- *
- * If the boundary_type is ATK_TEXT_BOUNDARY_WORD_START the returned string
- * is from the word start before the word start before the offset to 
- * the word start before the offset.
- *
- * The returned string will contain the word before the offset if the offset
- * is inside a word and will contain the word before the word before the 
- * offset if the offset is not inside a word.
- *
- * If the boundary_type is ATK_TEXT_BOUNDARY_WORD_END the returned string
- * is from the word end before the word end at or before the offset to the 
- * word end at or before the offset.
- *
- * The returned string will contain the word before the offset if the offset
- * is inside a word or if the offset is not inside a word.
- *
- * If the boundary type is ATK_TEXT_BOUNDARY_SENTENCE_START the returned
- * string is from the sentence start before the sentence start before 
- * the offset to the sentence start before the offset.
- *
- * The returned string will contain the sentence before the offset if the 
- * offset is inside a sentence and will contain the sentence before the 
- * sentence before the offset if the offset is not inside a sentence.
- *
- * If the boundary_type is ATK_TEXT_BOUNDARY_SENTENCE_END the returned string
- * is from the sentence end before the sentence end at or before the offset to 
- * the sentence end at or before the offset.
- *
- * The returned string will contain the sentence before the offset if the 
- * offset is inside a sentence or if the offset is not inside a sentence.
- *
- * If the boundary type is ATK_TEXT_BOUNDARY_LINE_START the returned
- * string is from the line start before the line start ar or before the offset 
- * to the line start ar or before the offset.
- *
- * If the boundary_type is ATK_TEXT_BOUNDARY_LINE_END the returned string
- * is from the line end before the line end before the offset to the 
- * line end before the offset.
+ * Deprecated: 2.9.3: Please use atk_text_get_string_at_offset() instead.
  *
- * Returns: the text before @offset bounded by the specified @boundary_type.
+ * Returns: a newly allocated string containing the text before @offset bounded
+ *   by the specified @boundary_type. Use g_free() to free the returned string.
  **/
 gchar*
 atk_text_get_text_before_offset (AtkText          *text,
@@ -473,6 +540,93 @@ atk_text_get_text_before_offset (AtkText          *text,
 }
 
 /**
+ * atk_text_get_string_at_offset:
+ * @text: an #AtkText
+ * @offset: position
+ * @granularity: An #AtkTextGranularity
+ * @start_offset: (out): the start offset of the returned string, or -1
+ *                if an error has occurred (e.g. invalid offset, not implemented)
+ * @end_offset: (out): the offset of the first character after the returned string,
+ *              or -1 if an error has occurred (e.g. invalid offset, not implemented)
+ *
+ * Gets a portion of the text exposed through an #AtkText according to a given @offset
+ * and a specific @granularity, along with the start and end offsets defining the
+ * boundaries of such a portion of text.
+ *
+ * If @granularity is ATK_TEXT_GRANULARITY_CHAR the character at the
+ * offset is returned.
+ *
+ * If @granularity is ATK_TEXT_GRANULARITY_WORD the returned string
+ * is from the word start at or before the offset to the word start after
+ * the offset.
+ *
+ * The returned string will contain the word at the offset if the offset
+ * is inside a word and will contain the word before the offset if the
+ * offset is not inside a word.
+ *
+ * If @granularity is ATK_TEXT_GRANULARITY_SENTENCE the returned string
+ * is from the sentence start at or before the offset to the sentence
+ * start after the offset.
+ *
+ * The returned string will contain the sentence at the offset if the offset
+ * is inside a sentence and will contain the sentence before the offset
+ * if the offset is not inside a sentence.
+ *
+ * If @granularity is ATK_TEXT_GRANULARITY_LINE the returned string
+ * is from the line start at or before the offset to the line
+ * start after the offset.
+ *
+ * If @granularity is ATK_TEXT_GRANULARITY_PARAGRAPH the returned string
+ * is from the start of the paragraph at or before the offset to the start
+ * of the following paragraph after the offset.
+ *
+ * Since: 2.10
+ *
+ * Returns: (nullable): a newly allocated string containing the text
+ *   at the @offset bounded by the specified @granularity. Use
+ *   g_free() to free the returned string.  Returns %NULL if the
+ *   offset is invalid or no implementation is available.
+ **/
+gchar* atk_text_get_string_at_offset (AtkText *text,
+                                      gint offset,
+                                      AtkTextGranularity granularity,
+                                      gint *start_offset,
+                                      gint *end_offset)
+{
+  AtkTextIface *iface;
+  gint local_start_offset, local_end_offset;
+  gint *real_start_offset, *real_end_offset;
+
+  g_return_val_if_fail (ATK_IS_TEXT (text), NULL);
+
+  if (start_offset)
+    {
+      *start_offset = -1;
+      real_start_offset = start_offset;
+    }
+  else
+    real_start_offset = &local_start_offset;
+
+  if (end_offset)
+    {
+      *end_offset = -1;
+      real_end_offset = end_offset;
+    }
+  else
+    real_end_offset = &local_end_offset;
+
+  if (offset < 0)
+    return NULL;
+
+  iface = ATK_TEXT_GET_IFACE (text);
+
+  if (iface->get_string_at_offset)
+    return (*(iface->get_string_at_offset)) (text, offset, granularity, real_start_offset, real_end_offset);
+  else
+    return NULL;
+}
+
+/**
  * atk_text_get_caret_offset:
  * @text: an #AtkText
  *
@@ -499,10 +653,10 @@ atk_text_get_caret_offset (AtkText *text)
  * atk_text_get_character_extents:
  * @text: an #AtkText
  * @offset: The offset of the text character for which bounding information is required.
- * @x: Pointer for the x cordinate of the bounding box.
- * @y: Pointer for the y cordinate of the bounding box.
+ * @x: Pointer for the x cordinate of the bounding box
+ * @y: Pointer for the y cordinate of the bounding box
  * @width: Pointer for the width of the bounding box
- * @height: Pointer for the height of the bounding box.
+ * @height: Pointer for the height of the bounding box
  * @coords: specify whether coordinates are relative to the screen or widget window 
  *
  * Get the bounding box containing the glyph representing the character at 
@@ -561,21 +715,24 @@ atk_text_get_character_extents (AtkText *text,
 }
 
 /**
- *atk_text_get_run_attributes:
+ * atk_text_get_run_attributes:
  *@text: an #AtkText
- *@offset: the offset at which to get the attributes
- *@start_offset: the address to put the start offset of the range
- *@end_offset: the address to put the end offset of the range
+ *@offset: the offset at which to get the attributes, -1 means the offset of
+ *the character to be inserted at the caret location.
+ *@start_offset: (out): the address to put the start offset of the range
+ *@end_offset: (out): the address to put the end offset of the range
  *
  *Creates an #AtkAttributeSet which consists of the attributes explicitly
  *set at the position @offset in the text. @start_offset and @end_offset are
  *set to the start and end of the range around @offset where the attributes are
- *invariant. See the enum AtkTextAttribute for types of text attributes that 
- *can be returned. Note that other attributes may also be returned.
+ *invariant. Note that @end_offset is the offset of the first character
+ *after the range.  See the enum AtkTextAttribute for types of text 
+ *attributes that can be returned. Note that other attributes may also be 
+ *returned.
  *
- *Returns: an #AtkAttributeSet which contains the attributes explicitly set
- *at @offset. This #AtkAttributeSet should be freed by a call to
- *atk_attribute_set_free().
+ *Returns: (transfer full): an #AtkAttributeSet which contains the attributes
+ * explicitly set at @offset. This #AtkAttributeSet should be freed by a call
+ * to atk_attribute_set_free().
  **/
 AtkAttributeSet* 
 atk_text_get_run_attributes (AtkText          *text,
@@ -598,7 +755,7 @@ atk_text_get_run_attributes (AtkText          *text,
   else
     real_end_offset = &local_end_offset;
 
-  if (offset < 0)
+  if (offset < -1)
     return NULL;
 
   iface = ATK_TEXT_GET_IFACE (text);
@@ -610,7 +767,7 @@ atk_text_get_run_attributes (AtkText          *text,
 }
 
 /**
- *atk_text_get_default_attributes:
+ * atk_text_get_default_attributes:
  *@text: an #AtkText
  *
  *Creates an #AtkAttributeSet which consists of the default values of
@@ -618,9 +775,9 @@ atk_text_get_run_attributes (AtkText          *text,
  *attributes that can be returned. Note that other attributes may also be 
  *returned.
  *
- *Returns: an #AtkAttributeSet which contains the default values of attributes.
- *at @offset. This #AtkAttributeSet should be freed by a call to
- *atk_attribute_set_free().
+ *Returns: (transfer full): an #AtkAttributeSet which contains the default
+ * values of attributes.  at @offset. this #atkattributeset should be freed by
+ * a call to atk_attribute_set_free().
  */
 AtkAttributeSet* 
 atk_text_get_default_attributes (AtkText          *text)
@@ -725,12 +882,14 @@ atk_text_get_n_selections (AtkText *text)
  * start of the text.  The selected region closest to the beginning
  * of the text region is assigned the number 0, etc.  Note that adding,
  * moving or deleting a selected region can change the numbering.
- * @start_offset: passes back the start position of the selected region
- * @end_offset: passes back the end position of the selected region
+ * @start_offset: (out): passes back the start position of the selected region
+ * @end_offset: (out): passes back the end position of (e.g. offset immediately past)
+ * the selected region
  *
  * Gets the text from the specified selection.
  *
- * Returns: the selected text.
+ * Returns: a newly allocated string containing the selected text. Use g_free()
+ *   to free the returned string.
  **/
 gchar*
 atk_text_get_selection (AtkText *text, 
@@ -768,7 +927,7 @@ atk_text_get_selection (AtkText *text,
  * atk_text_add_selection:
  * @text: an #AtkText
  * @start_offset: the start position of the selected region
- * @end_offset: the end position of the selected region
+ * @end_offset: the offset of the first character after the selected region.
  *
  * Adds a selection bounded by the specified offsets.
  *
@@ -829,7 +988,8 @@ atk_text_remove_selection (AtkText *text,
  * of the text region is assigned the number 0, etc.  Note that adding,
  * moving or deleting a selected region can change the numbering.
  * @start_offset: the new start position of the selection
- * @end_offset: the new end position of the selection
+ * @end_offset: the new end position of (e.g. offset immediately past) 
+ * the selection
  *
  * Changes the start and end offset of the specified selection.
  *
@@ -896,6 +1056,8 @@ atk_text_set_caret_offset (AtkText *text,
  * @rect: A pointer to a AtkTextRectangle which is filled in by this function.
  *
  * Get the bounding box for text within the specified range.
+ *
+ * Since: 1.3
  **/
 void
 atk_text_get_range_extents (AtkText          *text,
@@ -908,10 +1070,8 @@ atk_text_get_range_extents (AtkText          *text,
 
   g_return_if_fail (ATK_IS_TEXT (text));
   g_return_if_fail (rect);
+  g_return_if_fail (start_offset >= 0 && start_offset < end_offset);
 
-  if (start_offset < 0 || start_offset >= end_offset)
-    return;
   iface = ATK_TEXT_GET_IFACE (text);
 
   if (iface->get_range_extents)
@@ -921,15 +1081,17 @@ atk_text_get_range_extents (AtkText          *text,
 /**
  * atk_text_get_bounded_ranges:
  * @text: an #AtkText
- * @rect: An AtkTextRectagle giving the dimensions of the bounding box.
+ * @rect: An AtkTextRectangle giving the dimensions of the bounding box.
  * @coord_type: Specify whether coordinates are relative to the screen or widget window.
  * @x_clip_type: Specify the horizontal clip type.
  * @y_clip_type: Specify the vertical clip type.
  *
  * Get the ranges of text in the specified bounding box.
  *
- * Returns: Array of AtkTextRange. The last element of the array returned 
- *          by this function will be NULL.
+ * Since: 1.3
+ *
+ * Returns: (array zero-terminated=1): Array of AtkTextRange. The last
+ *          element of the array returned by this function will be NULL.
  **/
 AtkTextRange**
 atk_text_get_bounded_ranges (AtkText          *text,
@@ -1007,12 +1169,12 @@ atk_text_attribute_register (const gchar *name)
  *
  * Returns: a string containing the name; this string should not be freed
  **/
-G_CONST_RETURN gchar*
+const gchar*
 atk_text_attribute_get_name (AtkTextAttribute attr)
 {
   GTypeClass *type_class;
   GEnumValue *value;
-  gchar *name = NULL;
+  const gchar *name = NULL;
 
   type_class = g_type_class_ref (ATK_TYPE_TEXT_ATTRIBUTE);
   g_return_val_if_fail (G_IS_ENUM_CLASS (type_class), NULL);
@@ -1100,10 +1262,11 @@ atk_text_attribute_for_name (const gchar *name)
  *
  * Gets the value for the index of the #AtkTextAttribute
  *
- * Returns: a string containing the value; this string should not be freed;
- * NULL is returned if there are no values maintained for the attr value. 
+ * Returns: (nullable): a string containing the value; this string
+ * should not be freed; %NULL is returned if there are no values
+ * maintained for the attr value.
  **/
-G_CONST_RETURN gchar*
+const gchar*
 atk_text_attribute_get_value (AtkTextAttribute attr,
                               gint             index)
 {
@@ -1115,29 +1278,29 @@ atk_text_attribute_get_value (AtkTextAttribute attr,
     case ATK_TEXT_ATTR_STRIKETHROUGH:
     case ATK_TEXT_ATTR_BG_STIPPLE:
     case ATK_TEXT_ATTR_FG_STIPPLE:
-      g_assert (index >= 0 && index < 2);
-      return bool[index];
+      g_assert (index >= 0 && index < G_N_ELEMENTS (boolean_offsets));
+      return boolean + boolean_offsets[index];
     case ATK_TEXT_ATTR_UNDERLINE:
-      g_assert (index >= 0 && index < 4);
-      return underline[index];
+      g_assert (index >= 0 && index < G_N_ELEMENTS (underline_offsets));
+      return underline + underline_offsets[index];
     case ATK_TEXT_ATTR_WRAP_MODE:
-      g_assert (index >= 0 && index < 3);
-      return wrap_mode[index];
+      g_assert (index >= 0 && index < G_N_ELEMENTS (wrap_mode_offsets));
+      return wrap_mode + wrap_mode_offsets[index];
     case ATK_TEXT_ATTR_DIRECTION:
-      g_assert (index >= 0 && index < 3);
-      return direction[index];
+      g_assert (index >= 0 && index < G_N_ELEMENTS (direction_offsets));
+      return direction + direction_offsets[index];
     case ATK_TEXT_ATTR_JUSTIFICATION:
-      g_assert (index >= 0 && index < 3);
-      return justification[index];
+      g_assert (index >= 0 && index < G_N_ELEMENTS (justification_offsets));
+      return justification + justification_offsets[index];
     case ATK_TEXT_ATTR_STRETCH:
-      g_assert (index >= 0 && index < 9);
-      return stretch[index];
+      g_assert (index >= 0 && index < G_N_ELEMENTS (stretch_offsets));
+      return stretch + stretch_offsets[index];
     case ATK_TEXT_ATTR_VARIANT:
-      g_assert (index >= 0 && index < 2);
-      return variant[index];
+      g_assert (index >= 0 && index < G_N_ELEMENTS (variant_offsets));
+      return variant + variant_offsets[index];
     case ATK_TEXT_ATTR_STYLE:
-      g_assert (index >= 0 && index < 3);
-      return style[index];
+      g_assert (index >= 0 && index < G_N_ELEMENTS (style_offsets));
+      return style + style_offsets[index];
     default:
       return NULL;
    }
@@ -1298,15 +1461,20 @@ atk_text_real_get_bounded_ranges (AtkText          *text,
 
 /**
  * atk_text_free_ranges:
- * @ranges: A pointer to an array of  #AtkTextRange which is to be freed.
+ * @ranges: (array): A pointer to an array of #AtkTextRange which is
+ *   to be freed.
  *
  * Frees the memory associated with an array of AtkTextRange. It is assumed
  * that the array was returned by the function atk_text_get_bounded_ranges
  * and is NULL terminated.
+ *
+ * Since: 1.3
  **/
 void
 atk_text_free_ranges (AtkTextRange **ranges)
 {
+  AtkTextRange **first = ranges;
+
   if (ranges)
     {
       while (*ranges)
@@ -1314,10 +1482,32 @@ atk_text_free_ranges (AtkTextRange **ranges)
           AtkTextRange *range;
 
           range = *ranges;
-          *ranges++;
+          ranges++;
           g_free (range->content);
-          g_free (ranges);
+          g_free (range);
         }
-      g_free (ranges);
+      g_free (first);
     }
 }
+
+static AtkTextRange *
+atk_text_range_copy (AtkTextRange *src)
+{
+  AtkTextRange *dst = g_new0 (AtkTextRange, 1);
+  dst->bounds = src->bounds;
+  dst->start_offset = src->start_offset;
+  dst->end_offset = src->end_offset;
+  if (src->content)
+    dst->content = g_strdup (src->content);
+  return dst;
+}
+
+static void
+atk_text_range_free (AtkTextRange *range)
+{
+  g_free (range->content);
+  g_free (range);
+}
+
+G_DEFINE_BOXED_TYPE (AtkTextRange, atk_text_range, atk_text_range_copy,
+                     atk_text_range_free)