[gtk+/rendering-cleanup-next: 16/199] entry: Split else ifs into separate if statements
- From: Benjamin Otte <otte src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gtk+/rendering-cleanup-next: 16/199] entry: Split else ifs into separate if statements
- Date: Thu, 23 Sep 2010 21:20:26 +0000 (UTC)
commit ce885dbb9d55cf8824ad88a703d039d6cf5fa434
Author: Benjamin Otte <otte redhat com>
Date: Fri Sep 3 17:25:26 2010 +0200
entry: Split else ifs into separate if statements
Preparation for the draw vfunc
gtk/gtkentry.c | 120 +++++++++++++++++++++++++++++--------------------------
1 files changed, 63 insertions(+), 57 deletions(-)
---
diff --git a/gtk/gtkentry.c b/gtk/gtkentry.c
index 351086b..a1744c3 100644
--- a/gtk/gtkentry.c
+++ b/gtk/gtkentry.c
@@ -264,9 +264,10 @@ static void gtk_entry_size_request (GtkWidget *widget,
static void gtk_entry_size_allocate (GtkWidget *widget,
GtkAllocation *allocation);
static void gtk_entry_draw_frame (GtkWidget *widget,
- GdkEventExpose *event);
+ cairo_t *cr);
static void gtk_entry_draw_progress (GtkWidget *widget,
- GdkEventExpose *event);
+ cairo_t *cr,
+ GdkWindow *window);
static gint gtk_entry_expose (GtkWidget *widget,
GdkEventExpose *event);
static gint gtk_entry_button_press (GtkWidget *widget,
@@ -416,8 +417,10 @@ static void gtk_entry_enter_text (GtkEntry *entry,
static void gtk_entry_set_positions (GtkEntry *entry,
gint current_pos,
gint selection_bound);
-static void gtk_entry_draw_text (GtkEntry *entry);
+static void gtk_entry_draw_text (GtkEntry *entry,
+ cairo_t *cr);
static void gtk_entry_draw_cursor (GtkEntry *entry,
+ cairo_t *cr,
CursorType type);
static PangoLayout *gtk_entry_ensure_layout (GtkEntry *entry,
gboolean include_preedit);
@@ -3192,6 +3195,7 @@ should_prelight (GtkEntry *entry,
static void
draw_icon (GtkWidget *widget,
+ cairo_t *cr,
GtkEntryIconPosition icon_pos)
{
GtkEntry *entry = GTK_ENTRY (widget);
@@ -3199,7 +3203,6 @@ draw_icon (GtkWidget *widget,
EntryIconInfo *icon_info = priv->icons[icon_pos];
GdkPixbuf *pixbuf;
gint x, y, width, height;
- cairo_t *cr;
if (!icon_info)
return;
@@ -3257,10 +3260,8 @@ draw_icon (GtkWidget *widget,
pixbuf = temp_pixbuf;
}
- cr = gdk_cairo_create (icon_info->window);
gdk_cairo_set_source_pixbuf (cr, pixbuf, x, y);
cairo_paint (cr);
- cairo_destroy (cr);
g_object_unref (pixbuf);
}
@@ -3268,7 +3269,7 @@ draw_icon (GtkWidget *widget,
static void
gtk_entry_draw_frame (GtkWidget *widget,
- GdkEventExpose *event)
+ cairo_t *cr)
{
GtkEntryPrivate *priv = GTK_ENTRY_GET_PRIVATE (widget);
GtkStyle *style;
@@ -3306,12 +3307,12 @@ gtk_entry_draw_frame (GtkWidget *widget,
state = gtk_widget_has_focus (widget) ?
GTK_STATE_ACTIVE : gtk_widget_get_state (widget);
- gtk_paint_shadow (style, window,
+ gtk_cairo_paint_shadow (style, cr,
state, priv->shadow_type,
- &event->area, widget, "entry", x, y, width, height);
+ widget, "entry", x, y, width, height);
- gtk_entry_draw_progress (widget, event);
+ gtk_entry_draw_progress (widget, cr, window);
if (gtk_widget_has_focus (widget) && !priv->interior_focus)
{
@@ -3320,9 +3321,9 @@ gtk_entry_draw_frame (GtkWidget *widget,
width += 2 * priv->focus_width;
height += 2 * priv->focus_width;
- gtk_paint_focus (style, window,
+ gtk_cairo_paint_focus (style, cr,
gtk_widget_get_state (widget),
- &event->area, widget, "entry",
+ widget, "entry",
0, 0, width, height);
}
}
@@ -3413,7 +3414,8 @@ get_progress_area (GtkWidget *widget,
static void
gtk_entry_draw_progress (GtkWidget *widget,
- GdkEventExpose *event)
+ cairo_t *cr,
+ GdkWindow *window)
{
gint x, y, width, height;
GtkStateType state;
@@ -3423,11 +3425,11 @@ gtk_entry_draw_progress (GtkWidget *widget,
if ((width <= 0) || (height <= 0))
return;
- if (event->window != gtk_widget_get_window (widget))
+ if (window != gtk_widget_get_window (widget))
{
gint pos_x, pos_y;
- gdk_window_get_position (event->window, &pos_x, &pos_y);
+ gdk_window_get_position (window, &pos_x, &pos_y);
x -= pos_x;
y -= pos_y;
@@ -3437,9 +3439,9 @@ gtk_entry_draw_progress (GtkWidget *widget,
if (!gtk_widget_get_sensitive (widget))
state = GTK_STATE_INSENSITIVE;
- gtk_paint_box (gtk_widget_get_style (widget), event->window,
+ gtk_cairo_paint_box (gtk_widget_get_style (widget), cr,
state, GTK_SHADOW_OUT,
- &event->area, widget, "entry-progress",
+ widget, "entry-progress",
x, y,
width, height);
}
@@ -3452,6 +3454,12 @@ gtk_entry_expose (GtkWidget *widget,
GtkStyle *style;
GtkStateType state;
GtkEntryPrivate *priv = GTK_ENTRY_GET_PRIVATE (entry);
+ cairo_t *cr;
+ int i;
+
+ cr = gdk_cairo_create (event->window);
+ gdk_cairo_region (cr, event->region);
+ cairo_clip (cr);
style = gtk_widget_get_style (widget);
@@ -3459,60 +3467,57 @@ gtk_entry_expose (GtkWidget *widget,
GTK_STATE_ACTIVE : gtk_widget_get_state (widget);
if (gtk_widget_get_window (widget) == event->window)
- {
- gtk_entry_draw_frame (widget, event);
- }
- else if (entry->text_area == event->window)
+ gtk_entry_draw_frame (widget, cr);
+
+ if (entry->text_area == event->window)
{
gint width, height;
gdk_drawable_get_size (entry->text_area, &width, &height);
- gtk_paint_flat_box (style, entry->text_area,
+ gtk_cairo_paint_flat_box (style, cr,
state, GTK_SHADOW_NONE,
- &event->area, widget, "entry_bg",
+ widget, "entry_bg",
0, 0, width, height);
- gtk_entry_draw_progress (widget, event);
+ gtk_entry_draw_progress (widget, cr, entry->text_area);
if (entry->dnd_position != -1)
- gtk_entry_draw_cursor (GTK_ENTRY (widget), CURSOR_DND);
+ gtk_entry_draw_cursor (GTK_ENTRY (widget), cr, CURSOR_DND);
- gtk_entry_draw_text (GTK_ENTRY (widget));
+ gtk_entry_draw_text (GTK_ENTRY (widget), cr);
/* When no text is being displayed at all, don't show the cursor */
if (gtk_entry_get_display_mode (entry) != DISPLAY_BLANK &&
gtk_widget_has_focus (widget) &&
entry->selection_bound == entry->current_pos && entry->cursor_visible)
- gtk_entry_draw_cursor (GTK_ENTRY (widget), CURSOR_STANDARD);
+ gtk_entry_draw_cursor (GTK_ENTRY (widget), cr, CURSOR_STANDARD);
}
- else
+
+ for (i = 0; i < MAX_ICONS; i++)
{
- int i;
+ EntryIconInfo *icon_info = priv->icons[i];
- for (i = 0; i < MAX_ICONS; i++)
+ if (icon_info != NULL && event->window == icon_info->window)
{
- EntryIconInfo *icon_info = priv->icons[i];
-
- if (icon_info != NULL && event->window == icon_info->window)
- {
- gint width, height;
+ gint width, height;
- gdk_drawable_get_size (icon_info->window, &width, &height);
+ gdk_drawable_get_size (icon_info->window, &width, &height);
- gtk_paint_flat_box (style, icon_info->window,
- state, GTK_SHADOW_NONE,
- NULL, widget, "entry_bg",
- 0, 0, width, height);
+ gtk_cairo_paint_flat_box (style, cr,
+ state, GTK_SHADOW_NONE,
+ widget, "entry_bg",
+ 0, 0, width, height);
- gtk_entry_draw_progress (widget, event);
- draw_icon (widget, i);
+ gtk_entry_draw_progress (widget, cr, icon_info->window);
+ draw_icon (widget, cr, i);
- break;
- }
+ break;
}
}
+ cairo_destroy (cr);
+
return FALSE;
}
@@ -5582,10 +5587,10 @@ draw_text_with_color (GtkEntry *entry, cairo_t *cr, GdkColor *default_color)
}
static void
-gtk_entry_draw_text (GtkEntry *entry)
+gtk_entry_draw_text (GtkEntry *entry,
+ cairo_t *cr)
{
GtkWidget *widget = GTK_WIDGET (entry);
- cairo_t *cr;
/* Nothing to display at all */
if (gtk_entry_get_display_mode (entry) == DISPLAY_BLANK)
@@ -5611,8 +5616,6 @@ gtk_entry_draw_text (GtkEntry *entry)
&progress_x, &progress_y,
&progress_width, &progress_height);
- cr = gdk_cairo_create (entry->text_area);
-
/* If the color is the same, or the progress area has a zero
* size, then we only need to draw once. */
if ((text_color.pixel == bar_text_color.pixel) ||
@@ -5624,6 +5627,8 @@ gtk_entry_draw_text (GtkEntry *entry)
{
gdk_drawable_get_size (entry->text_area, &width, &height);
+ cairo_save (cr);
+
cairo_rectangle (cr, 0, 0, width, height);
cairo_clip (cr);
cairo_save (cr);
@@ -5648,14 +5653,15 @@ gtk_entry_draw_text (GtkEntry *entry)
cairo_clip (cr);
draw_text_with_color (entry, cr, &bar_text_color);
- }
- cairo_destroy (cr);
+ cairo_restore (cr);
+ }
}
}
static void
draw_insertion_cursor (GtkEntry *entry,
+ cairo_t *cr,
GdkRectangle *cursor_location,
gboolean is_primary,
PangoDirection direction,
@@ -5669,13 +5675,14 @@ draw_insertion_cursor (GtkEntry *entry,
else
text_dir = GTK_TEXT_DIR_RTL;
- gtk_draw_insertion_cursor (widget, entry->text_area, NULL,
+ gtk_cairo_draw_insertion_cursor (widget, cr,
cursor_location,
is_primary, text_dir, draw_arrow);
}
static void
gtk_entry_draw_cursor (GtkEntry *entry,
+ cairo_t *cr,
CursorType type)
{
GtkWidget *widget = GTK_WIDGET (entry);
@@ -5750,14 +5757,14 @@ gtk_entry_draw_cursor (GtkEntry *entry,
cursor_location.width = 0;
cursor_location.height = text_area_height - inner_border.top - inner_border.bottom;
- draw_insertion_cursor (entry,
+ draw_insertion_cursor (entry, cr,
&cursor_location, TRUE, dir1,
dir2 != PANGO_DIRECTION_NEUTRAL);
if (dir2 != PANGO_DIRECTION_NEUTRAL)
{
cursor_location.x = xoffset + x2;
- draw_insertion_cursor (entry,
+ draw_insertion_cursor (entry, cr,
&cursor_location, FALSE, dir2,
TRUE);
}
@@ -5766,9 +5773,10 @@ gtk_entry_draw_cursor (GtkEntry *entry,
{
GdkColor cursor_color;
GdkRectangle rect;
- cairo_t *cr;
gint x, y;
+ cairo_save (cr);
+
get_layout_position (entry, &x, &y);
rect.x = PANGO_PIXELS (cursor_rect.x) + x;
@@ -5776,8 +5784,6 @@ gtk_entry_draw_cursor (GtkEntry *entry,
rect.width = PANGO_PIXELS (cursor_rect.width);
rect.height = PANGO_PIXELS (cursor_rect.height);
- cr = gdk_cairo_create (entry->text_area);
-
_gtk_widget_get_cursor_color (widget, &cursor_color);
gdk_cairo_set_source_color (cr, &cursor_color);
gdk_cairo_rectangle (cr, &rect);
@@ -5792,7 +5798,7 @@ gtk_entry_draw_cursor (GtkEntry *entry,
pango_cairo_show_layout (cr, layout);
}
- cairo_destroy (cr);
+ cairo_restore (cr);
}
}
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]