[pango/ltr-line-breaking-fixes: 1/6] Simplify breakpoint disabling




commit 260dc99fb8b5a566c9f77305f08dd6933a24615e
Author: Matthias Clasen <mclasen redhat com>
Date:   Fri Nov 12 11:37:00 2021 -0500

    Simplify breakpoint disabling
    
    We only want this inside process_item, so we can
    make this a purely local thing, without modifying
    log_attrs.

 pango/pango-layout.c | 39 +++++++++------------------------------
 1 file changed, 9 insertions(+), 30 deletions(-)
---
diff --git a/pango/pango-layout.c b/pango/pango-layout.c
index fad92b3b..34787f0e 100644
--- a/pango/pango-layout.c
+++ b/pango/pango-layout.c
@@ -3525,29 +3525,6 @@ shape_tab (PangoLayoutLine  *line,
     }
 }
 
-#define DISABLE_BREAKPOINT_FLAG (1 << 16)
-
-static inline void
-disable_breakpoint (PangoLayout *layout,
-                    int          offset)
-{
-  layout->log_attrs[offset].reserved |= DISABLE_BREAKPOINT_FLAG;
-}
-
-static inline gboolean
-breakpoint_is_disabled (PangoLayout *layout,
-                        int          offset)
-{
-  return (layout->log_attrs[offset].reserved & DISABLE_BREAKPOINT_FLAG) != 0;
-}
-
-static void
-clear_breakpoint_flags (PangoLayout *layout)
-{
-  for (int i = 0; i < layout->n_chars + 1; i++)
-    layout->log_attrs[i].reserved = 0;
-}
-
 static inline gboolean
 can_break_at (PangoLayout *layout,
               gint         offset,
@@ -3565,8 +3542,6 @@ can_break_at (PangoLayout *layout,
 
   if (offset == layout->n_chars)
     return TRUE;
-  else if (breakpoint_is_disabled (layout, offset))
-    return FALSE;
   else if (wrap == PANGO_WRAP_WORD)
     return layout->log_attrs[offset].is_line_break;
   else if (wrap == PANGO_WRAP_CHAR)
@@ -3926,6 +3901,7 @@ process_item (PangoLayout     *layout,
       int orig_width = width;
       int break_extra_width = 0;
       gboolean retrying_with_char_breaks = FALSE;
+      gboolean *break_disabled;
 
       if (processing_new_item)
         {
@@ -3938,6 +3914,9 @@ process_item (PangoLayout     *layout,
           pango_glyph_item_get_logical_widths (&glyph_item, layout->text, state->log_widths);
         }
 
+      break_disabled = g_alloca (sizeof (gboolean) * (item->num_chars + 1));
+      memset (break_disabled, 0, sizeof (gboolean) * (item->num_chars + 1));
+
     retry_break:
 
       /* See how much of the item we can stuff in the line. */
@@ -3951,7 +3930,8 @@ process_item (PangoLayout     *layout,
             break;
 
           /* If there are no previous runs we have to take care to grab at least one char. */
-          if (can_break_at (layout, state->start_offset + num_chars, retrying_with_char_breaks) &&
+          if (!break_disabled[num_chars] &&
+              can_break_at (layout, state->start_offset + num_chars, retrying_with_char_breaks) &&
               (num_chars > 0 || line->runs))
             {
               break_num_chars = num_chars;
@@ -4012,13 +3992,14 @@ process_item (PangoLayout     *layout,
 
               if (break_needs_hyphen (layout, state, break_num_chars))
                 new_item->analysis.flags |= PANGO_ANALYSIS_FLAG_NEED_HYPHEN;
+
               /* Add the width back, to the line, reshape, subtract the new width */
               state->remaining_width += break_width;
               insert_run (line, state, new_item, FALSE);
 
               break_width = pango_glyph_string_get_width (((PangoGlyphItem *)(line->runs->data))->glyphs);
               if (break_width > state->remaining_width &&
-                  !breakpoint_is_disabled (layout, break_num_chars) &&
+                  !break_disabled[break_num_chars] &&
                   break_num_chars > 1)
                 {
                   /* Unsplit the item, disable the breakpoint, try again */
@@ -4027,7 +4008,7 @@ process_item (PangoLayout     *layout,
                   pango_item_free (new_item);
                   pango_item_unsplit (item, length, break_num_chars);
 
-                  disable_breakpoint (layout, break_num_chars);
+                  break_disabled[break_num_chars] = TRUE;
 
                   num_chars = item->num_chars;
                   width = orig_width;
@@ -4644,8 +4625,6 @@ pango_layout_check_lines (PangoLayout *layout)
     }
   while (!done);
 
-  clear_breakpoint_flags (layout);
-
   g_free (state.log_widths);
   g_list_free_full (state.baseline_shifts, g_free);
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]