From f24633751a7961da60e151665a12c1aedb4734f9 Mon Sep 17 00:00:00 2001 From: infirit Date: Tue, 28 Jul 2015 14:13:48 +0200 Subject: Use accessor functions instead of direct access --- src/themes/slider/theme.c | 15 +++++++++------ src/themes/standard/theme.c | 2 +- 2 files changed, 10 insertions(+), 7 deletions(-) (limited to 'src') diff --git a/src/themes/slider/theme.c b/src/themes/slider/theme.c index c6673d4..6ea75eb 100644 --- a/src/themes/slider/theme.c +++ b/src/themes/slider/theme.c @@ -117,6 +117,7 @@ static void fill_background(GtkWidget* widget, WindowData* windata, cairo_t* cr) GdkRGBA fg_color; GdkRGBA bg_color; #else + GtkStyle *style; GdkColor color; #endif double r, g, b; @@ -152,7 +153,9 @@ static void fill_background(GtkWidget* widget, WindowData* windata, cairo_t* cr) cairo_stroke(cr); #else - color = widget->style->bg[GTK_STATE_NORMAL]; + style = gtk_widget_get_style (widget); + + color = style->bg[GTK_STATE_NORMAL]; r = (float) color.red / 65535.0; g = (float) color.green / 65535.0; b = (float) color.blue / 65535.0; @@ -162,7 +165,7 @@ static void fill_background(GtkWidget* widget, WindowData* windata, cairo_t* cr) /* Should we show urgency somehow? Probably doesn't * have any meaningful value to the user... */ - color = widget->style->text_aa[GTK_STATE_NORMAL]; + color = style->text_aa[GTK_STATE_NORMAL]; r = (float) color.red / 65535.0; g = (float) color.green / 65535.0; b = (float) color.blue / 65535.0; @@ -461,13 +464,13 @@ static void override_style(GtkWidget* widget, GtkStyle* previous_style) GdkColor fg; GdkColor bg; - style = gtk_style_copy(widget->style); + style = gtk_style_copy(gtk_widget_get_style (widget)); if (previous_style == NULL || (previous_style != NULL && (previous_style->bg[GTK_STATE_NORMAL].red != style->bg[GTK_STATE_NORMAL].red || previous_style->bg[GTK_STATE_NORMAL].green != style->bg[GTK_STATE_NORMAL].green || previous_style->bg[GTK_STATE_NORMAL].blue != style->bg[GTK_STATE_NORMAL].blue))) { state = (GtkStateType) 0; - while (state < (GtkStateType) G_N_ELEMENTS(widget->style->bg)) + while (state < (GtkStateType) G_N_ELEMENTS(style->bg)) { color_reverse(&style->bg[state], &bg); gtk_widget_modify_bg(widget, state, &bg); @@ -479,7 +482,7 @@ static void override_style(GtkWidget* widget, GtkStyle* previous_style) { state = (GtkStateType) 0; - while (state < (GtkStateType) G_N_ELEMENTS(widget->style->fg)) + while (state < (GtkStateType) G_N_ELEMENTS(style->fg)) { color_reverse(&style->fg[state], &fg); gtk_widget_modify_fg(widget, state, &fg); @@ -930,7 +933,7 @@ static gboolean on_countdown_expose(GtkWidget* pie, GdkEventExpose* event, Windo g = color.green; b = color.blue; #else - color = windata->win->style->bg[GTK_STATE_NORMAL]; + color = style->bg[GTK_STATE_NORMAL]; r = (float) color.red / 65535.0; g = (float) color.green / 65535.0; b = (float) color.blue / 65535.0; diff --git a/src/themes/standard/theme.c b/src/themes/standard/theme.c index 9796959..7d56893 100644 --- a/src/themes/standard/theme.c +++ b/src/themes/standard/theme.c @@ -456,7 +456,7 @@ static void draw_border(GtkWidget* widget, WindowData *windata, cairo_t* cr) #if GTK_CHECK_VERSION(3, 0, 0) /* FIXME!!! */ #else - gdk_window_shape_combine_region(windata->win->window, windata->window_region, 0, 0); + gdk_window_shape_combine_region (gtk_widget_get_window (windata->win), windata->window_region, 0, 0); #endif g_free(windata->border_points); windata->border_points = NULL; -- cgit v1.2.1