Remove an assertion that we hit

It appears that we mess up accounting for blinking
cursors sometimes, and can hit blink_cb when there
is a nonempty selection.

Instead of asserting, warn and stop blinking.

Related: #4767

1 files changed, 7 insertions, 1 deletions

diff --git a/gtk/gtktext.c b/gtk/gtktext.c
index 6bbb6b3098..92903949fe 100644
--- a/gtk/gtktext.c
+++ b/gtk/gtktext.c
@@ -6535,7 +6535,13 @@ blink_cb (GtkWidget     *widget,
       return G_SOURCE_REMOVE;
     }
 
-  g_assert (priv->selection_bound == priv->current_pos);
+  if (priv->selection_bound != priv->current_pos)
+    {
+      g_warning ("GtkText - unexpected blinking selection. Removing");
+
+      gtk_text_check_cursor_blink (self);
+      return G_SOURCE_REMOVE;
+    }
 
   blink_timeout = get_cursor_blink_timeout (self);
   blink_time = get_cursor_time (self);