[gtk+] Fix gtk_text_iter_forward_to_tag_toggle() for end iter

[Sébastien Wilmet <swilmet src gnome org>]

commit 08d49fdd5821e18d9946ad975b04f233b22cfdd7
Author: Sébastien Wilmet <swilmet gnome org>
Date:   Wed Apr 3 13:41:44 2013 +0200

    Fix gtk_text_iter_forward_to_tag_toggle() for end iter
    
    The function must return TRUE only if there is a tag toggle _after_ the
    iter, not _at_ the iter. So for the end iter, the function must always
    return FALSE.
    
    Add also unit tests for gtk_text_iter_forward_to_tag_toggle().
    
    https://bugzilla.gnome.org/show_bug.cgi?id=691266

 gtk/gtktextiter.c        |    3 ++
 testsuite/gtk/textiter.c |   52 ++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 55 insertions(+), 0 deletions(-)
---
diff --git a/gtk/gtktextiter.c b/gtk/gtktextiter.c
index 4821c62..14f1e76 100644
--- a/gtk/gtktextiter.c
+++ b/gtk/gtktextiter.c
@@ -4163,6 +4163,9 @@ gtk_text_iter_forward_to_tag_toggle (GtkTextIter *iter,
 
   check_invariants (iter);
 
+  if (gtk_text_iter_is_end (iter))
+    return FALSE;
+
   current_line = real->line;
   next_line = _gtk_text_line_next_could_contain_tag (current_line,
                                                      real->tree, tag);
diff --git a/testsuite/gtk/textiter.c b/testsuite/gtk/textiter.c
index bee33de..53c8eeb 100644
--- a/testsuite/gtk/textiter.c
+++ b/testsuite/gtk/textiter.c
@@ -255,6 +255,57 @@ test_search_caseless (void)
   check_found_backward ("This is some \303\200\n\303\200 text", "a\314\200\na\314\200", flags, 13, 16, 
"\303\200\n\303\200");
 }
 
+static void
+test_forward_to_tag_toggle (void)
+{
+  GtkTextBuffer *buffer;
+  GtkTextTag *bold_tag;
+  GtkTextTag *editable_tag;
+  GtkTextIter iter;
+  gint offset;
+
+  buffer = gtk_text_buffer_new (NULL);
+
+  bold_tag = gtk_text_buffer_create_tag (buffer, "bold",
+                                         "weight", PANGO_WEIGHT_BOLD,
+                                         NULL);
+
+  editable_tag = gtk_text_buffer_create_tag (buffer, "not-editable",
+                                             "editable", FALSE,
+                                             NULL);
+
+  gtk_text_buffer_get_start_iter (buffer, &iter);
+
+  gtk_text_buffer_insert (buffer, &iter, "a", -1);
+  gtk_text_buffer_insert_with_tags (buffer, &iter, "b", -1, bold_tag, NULL);
+  gtk_text_buffer_insert_with_tags (buffer, &iter, "c", -1, editable_tag, NULL);
+
+  /* Go to the first "on" toggle */
+  gtk_text_buffer_get_start_iter (buffer, &iter);
+  g_assert (gtk_text_iter_forward_to_tag_toggle (&iter, NULL));
+  offset = gtk_text_iter_get_offset (&iter);
+  g_assert_cmpint (offset, ==, 1);
+
+  /* Go to the last "off" toggle for the bold tag */
+  g_assert (gtk_text_iter_forward_to_tag_toggle (&iter, bold_tag));
+  offset = gtk_text_iter_get_offset (&iter);
+  g_assert_cmpint (offset, ==, 2);
+
+  g_assert (!gtk_text_iter_forward_to_tag_toggle (&iter, bold_tag));
+
+  /* Go to the first "on" toggle for the editable tag */
+  gtk_text_buffer_get_start_iter (buffer, &iter);
+  g_assert (gtk_text_iter_forward_to_tag_toggle (&iter, editable_tag));
+  offset = gtk_text_iter_get_offset (&iter);
+  g_assert_cmpint (offset, ==, 2);
+
+  /* Test with the end iter */
+  gtk_text_buffer_get_end_iter (buffer, &iter);
+  g_assert (!gtk_text_iter_forward_to_tag_toggle (&iter, editable_tag));
+
+  g_object_unref (buffer);
+}
+
 int
 main (int argc, char** argv)
 {
@@ -264,6 +315,7 @@ main (int argc, char** argv)
   g_test_add_func ("/TextIter/Search Full Buffer", test_full_buffer);
   g_test_add_func ("/TextIter/Search", test_search);
   g_test_add_func ("/TextIter/Search Caseless", test_search_caseless);
+  g_test_add_func ("/TextIter/Forward To Tag Toggle", test_forward_to_tag_toggle);
 
   return g_test_run();
 }