[gtk+] iconhelper: Don't store surface size



commit b2584eb8edcef4ff23fe1e0c2cb3613412b36ba0
Author: Benjamin Otte <otte redhat com>
Date:   Sun Nov 29 20:21:25 2015 +0100

    iconhelper: Don't store surface size
    
    Instead, compute it on demand.
    
    This avoids having to cache it in lots of places and simplifies code.

 gtk/gtkiconhelper.c |   21 +--------------------
 1 files changed, 1 insertions(+), 20 deletions(-)
---
diff --git a/gtk/gtkiconhelper.c b/gtk/gtkiconhelper.c
index fff1961..e835034 100644
--- a/gtk/gtkiconhelper.c
+++ b/gtk/gtkiconhelper.c
@@ -43,8 +43,6 @@ struct _GtkIconHelperPrivate {
   GtkStateFlags last_rendered_state;
 
   cairo_surface_t *rendered_surface;
-  gint rendered_surface_width;
-  gint rendered_surface_height;
   GtkStateFlags last_surface_state;
   gint last_surface_scale;
 };
@@ -548,10 +546,6 @@ ensure_surface_from_surface (GtkIconHelper   *self,
 {
   self->priv->rendered_surface =
     cairo_surface_reference (orig_surface);
-
-  get_surface_size (self, orig_surface,
-                   &self->priv->rendered_surface_width,
-                   &self->priv->rendered_surface_height);
 }
 
 static gboolean
@@ -631,9 +625,6 @@ ensure_surface_from_pixbuf (GtkIconHelper   *self,
   g_object_unref (pixbuf);
   pixbuf = stated;
 
-  self->priv->rendered_surface_width = (width + scale - 1) / scale;
-  self->priv->rendered_surface_height = (height + scale - 1) / scale;
-
   self->priv->rendered_surface = gdk_cairo_surface_create_from_pixbuf (pixbuf, scale, self->priv->window);
   g_object_unref (pixbuf);
 }
@@ -653,11 +644,6 @@ ensure_surface_for_icon_set (GtkIconHelper *self,
                                      self->priv->icon_size,
                                      scale, self->priv->window);
   G_GNUC_END_IGNORE_DEPRECATIONS;
-
-  if (self->priv->rendered_surface)
-    get_surface_size (self, self->priv->rendered_surface, 
-                     &self->priv->rendered_surface_width, 
-                     &self->priv->rendered_surface_height);
 }
 
 static void
@@ -706,10 +692,6 @@ ensure_stated_surface_from_info (GtkIconHelper *self,
     {
       surface = gdk_cairo_surface_create_from_pixbuf (destination, scale, self->priv->window);
 
-      self->priv->rendered_surface_width = 
-       (gdk_pixbuf_get_width (destination) + scale - 1) / scale;
-      self->priv->rendered_surface_height = 
-       (gdk_pixbuf_get_height (destination) + scale - 1) / scale;
       g_object_unref (destination);
     }
 
@@ -869,8 +851,7 @@ _gtk_icon_helper_get_size (GtkIconHelper *self,
 
       if (surface != NULL)
         {
-          width = self->priv->rendered_surface_width;
-          height = self->priv->rendered_surface_height;
+          get_surface_size (self, surface, &width, &height);
           cairo_surface_destroy (surface);
         }
       else if (self->priv->icon_size != GTK_ICON_SIZE_INVALID)


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