void action_execute_startup(void)
{
- actions_register("Execute", setup_func, free_func, run_func, NULL, NULL);
+ actions_register("Execute", setup_func, free_func, run_func);
}
static gpointer setup_func(xmlNodePtr node)
static gboolean run_func(ObActionsData *data, gpointer options);
-static void prompt_cb(ObPrompt *p, gint result, gpointer options)
+static gboolean prompt_cb(ObPrompt *p, gint result, gpointer options)
{
if (result)
run_func(NULL, options);
+ return TRUE; /* call the cleanup func */
+}
+static void prompt_cleanup(ObPrompt *p, gpointer options)
+{
prompt_unref(p);
free_func(options);
}
};
ocp = dup_options(options);
- p = prompt_new(o->prompt, answers, 2, 0, 0, prompt_cb, ocp);
+ p = prompt_new(o->prompt, _("Execute"), answers, 2, 0, 0,
+ prompt_cb, prompt_cleanup, ocp);
prompt_show(p, NULL, FALSE);
return FALSE;
if (data->client) {
gchar *c, *before, *expand;
- /* replace occurances of $pid and $window */
+ /* replace occurrences of $pid and $wid */
expand = NULL;
before = cmd;
before = c + 4; /* 4 = strlen("$pid") */
}
-
- if ((c[1] == 'w' || c[1] == 'W') &&
- (c[2] == 'i' || c[2] == 'I') &&
- (c[3] == 'n' || c[3] == 'N') &&
- (c[4] == 'd' || c[4] == 'D') &&
- (c[5] == 'o' || c[5] == 'O') &&
- (c[6] == 'w' || c[6] == 'W') &&
- !g_ascii_isalnum(c[7]))
+ else if ((c[1] == 'w' || c[1] == 'W') &&
+ (c[2] == 'i' || c[2] == 'I') &&
+ (c[3] == 'd' || c[3] == 'D') &&
+ !g_ascii_isalnum(c[4]))
{
- /* found $window */
+ /* found $wid */
gchar *tmp;
*c = '\0';
data->client->window);
g_free(tmp);
- before = c + 7; /* 4 = strlen("$window") */
+ before = c + 4; /* 4 = strlen("$wid") */
}
+ else
+ before = c + 1; /* no infinite loops plz */
}
if (expand) {
e = NULL;
if (!g_shell_parse_argv(cmd, NULL, &argv, &e)) {
- g_message(_("Failed to execute \"%s\": %s"), o->cmd, e->message);
+ g_message(e->message, o->cmd);
g_error_free(e);
}
else {
G_SPAWN_DO_NOT_REAP_CHILD,
NULL, NULL, NULL, &e);
if (!ok) {
- g_message(_("Failed to execute \"%s\": %s"),
- o->cmd, e->message);
+ g_message(e->message, o->cmd);
g_error_free(e);
}