pango r2672 - in trunk: . pango

[behdad svn gnome org]

Author: behdad
Date: Wed Aug  6 08:35:12 2008
New Revision: 2672
URL: http://svn.gnome.org/viewvc/pango?rev=2672&view=rev

Log:
2008-07-19  Behdad Esfahbod  <behdad gnome org>

        * pango/pango-layout.c (process_line): Justify lines broken by U+2028
        LINE SEPARATOR.  Also affects return value of pango_layout_is_wrapped().

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	Wed Aug  6 08:35:12 2008
@@ -3545,15 +3545,9 @@
 	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.
-	   */
+	  /* A line-separate is just a forced break.  Set wrapped, so we do
+	   * justification */
+	  wrapped = TRUE;
 	  goto done;
 	}
     }