summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthias Clasen <mclasen@redhat.com>2021-07-05 21:58:36 -0400
committerMatthias Clasen <mclasen@redhat.com>2021-07-06 01:09:46 -0400
commite610d051bf6fdbab9e93da29d40a53df7011ef02 (patch)
tree550df7a313676344e1aa7ce819d2b365ff4a658c
parent1000535b0d95472804cb8aed0c049e9b950781b1 (diff)
downloadpango-e610d051bf6fdbab9e93da29d40a53df7011ef02.tar.gz
layout: Remove an overeager assertion
pango_layout_get_direction contained an assertion that would be hit when just iterating over all chars in the text of the layout (it gets hit when you get to the final newline). Remove the assertion, so that this function is less of a trap.
-rw-r--r--pango/pango-layout.c8
1 files changed, 3 insertions, 5 deletions
diff --git a/pango/pango-layout.c b/pango/pango-layout.c
index aae46990..f9d4cba1 100644
--- a/pango/pango-layout.c
+++ b/pango/pango-layout.c
@@ -1895,9 +1895,9 @@ pango_layout_index_to_line_x (PangoLayout *layout,
* of a run.
*
* Motion here is in cursor positions, not in characters, so a single
- * call to [method@Pango.Layout.move_cursor_visually] may move the cursor over
- * multiple characters when multiple characters combine to form a single
- * grapheme.
+ * call to [method@Pango.Layout.move_cursor_visually] may move the cursor
+ * over multiple characters when multiple characters combine to form a
+ * single grapheme.
*/
void
pango_layout_move_cursor_visually (PangoLayout *layout,
@@ -2378,8 +2378,6 @@ pango_layout_line_get_char_direction (PangoLayoutLine *layout_line,
run_list = run_list->next;
}
- g_assert_not_reached ();
-
return PANGO_DIRECTION_LTR;
}