pango r2545 - in trunk: . pango
- From: behdad svn gnome org
- To: svn-commits-list gnome org
- Subject: pango r2545 - in trunk: . pango
- Date: Tue, 15 Jan 2008 20:52:36 +0000 (GMT)
Author: behdad
Date: Tue Jan 15 20:52:36 2008
New Revision: 2545
URL: http://svn.gnome.org/viewvc/pango?rev=2545&view=rev
Log:
2008-01-15 Behdad Esfahbod <behdad gnome org>
* pango/pango-layout.c (debug), (process_line),
(pango_layout_line_postprocess): Minor cleanup.
Modified:
trunk/ChangeLog
trunk/pango/pango-layout.c
Modified: trunk/pango/pango-layout.c
==============================================================================
--- trunk/pango/pango-layout.c (original)
+++ trunk/pango/pango-layout.c Tue Jan 15 20:52:36 2008
@@ -3104,19 +3104,7 @@
void
debug (const char *where, PangoLayoutLine *line, ParaBreakState *state)
{
- int line_width = 0;
-
- GSList *tmp_list;
-
- tmp_list = line->runs;
- while (tmp_list)
- {
- PangoLayoutRun *run = tmp_list->data;
-
- line_width += pango_glyph_string_get_width (run->glyphs);
-
- tmp_list = tmp_list->next;
- }
+ int line_width = pango_layout_line_get_width (line);
g_message ("rem %d + line %d = %d %s",
state->remaining_width,
@@ -3511,12 +3499,20 @@
case BREAK_LINE_SEPARATOR:
state->items = g_list_delete_link (state->items, state->items);
state->start_offset += old_num_chars;
+ /* FIXME We don't set wrapped here. Which means we don't justify
+ * lines that end in a line separator. Not sure which behavior is
+ * more desired. For now that we do greedy line breaking, not
+ * justifying lines of potentially very small length is a good idea,
+ * but when we do a better job at that, if there's a different
+ * between line separator and paragraph separator, it should be that
+ * line separator acts just like a forced break, with no effect on
+ * justification.
+ */
goto done;
}
}
done:
- layout->is_wrapped |= wrapped;
pango_layout_line_postprocess (line, state, wrapped);
add_line (line, state);
state->line_of_par++;
@@ -5091,8 +5087,7 @@
if (G_UNLIKELY (state->line_width >= 0 &&
should_ellipsize_current_line (line->layout, state)))
{
- ellipsized = _pango_layout_line_ellipsize (line, state->attrs, state->line_width);
- line->layout->is_ellipsized |= ellipsized;
+ ellipsized = _pango_layout_line_ellipsize (line, state->attrs, state->line_width);
}
/* Truncate the logical-final whitespace in the line if we broke the line at it
@@ -5126,6 +5121,9 @@
}
DEBUG ("after justification", line, state);
+
+ line->layout->is_wrapped |= wrapped;
+ line->layout->is_ellipsized |= ellipsized;
}
static void
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]