Better default icon management
authorRodolfo García Peñas (kix) <kix@kix.es>
Tue, 23 Oct 2012 21:59:57 +0000 (23 23:59 +0200)
committerCarlos R. Mafra <crmafra@gmail.com>
Sun, 28 Oct 2012 10:58:43 +0000 (28 10:58 +0000)
This patch replaces the Pixmap icons from screen.h with only one
RImage. This image is not processed yet, therefore could be used
for icons with title or without it (replacing def_ticon_pixmap and
def_icon_pixmap variables).

Now the code is better because the Pixmap is generated and saved in
the icon structure.

See that before this patch, the icon->pixmap was set to None!!
With this patch, the icon->pixmap is saved, using the common
method used in the other functions to create Pixmaps, the function
icon_update_pixmap().

src/defaults.c
src/icon.c
src/screen.h

index 4bf9aff..458093c 100644 (file)
@@ -2381,18 +2381,14 @@ static int setIconTile(WScreen * scr, WDefaultEntry * entry, WTexture ** texture
 
        scr->icon_tile_pixmap = pixmap;
 
-       if (scr->def_icon_pixmap) {
-               XFreePixmap(dpy, scr->def_icon_pixmap);
-               scr->def_icon_pixmap = None;
-       }
-       if (scr->def_ticon_pixmap) {
-               XFreePixmap(dpy, scr->def_ticon_pixmap);
-               scr->def_ticon_pixmap = None;
+       if (scr->def_icon_rimage) {
+               RReleaseImage(scr->def_icon_rimage);
+               scr->def_icon_rimage = NULL;
        }
 
-       if (scr->icon_back_texture) {
+       if (scr->icon_back_texture)
                wTextureDestroy(scr, (WTexture *) scr->icon_back_texture);
-       }
+
        scr->icon_back_texture = wTextureMakeSolid(scr, &((*texture)->any.color));
 
        if (scr->clip_balloon)
index 2739ca1..b0da0d8 100644 (file)
@@ -628,28 +628,20 @@ void wIconUpdate(WIcon *icon)
 
 static void get_pixmap_icon_from_user_icon(WIcon *icon)
 {
-       RImage *image = NULL;
        WScreen *scr = icon->core->screen_ptr;
 
+       /* If the icon has image, update it and continue */
        if (icon->file_image) {
                icon_update_pixmap(icon, icon->file_image);
-       } else {
-               /* make default icons */
-               if (!scr->def_icon_pixmap) {
-                       image = get_default_image(scr);
-                       scr->def_icon_pixmap = makeIcon(scr, image, False, False, icon->tile_type, icon->highlighted);
-                       scr->def_ticon_pixmap = makeIcon(scr, image, True, False, icon->tile_type, icon->highlighted);
-                       if (image)
-                               RReleaseImage(image);
-               }
+               return;
+       }
 
-               if (icon->show_title)
-                       XSetWindowBackgroundPixmap(dpy, icon->core->window, scr->def_ticon_pixmap);
-               else
-                       XSetWindowBackgroundPixmap(dpy, icon->core->window, scr->def_icon_pixmap);
+       /* If the icon don't have image, we should use the default image. */
+       if (!scr->def_icon_rimage)
+               scr->def_icon_rimage = get_default_image(scr);
 
-               icon->pixmap = None;
-       }
+       /* Now, create the pixmap using the default (saved) image */
+       icon_update_pixmap(icon, scr->def_icon_rimage);
 }
 
 /* This function creates the RImage using the default icon */
index f021096..dfdbd63 100644 (file)
@@ -229,13 +229,12 @@ typedef struct _WScreen {
 
     struct RImage *icon_tile;
     struct RImage *clip_tile;
-    Pixmap icon_tile_pixmap;          /* for app supplied icons */
 
-    Pixmap def_icon_pixmap;           /* default icons */
-    Pixmap def_ticon_pixmap;
+    Pixmap icon_tile_pixmap;           /* For app supplied icons */
 
-    struct WDialogData *dialog_data;
+    struct RImage *def_icon_rimage;    /* Default RImage icon */
 
+    struct WDialogData *dialog_data;
 
     struct W_GeometryView *gview;      /* size/position view */