From e72a1232fa73450c4bfefef5657971ecbacb8f70 Mon Sep 17 00:00:00 2001 From: Mikael Magnusson Date: Fri, 9 Jun 2006 14:52:06 +0000 Subject: [PATCH] if this works i will be a bit amazed, add class matching, and allow to match either name or class or both --- openbox/client.c | 7 ++++++- openbox/config.c | 23 +++++++++++++++++++---- 2 files changed, 25 insertions(+), 5 deletions(-) diff --git a/openbox/client.c b/openbox/client.c index 4d0ad534..ccd30787 100644 --- a/openbox/client.c +++ b/openbox/client.c @@ -213,7 +213,12 @@ static ObAppSettings *get_settings(ObClient *client) while (a) { ObAppSettings *app = (ObAppSettings *) a->data; - if (!strcmp(app->name, client->name)) { + if ( + (app->name && && !app->class && !strcmp(app->name, client->name)) + || (app->class && !app->name && !strcmp(app->class, client->class)) + || (app->class && app->name && !strcmp(app->class, client->class) + && !strcmp(app->name, client->name)) + ) { ob_debug("Window matching: %s\n", app->name); /* Match if no role was specified in the per app setting, or if the string * matches the beginning of the role, since apps like to set the role to diff --git a/openbox/config.c b/openbox/config.c index a456a688..574d13ee 100644 --- a/openbox/config.c +++ b/openbox/config.c @@ -115,14 +115,28 @@ static void parse_per_app_settings(ObParseInst *i, xmlDocPtr doc, xmlNodePtr node, gpointer d) { xmlNodePtr app = parse_find_node("application", node->children); - gchar *name; + gchar *name, *class; + gboolean name_set, class_set; + gboolean x_pos_given; while (app) { - gboolean x_pos_given = FALSE; - if (parse_attr_string("name", app, &name)) { + name_set = class_set = x_pos_given = FALSE; + + class_set = parse_attr_string("class", app, &class); + name_set = parse_attr_string("name", app, &name); + if (class_set || name_set) { xmlNodePtr n, c; ObAppSettings *settings = g_new0(ObAppSettings, 1); - settings->name = name; + + if (name_set) + settings->name = name; + else + settings->name = NULL; + if (class_set) + settings->class = class; + else + settings->class = NULL; + if (!parse_attr_string("role", app, &settings->role)) settings->role = NULL; @@ -810,6 +824,7 @@ void config_shutdown() ObAppSettings *itd = (ObAppSettings *)it->data; g_free(itd->name); g_free(itd->role); + g_free(itd->class); g_free(it->data); } g_slist_free(config_per_app_settings); -- 2.45.2