]> Dogcows Code - chaz/tint2/blobdiff - src/launcher/launcher.c
launcher: use the proper fallback icon; use getline (dynamically resized buffer)...
[chaz/tint2] / src / launcher / launcher.c
index cf9e1365ea541d630590c7d491db39688ecbd9b1..1166dc0d7728a66c7e372d3084d16aee50bfe73c 100644 (file)
@@ -37,7 +37,7 @@ int launcher_max_icon_size;
 char *icon_theme_name; 
 XSettingsClient *xsettings_client;
 
-#define ICON_FALLBACK "exec"
+#define ICON_FALLBACK "application-x-executable"
 
 char *icon_path(Launcher *launcher, const char *icon_name, int size);
 void launcher_load_themes(Launcher *launcher);
@@ -78,6 +78,8 @@ void init_launcher_panel(void *p)
        launcher->area._resize = resize_launcher;
        launcher->area.resize = 1;
        launcher->area.redraw = 1;
+       if (launcher->area.bg == 0)
+               launcher->area.bg = &g_array_index(backgrounds, Background, 0);
 
        // check consistency
        if (launcher->list_apps == NULL)
@@ -403,7 +405,8 @@ void expand_exec(DesktopEntry *entry, const char *path)
 int launcher_read_desktop_file(const char *path, DesktopEntry *entry)
 {
        FILE *fp;
-       char line[4096];
+       char *line = NULL;
+       size_t line_size;
        char *key, *value;
 
        entry->name = entry->icon = entry->exec = NULL;
@@ -413,7 +416,7 @@ int launcher_read_desktop_file(const char *path, DesktopEntry *entry)
                return 0;
        }
 
-       while (fgets(line, sizeof(line), fp) != NULL) {
+       while (getline(&line, &line_size, fp) >= 0) {
                int len = strlen(line);
                if (len == 0)
                        continue;
@@ -434,6 +437,7 @@ int launcher_read_desktop_file(const char *path, DesktopEntry *entry)
 
        expand_exec(entry, path);
        
+       free(line);
        return 1;
 }
 
@@ -463,7 +467,8 @@ IconTheme *load_theme(char *name)
        IconTheme *theme;
        char *file_name;
        FILE *f;
-       char line[2048];
+       char *line = NULL;
+       size_t line_size;
 
        if (name == NULL)
                return NULL;
@@ -500,7 +505,7 @@ IconTheme *load_theme(char *name)
 
        IconThemeDir *current_dir = NULL;
        int inside_header = 1;
-       while (fgets(line, sizeof(line), f) != NULL) {
+       while (getline(&line, &line_size, f) >= 0) {
                char *key, *value;
                
                int line_len = strlen(line);
@@ -594,6 +599,8 @@ IconTheme *load_theme(char *name)
        }
        fclose(f);
        
+       free(line);
+       
        return theme;
 }
 
@@ -790,17 +797,30 @@ char *icon_path(Launcher *launcher, const char *icon_name, int size)
        }
 
        GSList *basenames = NULL;
-       char *file_name = g_build_filename(g_get_home_dir(), ".icons", NULL);
-       basenames = g_slist_append(basenames, file_name);
+       char *home_icons = g_build_filename(g_get_home_dir(), ".icons", NULL);
+       basenames = g_slist_append(basenames, home_icons);
        basenames = g_slist_append(basenames, "/usr/share/icons");
        basenames = g_slist_append(basenames, "/usr/share/pixmaps");
 
        GSList *extensions = NULL;
-       extensions = g_slist_append(extensions, "png");
-       extensions = g_slist_append(extensions, "xpm");
+       extensions = g_slist_append(extensions, ".png");
+       extensions = g_slist_append(extensions, ".xpm");
+       // if the icon name already contains one of the extensions (e.g. vlc.png instead of vlc) add a special entry
+       GSList *ext;
+       for (ext = extensions; ext; ext = g_slist_next(ext)) {
+               char *extension = (char*) ext->data;
+               if (strlen(icon_name) > strlen(extension) &&
+                       strcmp(extension, icon_name + strlen(icon_name) - strlen(extension)) == 0) {
+                       extensions = g_slist_append(extensions, "");
+                       break;
+               }
+       }
 
-       // Stage 1: exact size match
        GSList *theme;
+       // Stage 1: exact size match
+       // the theme must have a higher priority than having an exact size match, so we will just use
+       // the code that searches for the best size match (it will find the exact size match if one exists)
+       /*
        for (theme = launcher->list_themes; theme; theme = g_slist_next(theme)) {
                GSList *dir;
                for (dir = ((IconTheme*)theme->data)->list_directories; dir; dir = g_slist_next(dir)) {
@@ -816,11 +836,13 @@ char *icon_path(Launcher *launcher, const char *icon_name, int size)
                                                char *file_name = malloc(strlen(base_name) + strlen(theme_name) +
                                                        strlen(dir_name) + strlen(icon_name) + strlen(extension) + 100);
                                                // filename = directory/$(themename)/subdirectory/iconname.extension
-                                               sprintf(file_name, "%s/%s/%s/%s.%s", base_name, theme_name, dir_name, icon_name, extension);
+                                               sprintf(file_name, "%s/%s/%s/%s%s", base_name, theme_name, dir_name, icon_name, extension);
+                                               //printf("found exact: %s\n", file_name);
                                                //printf("checking %s\n", file_name);
                                                if (g_file_test(file_name, G_FILE_TEST_EXISTS)) {
                                                        g_slist_free(basenames);
                                                        g_slist_free(extensions);
+                                                       g_free(home_icons);
                                                        return file_name;
                                                } else {
                                                        free(file_name);
@@ -832,15 +854,23 @@ char *icon_path(Launcher *launcher, const char *icon_name, int size)
                }
        }
        g_free (file_name);
+       */
 
        // Stage 2: best size match
        // Contrary to the freedesktop spec, we are not choosing the closest icon in size, but the next larger icon
        // otherwise the quality is usually crap (for size 22, if you can choose 16 or 32, you're better with 32)
        // We do fallback to the closest size if we cannot find a larger or equal icon
+       
+       // These 3 variables are used for keeping the closest size match
        int minimal_size = INT_MAX;
        char *best_file_name = NULL;
+       GSList *best_file_theme = NULL;
+       
+       // These 3 variables are used for keeping the next larger match
        int next_larger_size = -1;
        char *next_larger = NULL;
+       GSList *next_larger_theme = NULL;
+       
        for (theme = launcher->list_themes; theme; theme = g_slist_next(theme)) {
                GSList *dir;
                for (dir = ((IconTheme*)theme->data)->list_directories; dir; dir = g_slist_next(dir)) {
@@ -855,23 +885,32 @@ char *icon_path(Launcher *launcher, const char *icon_name, int size)
                                        char *file_name = malloc(strlen(base_name) + strlen(theme_name) +
                                        strlen(dir_name) + strlen(icon_name) + strlen(extension) + 100);
                                        // filename = directory/$(themename)/subdirectory/iconname.extension
-                                       sprintf(file_name, "%s/%s/%s/%s.%s", base_name, theme_name, dir_name, icon_name, extension);
+                                       sprintf(file_name, "%s/%s/%s/%s%s", base_name, theme_name, dir_name, icon_name, extension);
                                        if (g_file_test(file_name, G_FILE_TEST_EXISTS)) {
-                                               if (directory_size_distance((IconThemeDir*)dir->data, size) < minimal_size) {
+                                               //printf("found: %s\n", file_name);
+                                               // Closest match
+                                               if (directory_size_distance((IconThemeDir*)dir->data, size) < minimal_size && (!best_file_theme ? 1 : theme == best_file_theme)) {
                                                        if (best_file_name) {
                                                                free(best_file_name);
                                                                best_file_name = NULL;
                                                        }
                                                        best_file_name = strdup(file_name);
                                                        minimal_size = directory_size_distance((IconThemeDir*)dir->data, size);
+                                                       best_file_theme = theme;
+                                                       //printf("best_file_name = %s; minimal_size = %d\n", best_file_name, minimal_size);
                                                }
-                                               if (((IconThemeDir*)dir->data)->size >= size && (next_larger_size == -1 || ((IconThemeDir*)dir->data)->size < next_larger_size)) {
+                                               // Next larger match
+                                               if (((IconThemeDir*)dir->data)->size >= size &&
+                                                       (next_larger_size == -1 || ((IconThemeDir*)dir->data)->size < next_larger_size) &&
+                                                       (!next_larger_theme ? 1 : theme == next_larger_theme)) {
                                                        if (next_larger) {
                                                                free(next_larger);
                                                                next_larger = NULL;
                                                        }
                                                        next_larger = strdup(file_name);
                                                        next_larger_size = ((IconThemeDir*)dir->data)->size;
+                                                       next_larger_theme = theme;
+                                                       //printf("next_larger = %s; next_larger_size = %d\n", next_larger, next_larger_size);
                                                }
                                        }
                                        free(file_name);
@@ -883,11 +922,13 @@ char *icon_path(Launcher *launcher, const char *icon_name, int size)
                g_slist_free(basenames);
                g_slist_free(extensions);
                free(best_file_name);
+               g_free(home_icons);
                return next_larger;
        }
        if (best_file_name) {
                g_slist_free(basenames);
                g_slist_free(extensions);
+               g_free(home_icons);
                return best_file_name;
        }
 
@@ -902,11 +943,12 @@ char *icon_path(Launcher *launcher, const char *icon_name, int size)
                                char *file_name = malloc(strlen(base_name) + strlen(icon_name) +
                                        strlen(extension) + 100);
                                // filename = directory/iconname.extension
-                               sprintf(file_name, "%s/%s.%s", base_name, icon_name, extension);
+                               sprintf(file_name, "%s/%s%s", base_name, icon_name, extension);
                                //printf("checking %s\n", file_name);
                                if (g_file_test(file_name, G_FILE_TEST_EXISTS)) {
                                        g_slist_free(basenames);
                                        g_slist_free(extensions);
+                                       g_free(home_icons);
                                        return file_name;
                                } else {
                                        free(file_name);
@@ -918,6 +960,9 @@ char *icon_path(Launcher *launcher, const char *icon_name, int size)
 
        fprintf(stderr, "Could not find icon %s\n", icon_name);
 
+       g_slist_free(basenames);
+       g_slist_free(extensions);
+       g_free(home_icons);
        return NULL;
 }
 
This page took 0.035903 seconds and 4 git commands to generate.