changeset 140:e57b0207e180

editors.c was almost completely rewritten: - centralized template parsing - better control of executed editors - possibility to get editor exit status via callback

--- a/src/editors.c	Wed Sep 12 21:17:31 2007 +0000
+++ b/src/editors.c	Sun Sep 30 21:10:54 2007 +0000
@@ -18,31 +18,40 @@
 #include "ui_spinner.h"
 #include "ui_utildlg.h"
 
+#include "filelist.h"
+
 #include <errno.h>
 
 
 #define EDITOR_WINDOW_WIDTH 500
 #define EDITOR_WINDOW_HEIGHT 300
 
-#define COMMAND_SHELL "sh"
+#define COMMAND_SHELL "/bin/sh"
 #define COMMAND_OPT  "-c"
 
 
 typedef struct _EditorVerboseData EditorVerboseData;
 struct _EditorVerboseData {
-	int fd;
-
 	GenericDialog *gd;
 	GtkWidget *button_close;
 	GtkWidget *button_stop;
 	GtkWidget *text;
 	GtkWidget *progress;
 	GtkWidget *spinner;
+};
+
+typedef struct _EditorData EditorData;
+struct _EditorData {
+	gint flags;
+	GPid pid;
+	gchar *command_template;
+	GList *list;
 	gint count;
 	gint total;
-
-	gchar *command_template;
-	GList *list;
+	gboolean stopping;
+	EditorVerboseData *vd;
+	EditorCallback callback;
+	gpointer data;
 };
 
 
@@ -60,9 +69,9 @@
 	/* special slots */
 #if 1
 	/* for testing */
-	"External Copy command", "%vset -x;cp %p %t",
-	"External Move command", "%vset -x;mv %p %t",
-	"External Rename command", "%vset -x;mv %p %t",
+	"External Copy command", "%vset -x;cp %p %d",
+	"External Move command", "%vset -x;mv %p %d",
+	"External Rename command", "%vset -x;mv %p %d",
 	"External Delete command", "%vset -x;rm %f",
 	"External New Folder command", NULL
 #else
@@ -74,9 +83,10 @@
 #endif
 };
 
-static void editor_verbose_window_progress(EditorVerboseData *vd, const gchar *text);
-static gint editor_command_next(EditorVerboseData *vd);
-
+static void editor_verbose_window_progress(EditorData *ed, const gchar *text);
+static gint editor_command_next_start(EditorData *ed);
+static gint editor_command_next_finish(EditorData *ed, gint status);
+static gint editor_command_done(EditorData *ed);
 
 /*
  *-----------------------------------------------------------------------------
@@ -97,24 +107,35 @@
 		}
 }
 
+static void editor_verbose_data_free(EditorData *ed)
+{
+	if (!ed->vd) return;
+	g_free(ed->vd);
+	ed->vd = NULL;
+}
+
+static void editor_data_free(EditorData *ed)
+{
+	editor_verbose_data_free(ed);
+	g_free(ed->command_template);
+	g_free(ed);
+}
+
 static void editor_verbose_window_close(GenericDialog *gd, gpointer data)
 {
-	EditorVerboseData *vd = data;
+	EditorData *ed = data;
 
 	generic_dialog_close(gd);
-	g_free(vd->command_template);
-	g_free(vd);
+	editor_verbose_data_free(ed);
+	if (ed->pid == -1) editor_data_free(ed); /* the process has already terminated */
 }
 
 static void editor_verbose_window_stop(GenericDialog *gd, gpointer data)
 {
-	EditorVerboseData *vd = data;
-
-	filelist_free(vd->list);
-	vd->list = NULL;
-
-	vd->count = 0;
-	editor_verbose_window_progress(vd, _("stopping..."));
+	EditorData *ed = data;
+	ed->stopping = TRUE;
+	ed->count = 0;
+	editor_verbose_window_progress(ed, _("stopping..."));
 }
 
 static void editor_verbose_window_enable_close(EditorVerboseData *vd)
@@ -126,7 +147,7 @@
 	gtk_widget_set_sensitive(vd->button_close, TRUE);
 }
 
-static EditorVerboseData *editor_verbose_window(const gchar *template, const gchar *text)
+static EditorVerboseData *editor_verbose_window(EditorData *ed, const gchar *text)
 {
 	EditorVerboseData *vd;
 	GtkWidget *scrolled;
@@ -135,15 +156,9 @@
 
 	vd = g_new0(EditorVerboseData, 1);
 
-	vd->list = NULL;
-	vd->command_template = g_strdup(template);
-	vd->total = 0;
-	vd->count = 0;
-	vd->fd = -1;
-
 	vd->gd = file_util_gen_dlg(_("Edit command results"), "GQview", "editor_results",
 				   NULL, FALSE,
-				   NULL, vd);
+				   NULL, ed);
 	buf = g_strdup_printf(_("Output of %s"), text);
 	generic_dialog_add_message(vd->gd, NULL, buf, NULL);
 	g_free(buf);
@@ -182,6 +197,7 @@
 	
 	gtk_widget_show(vd->gd->dialog);
 
+	ed->vd = vd;
 	return vd;
 }
 
@@ -195,152 +211,65 @@
 	gtk_text_buffer_insert(buffer, &iter, text, len);
 }
 
-static void editor_verbose_window_progress(EditorVerboseData *vd, const gchar *text)
+static void editor_verbose_window_progress(EditorData *ed, const gchar *text)
 {
-	if (vd->total)
+	if (!ed->vd) return;
+
+	if (ed->total)
 		{
-		gtk_progress_bar_set_fraction(GTK_PROGRESS_BAR(vd->progress), (double)vd->count / vd->total);
+		gtk_progress_bar_set_fraction(GTK_PROGRESS_BAR(ed->vd->progress), (double)ed->count / ed->total);
 		}
 
-	gtk_progress_bar_set_text(GTK_PROGRESS_BAR(vd->progress), (text) ? text : "");
+	gtk_progress_bar_set_text(GTK_PROGRESS_BAR(ed->vd->progress), (text) ? text : "");
 }
 
 static gboolean editor_verbose_io_cb(GIOChannel *source, GIOCondition condition, gpointer data)
 {
-	EditorVerboseData *vd = data;
+	EditorData *ed = data;
 	gchar buf[512];
 	gsize count;
 
-	switch (condition)
+	if (condition & G_IO_IN)
 		{
-		case G_IO_IN:
-			while (g_io_channel_read_chars(source, buf, sizeof(buf), &count, NULL) == G_IO_STATUS_NORMAL)
+		while (g_io_channel_read_chars(source, buf, sizeof(buf), &count, NULL) == G_IO_STATUS_NORMAL)
+			{
+			if (!g_utf8_validate(buf, count, NULL))
 				{
-				if (!g_utf8_validate(buf, count, NULL))
+				gchar *utf8;
+				utf8 = g_locale_to_utf8(buf, count, NULL, NULL, NULL);
+				if (utf8)
 					{
-					gchar *utf8;
-					utf8 = g_locale_to_utf8(buf, count, NULL, NULL, NULL);
-					if (utf8)
-						{
-						editor_verbose_window_fill(vd, utf8, -1);
-						g_free(utf8);
-						}
-					else
-						{
-						editor_verbose_window_fill(vd, "GQview: Error converting text to valid utf8\n", -1);
-						}
+					editor_verbose_window_fill(ed->vd, utf8, -1);
+					g_free(utf8);
 					}
 				else
 					{
-					editor_verbose_window_fill(vd, buf, count);
+					editor_verbose_window_fill(ed->vd, "GQview: Error converting text to valid utf8\n", -1);
 					}
 				}
-			break;
-		case G_IO_ERR:
-			printf("Error reading from command\n");
-		case G_IO_HUP:
-			if (debug) printf("Editor command HUP\n");
-		default:
-			while (g_source_remove_by_user_data(vd));
-			close(vd->fd);
-			vd->fd = -1;
-			editor_command_next(vd);
-			return FALSE;
-			break;
-		}
-
-	return TRUE;
-}
-
-static int command_pipe(char *command)
-{
-	char *args[4];
-	int fpipe[2];
-	pid_t fpid;
-
-	args[0] = COMMAND_SHELL;
-	args[1] = COMMAND_OPT;
-	args[2] = command;
-	args[3] = NULL;
-
-	if (pipe(fpipe) < 0)
-		{
-		printf("pipe setup failed: %s\n", strerror(errno));
-		return -1;
+			else
+				{
+				editor_verbose_window_fill(ed->vd, buf, count);
+				}
+			}
 		}
 
-	fpid = fork();
-	if (fpid < 0)
-		{
-		/* fork failed */
-		printf("fork failed: %s\n", strerror(errno));
-		}
-	else if (fpid == 0)
-		{
-		/* child */
-		gchar *msg;
-
-		dup2(fpipe[1], 1);
-		dup2(fpipe[1], 2);
-		close(fpipe[0]);
-
-		execvp(args[0], args);
-
-		msg = g_strdup_printf("Unable to exec command:\n%s\n\n%s\n", command, strerror(errno));
-		write(1, msg, strlen(msg));
-
-		_exit(1);
-		}
-	else
+	if (condition & (G_IO_ERR | G_IO_HUP))
 		{
-		/* parent */
-		fcntl(fpipe[0], F_SETFL, O_NONBLOCK);
-		close(fpipe[1]);
-
-		return fpipe[0];
-		}
-
-	return -1;
-}
-
-static gint editor_verbose_start(EditorVerboseData *vd, gchar *command)
-	{
-	GIOChannel *channel;
-	int fd;
-
-	fd = command_pipe(command);
-	if (fd < 0)
-		{
-		gchar *buf;
-
-		buf = g_strdup_printf(_("Failed to run command:\n%s\n"), command);
-		editor_verbose_window_fill(vd, buf, strlen(buf));
-		g_free(buf);
-
+		g_io_channel_shutdown(source, TRUE, NULL);
 		return FALSE;
 		}
 
-	vd->fd = fd;
-	channel = g_io_channel_unix_new(fd);
-
-	g_io_add_watch_full(channel, G_PRIORITY_DEFAULT, G_IO_IN,
-			    editor_verbose_io_cb, vd, NULL);
-	g_io_add_watch_full(channel, G_PRIORITY_DEFAULT, G_IO_ERR,
-			    editor_verbose_io_cb, vd, NULL);
-	g_io_add_watch_full(channel, G_PRIORITY_DEFAULT, G_IO_HUP,
-			    editor_verbose_io_cb, vd, NULL);
-	g_io_channel_unref(channel);
-
 	return TRUE;
 }
 
 typedef enum {
 	PATH_FILE,
-	PATH_TARGET
+	PATH_DEST
 } PathType;
 
 
-static gchar *editor_command_path_parse(const FileData *fd, PathType type)
+static gchar *editor_command_path_parse(const FileData *fd, PathType type, const gchar *extensions)
 {
 	GString *string;
 	gchar *pathl;
@@ -352,7 +281,7 @@
 		{
 		p = fd->path;
 		}
-	else if (type == PATH_TARGET)
+	else if (type == PATH_DEST)
 		{
 		if (fd->change && fd->change->dest)
 			p = fd->change->dest;
@@ -378,149 +307,6 @@
 	return pathl;
 }
 
-static gint editor_command_one(const gchar *template, const FileData *fd, EditorVerboseData *vd)
-{
-	GString *result = NULL;
-	gchar *pathl, *targetl;
-	gchar *found;
-	const gchar *ptr;
-	gchar path_buffer[512];
-	gchar *current_path;
-	gint path_change = FALSE;
-	gint ret;
-
-	current_path = getcwd(path_buffer, sizeof(path_buffer));
-
-	result = g_string_new("");
-	pathl = editor_command_path_parse(fd, PATH_FILE);
-	targetl = editor_command_path_parse(fd, PATH_TARGET);
-
-	ptr = template;
-	while ( (found = strstr(ptr, "%")) )
-		{
-		result = g_string_append_len(result, ptr, found - ptr);
-		ptr = found + 2;
-		switch (found[1])
-			{
-			case 'p':
-				result = g_string_append_c(result, '"');
-				result = g_string_append(result, pathl);
-				result = g_string_append_c(result, '"');
-				break;
-			case 't':
-				result = g_string_append_c(result, '"');
-				result = g_string_append(result, targetl);
-				result = g_string_append_c(result, '"');
-				break;
-			case '%':
-				result = g_string_append_c(result, '%');
-				break;
-			default:
-				break;
-			}
-		}
-	result = g_string_append(result, ptr);
-
-	if (debug) printf("system command: %s\n", result->str);
-
-	if (current_path)
-		{
-		gchar *base;
-		base = remove_level_from_path(fd->path);
-		if (chdir(base) == 0) path_change = TRUE;
-		g_free(base);
-		}
-
-	if (vd)
-		{
-		result = g_string_append(result, " 2>&1");
-		ret = editor_verbose_start(vd, result->str);
-		}
-	else
-		{
-		ret = !system(result->str);
-		}
-
-	if (path_change) chdir(current_path);
-
-	g_string_free(result, TRUE);
-	g_free(pathl);
-	g_free(targetl);
-
-	return ret;
-}
-
-static gint editor_command_next(EditorVerboseData *vd)
-{
-	const gchar *text;
-
-	editor_verbose_window_fill(vd, "\n", 1);
-
-	while (vd->list)
-		{
-		FileData *fd;
-		gint success;
-
-		fd = vd->list->data;
-		vd->list = g_list_remove(vd->list, fd);
-
-		editor_verbose_window_progress(vd, fd->path);
-
-		vd->count++;
-		success = editor_command_one(vd->command_template, fd, vd);
-		if (success)
-			{
-			gtk_widget_set_sensitive(vd->button_stop, (vd->list != NULL) );
-			editor_verbose_window_fill(vd, fd->path, strlen(fd->path));
-			editor_verbose_window_fill(vd, "\n", 1);
-			}
-
-		file_data_unref(fd);
-		if (success) return TRUE;
-		}
-
-	if (vd->count == vd->total)
-		{
-		text = _("done");
-		}
-	else
-		{
-		text = _("stopped by user");
-		}
-	vd->count = 0;
-	editor_verbose_window_progress(vd, text);
-	editor_verbose_window_enable_close(vd);
-	return FALSE;
-}
-
-static gint editor_command_start(const gchar *template, const gchar *text, GList *list)
-{
-	EditorVerboseData *vd;
-
-	vd = editor_verbose_window(template, text);
-	vd->list = filelist_copy(list);
-	vd->total = g_list_length(list);
-
-	return editor_command_next(vd);
-}
-
-static gint editor_line_break(const gchar *template, gchar **front, const gchar **end)
-{
-	gchar *found;
-
-	*front = g_strdup(template);
-	found = strstr(*front, "%f");
-
-	if (found)
-		{
-		*found = '\0';
-		*end = found + 2;
-		return TRUE;
-		}
-
-	*end = "";
-	return FALSE;
-}
 
 /*
  * The supported macros for editor commands:
@@ -549,142 +335,453 @@
  *
  * [1] Note: %v,%V may also be preceded by "%w".
  */
-static gint editor_command_run(const gchar *template, const gchar *text, GList *list)
+
+
+gint editor_command_parse(const gchar *template, GList *list, gchar **output)
 {
-	gint verbose = FALSE;
-	gint for_each = FALSE;
-	gint ret = TRUE;
+	gint flags = 0;
+	const gchar *p = template;
+	GString *result = NULL;
+	gchar *extensions = NULL;
+	GList *work;
+
+	if (output)
+		result = g_string_new("");
+
+	if (!template || template[0] == '\0') 
+		{
+		flags |= EDITOR_ERROR_EMPTY;
+		goto err;
+		}
 
-	if (!template || template[0] == '\0') return;
-
-	for_each = (strstr(template, "%p") != NULL);
+	
+	/* global flags */
+	while (*p == '%')
+		{
+		switch (*++p)
+			{
+			case 'w':
+				flags |= EDITOR_KEEP_FS;
+				p++;
+				break;
+			case 'v':
+				flags |= EDITOR_VERBOSE;
+				p++;
+				break;
+			case 'V':
+				flags |= EDITOR_VERBOSE_MULTI;
+				p++;
+				break;
+			}
+		}
+	
+	/* command */
+	
+	while (*p)
+		{
+		if (*p != '%')
+			{
+			if (output) result = g_string_append_c(result, *p);
+			}
+		else /* *p == '%' */
+			{
+			extensions = NULL;
+			gchar *pathl = NULL;
 
-	/* no window state change flag, skip */
-	if (strncmp(template, "%w", 2) == 0) template += 2;
+			p++;
+			
+			/* for example "%f" or "%{crw,raw,cr2}f" */
+			if (*p == '{')
+				{
+				p++;
+				gchar *end = strchr(p, '}');
+				if (!end)
+					{
+					flags |= EDITOR_ERROR_SYNTAX;
+					goto err;
+					}
+				
+				extensions = g_strndup(p, end - p);
+				p = end + 1;
+				}
+			
+			switch (*p) 
+				{
+				case 'd':
+					flags |= EDITOR_DEST;
+				case 'p':
+					flags |= EDITOR_FOR_EACH;
+					if (flags & EDITOR_SINGLE_COMMAND)
+						{
+						flags |= EDITOR_ERROR_INCOMPATIBLE;
+						goto err;
+						}
+					if (output)
+						{
+						/* use the first file from the list */
+						if (!list || !list->data) 
+							{
+							flags |= EDITOR_ERROR_NO_FILE;
+							goto err;
+							}
+						pathl = editor_command_path_parse((FileData *)list->data, (*p == 'd') ? PATH_DEST : PATH_FILE, extensions);
+						if (!pathl) 
+							{
+							flags |= EDITOR_ERROR_NO_FILE;
+							goto err;
+							}
+						result = g_string_append_c(result, '"');
+						result = g_string_append(result, pathl);
+						g_free(pathl);
+						result = g_string_append_c(result, '"');
+						}
+					break;	
 
-	if (strncmp(template, "%v", 2) == 0)
-		{
-		template += 2;
-		verbose = TRUE;
-		}
-	else if (strncmp(template, "%V", 2) == 0)
-		{
-		template += 2;
-		if (!for_each || list->next) verbose = TRUE;
+				case 'f':
+					flags |= EDITOR_SINGLE_COMMAND;
+					if (flags & (EDITOR_FOR_EACH | EDITOR_DEST))
+						{
+						flags |= EDITOR_ERROR_INCOMPATIBLE;
+						goto err;
+						}
+
+					if (output)
+						{
+						/* use whole list */
+						GList *work = list;
+						gboolean ok = FALSE;
+						while (work)
+							{
+							FileData *fd = work->data;
+							pathl = editor_command_path_parse(fd, PATH_FILE, extensions);
+
+							if (pathl)
+								{
+								ok = TRUE;
+								if (work != list) g_string_append_c(result, ' ');
+								result = g_string_append_c(result, '"');
+								result = g_string_append(result, pathl);
+								g_free(pathl);
+								result = g_string_append_c(result, '"');
+								}
+							work = work->next;
+							}
+						if (!ok) 
+							{
+							flags |= EDITOR_ERROR_NO_FILE;
+							goto err;
+							}
+						}
+					break;	
+				default:
+					flags |= EDITOR_ERROR_SYNTAX;
+					goto err;
+				}
+			if (extensions) g_free(extensions);
+			extensions = NULL;
+			}
+		p++;
 		}
 
-	if (for_each)
+	if (output) *output = g_string_free(result, FALSE);
+	return flags;
+
+			
+err:
+	if (output) 
 		{
-		if (verbose)
-			{
-			editor_command_start(template, text, list);
-			}
-		else
+		g_string_free(result, TRUE);
+		*output = NULL;
+		}
+	if (extensions) g_free(extensions);
+	return flags;
+}
+
+static void editor_child_exit_cb (GPid pid, gint status, gpointer data)
+{
+	EditorData *ed = data;
+	g_spawn_close_pid(pid);
+	ed->pid = -1;
+	
+	editor_command_next_finish(ed, status);
+}
+
+
+static gint editor_command_one(const gchar *template, GList *list, EditorData *ed)
+{
+	gchar *command;
+	gchar *working_directory;
+	FileData *fd = list->data;
+	gchar *args[4];
+	GPid pid;
+        gint standard_output;
+        gint standard_error;
+	gboolean ok;
+
+
+	ed->pid = -1;
+
+	working_directory = remove_level_from_path(fd->path);
+	
+	ed->flags = editor_command_parse(template, list, &command);
+
+	ok = !(ed->flags & EDITOR_ERROR_MASK);
+
+
+	args[0] = COMMAND_SHELL;
+	args[1] = COMMAND_OPT;
+	args[2] = command;
+	args[3] = NULL;
+	
+	if (ok)
+		{
+		ok = g_spawn_async_with_pipes(working_directory, args, NULL, 
+				      G_SPAWN_DO_NOT_REAP_CHILD, /* GSpawnFlags */
+                                      NULL, NULL,
+                                      &pid, 
+				      NULL, 
+				      ed->vd ? &standard_output : NULL, 
+				      ed->vd ? &standard_error : NULL, 
+				      NULL);
+		
+		if (!ok) ed->flags |= EDITOR_ERROR_CANT_EXEC;
+		}
+
+	if (ok) 
+		{
+		g_child_watch_add(pid, editor_child_exit_cb, ed);
+		ed->pid = pid;
+		}
+	
+	
+	if (ed->vd)
+		{
+
+		if (!ok)
 			{
-			GList *work;
+			gchar *buf;
+
+			buf = g_strdup_printf(_("Failed to run command:\n%s\n"), command);
+			editor_verbose_window_fill(ed->vd, buf, strlen(buf));
+			g_free(buf);
+
+			}
+		else 
+			{
+		
+			GIOChannel *channel_output;
+			GIOChannel *channel_error;
+			channel_output = g_io_channel_unix_new(standard_output);
+			g_io_channel_set_flags(channel_output, G_IO_FLAG_NONBLOCK, NULL);
+
+			g_io_add_watch_full(channel_output, G_PRIORITY_HIGH, G_IO_IN | G_IO_ERR | G_IO_HUP,
+					    editor_verbose_io_cb, ed, NULL);
+			g_io_channel_unref(channel_output);
+
+			channel_error = g_io_channel_unix_new(standard_error);
+			g_io_channel_set_flags(channel_error, G_IO_FLAG_NONBLOCK, NULL);
+
+			g_io_add_watch_full(channel_error, G_PRIORITY_HIGH, G_IO_IN | G_IO_ERR | G_IO_HUP,
+					    editor_verbose_io_cb, ed, NULL);
+			g_io_channel_unref(channel_error);
+			}
+		}
+	
 
-			work = list;
-			while (work)
+	
+	g_free(command);
+	g_free(working_directory);
+
+	return ed->flags & EDITOR_ERROR_MASK;
+}
+
+static gint editor_command_next_start(EditorData *ed)
+{
+
+	if (ed->vd) editor_verbose_window_fill(ed->vd, "\n", 1);
+
+	if (ed->list && ed->count < ed->total)
+		{
+		FileData *fd;
+		gint error;
+
+		fd = ed->list->data;
+
+		if (ed->vd)
+			{
+			editor_verbose_window_progress(ed, (ed->flags & EDITOR_FOR_EACH) ? fd->path : _("running..."));
+			}
+		ed->count++;
+
+		error = editor_command_one(ed->command_template, ed->list, ed);
+		if (!error && ed->vd)
+			{
+			gtk_widget_set_sensitive(ed->vd->button_stop, (ed->list != NULL) );
+			if (ed->flags & EDITOR_FOR_EACH)
 				{
-				FileData *fd = work->data;
-				ret = editor_command_one(template, fd, NULL);
-				work = work->next;
+				editor_verbose_window_fill(ed->vd, fd->path, strlen(fd->path));
+				editor_verbose_window_fill(ed->vd, "\n", 1);
 				}
 			}
+
+		if (!error) 
+			return 0;
+		else
+			/* command was not started, call the finish immediately */
+			return editor_command_next_finish(ed, 0);
+		}
+	
+	/* everything is done */
+	editor_command_done(ed);
+
+}
+
+static gint editor_command_next_finish(EditorData *ed, gint status)
+{
+	gint cont = ed->stopping ? EDITOR_CB_SKIP : EDITOR_CB_CONTINUE;
+
+	if (status)
+		ed->flags |= EDITOR_ERROR_STATUS;
+
+	if (ed->flags & EDITOR_FOR_EACH)
+		{
+		/* handle the first element from the list */
+		GList *fd_element = ed->list;
+		ed->list = g_list_remove_link(ed->list, fd_element);
+		if (ed->callback)
+			cont = ed->callback(ed->list ? ed : NULL, ed->flags, fd_element, ed->data);
+		filelist_free(fd_element);
 		}
 	else
 		{
-		gchar *front;
-		const gchar *end;
-		GList *work;
-		GString *result = NULL;
-		gint parser_match;
-
-		parser_match = editor_line_break(template, &front, &end);
-		result = g_string_new((parser_match) ? "" : " ");
-
-		work = list;
-		while (work)
-			{
-			FileData *fd = work->data;
-			gchar *pathl;
+		/* handle whole list */
+		if (ed->callback)
+			cont = ed->callback(NULL, ed->flags, ed->list, ed->data);
+		filelist_free(ed->list);
+		ed->list = NULL;
+		}
 
-			if (work != list) g_string_append_c(result, ' ');
-			result = g_string_append_c(result, '"');
-			pathl = editor_command_path_parse(fd, PATH_FILE);
-			result = g_string_append(result, pathl);
-			g_free(pathl);
-			result = g_string_append_c(result, '"');
-			work = work->next;
-			}
+	if (cont == EDITOR_CB_SUSPEND)
+		return ed->flags & EDITOR_ERROR_MASK;
+	else if (cont == EDITOR_CB_SKIP)
+		return editor_command_done(ed);
+	else
+		return editor_command_next_start(ed);
+
+}
 
-		result = g_string_prepend(result, front);
-		result = g_string_append(result, end);
-		if (verbose) result = g_string_append(result, " 2>&1 ");
-		result = g_string_append(result, "&");
-
-		if (debug) printf("system command: %s\n", result->str);
+static gint editor_command_done(EditorData *ed)
+{
+	gint flags;
+	const gchar *text;
 
-		if (verbose)
+	if (ed->vd)
+		{
+		if (ed->count == ed->total)
 			{
-			EditorVerboseData *vd;
-
-			vd = editor_verbose_window(template, text);
-			editor_verbose_window_progress(vd, _("running..."));
-			ret = editor_verbose_start(vd, result->str);
+			text = _("done");
 			}
 		else
 			{
-			ret = !system(result->str);
+			text = _("stopped by user");
 			}
+		editor_verbose_window_progress(ed, text);
+		editor_verbose_window_enable_close(ed->vd);
+		}
+
+	/* free the not-handled items */
+	if (ed->list)
+		{
+		ed->flags |= EDITOR_ERROR_SKIPPED;
+		if (ed->callback) ed->callback(NULL, ed->flags, ed->list, ed->data);
+		filelist_free(ed->list);
+		ed->list = NULL;
+		}
 
-		g_free(front);
-		g_string_free(result, TRUE);
-		}
-	return ret;
+	ed->count = 0;
+
+	flags = ed->flags & EDITOR_ERROR_MASK;
+
+	if (!ed->vd) editor_data_free(ed);
+
+	return flags;
+}
+
+void editor_resume(gpointer ed)
+{
+	editor_command_next_start(ed);
+}
+void editor_skip(gpointer ed)
+{
+	editor_command_done(ed);	
 }
 
-gint start_editor_from_filelist(gint n, GList *list)
+static gint editor_command_start(const gchar *template, const gchar *text, GList *list, EditorCallback cb, gpointer data)
+{
+	EditorData *ed;
+	gint flags = editor_command_parse(template, NULL, NULL);
+	
+	if (flags & EDITOR_ERROR_MASK) return flags & EDITOR_ERROR_MASK;
+
+	ed = g_new0(EditorData, 1);
+	ed->list = filelist_copy(list);
+	ed->flags = flags;
+	ed->command_template = g_strdup(template);
+	ed->total = (flags & EDITOR_SINGLE_COMMAND) ? 1 : g_list_length(list);
+	ed->count = 0;
+	ed->stopping = FALSE;
+	ed->callback = cb;
+	ed->data =  data;
+	
+	if ((flags & EDITOR_VERBOSE_MULTI) && list && list->next)
+		flags |= EDITOR_VERBOSE;
+	
+	
+	if (flags & EDITOR_VERBOSE)
+		editor_verbose_window(ed, text);
+		
+	editor_command_next_start(ed); 
+	/* errors from editor_command_next_start will be handled via callback */
+	return flags & EDITOR_ERROR_MASK;
+}
+
+gint start_editor_from_filelist_full(gint n, GList *list, EditorCallback cb, gpointer data)
 {
 	gchar *command;
-	gint ret;
+	gint error;
 
 	if (n < 0 || n >= GQVIEW_EDITOR_SLOTS || !list ||
 	    !editor_command[n] ||
 	    strlen(editor_command[n]) == 0) return FALSE;
 
 	command = g_locale_from_utf8(editor_command[n], -1, NULL, NULL, NULL);
-	ret = editor_command_run(command, editor_name[n], list);
+	error = editor_command_start(command, editor_name[n], list, cb, data);
 	g_free(command);
-	return ret;
+	return error;
 }
 
-gint start_editor_from_file(gint n, FileData *fd)
+gint start_editor_from_filelist(gint n, GList *list)
+{
+	return start_editor_from_filelist_full(n, list,  NULL, NULL);
+}
+
+
+gint start_editor_from_file_full(gint n, FileData *fd, EditorCallback cb, gpointer data)
 {
 	GList *list;
-	gint ret;
+	gint error;
 
 	if (!fd) return FALSE;
 
 	list = g_list_append(NULL, fd);
-	ret = start_editor_from_filelist(n, list);
+	error = start_editor_from_filelist_full(n, list, cb, data);
 	g_list_free(list);
-	return ret;
+	return error;
 }
 
-gint start_editor_from_pair(gint n, const gchar *source, const gchar *target)
+gint start_editor_from_file(gint n, FileData *fd)
 {
-	GList *list;
-	gint ret;
-
-	if (!source) return FALSE;
-	if (!target) return FALSE;
-
-	list = g_list_append(NULL, (gchar *)source);
-	list = g_list_append(list, (gchar *)target);
-	ret = start_editor_from_filelist(n, list);
-	g_list_free(list);
-	return ret;
+	return start_editor_from_file_full(n, fd, NULL, NULL);
 }
 
 gint editor_window_flag_set(gint n)
@@ -693,7 +790,18 @@
 	    !editor_command[n] ||
 	    strlen(editor_command[n]) == 0) return TRUE;
 
-	return (strncmp(editor_command[n], "%w", 2) == 0);
+	return (editor_command_parse(editor_command[n], NULL, NULL) & EDITOR_KEEP_FS);
 }
 
 
+const gchar *editor_get_error_str(gint flags)
+{
+	if (flags & EDITOR_ERROR_EMPTY) return _("Editor template is empty.");
+	if (flags & EDITOR_ERROR_SYNTAX) return _("Editor template has incorrect syntax.");
+	if (flags & EDITOR_ERROR_INCOMPATIBLE) return _("Editor template uses incompatible macros.");
+	if (flags & EDITOR_ERROR_NO_FILE) return _("Can't find matching file type.");
+	if (flags & EDITOR_ERROR_CANT_EXEC) return _("Can't execute external editor.");
+	if (flags & EDITOR_ERROR_STATUS) return _("External editor returned error status.");
+	if (flags & EDITOR_ERROR_SKIPPED) return _("File was skipped.");
+	return _("Unknown error.");
+}

--- a/src/editors.h	Wed Sep 12 21:17:31 2007 +0000
+++ b/src/editors.h	Sun Sep 30 21:10:54 2007 +0000
@@ -13,13 +13,62 @@
 #ifndef EDITORS_H
 #define EDITORS_H
 
+enum {
+	EDITOR_KEEP_FS            = 0x00000001,
+	EDITOR_VERBOSE            = 0x00000002,
+	EDITOR_VERBOSE_MULTI      = 0x00000004,
+	
+	EDITOR_DEST               = 0x00000100,
+	EDITOR_FOR_EACH           = 0x00000200,
+	EDITOR_SINGLE_COMMAND     = 0x00000400,
+	
+	EDITOR_ERROR_EMPTY        = 0x00020000,
+	EDITOR_ERROR_SYNTAX       = 0x00040000,
+	EDITOR_ERROR_INCOMPATIBLE = 0x00080000,
+	EDITOR_ERROR_NO_FILE      = 0x00100000,
+	EDITOR_ERROR_CANT_EXEC    = 0x00200000,
+	EDITOR_ERROR_STATUS       = 0x00400000,
+	EDITOR_ERROR_SKIPPED      = 0x00800000,
+
+	EDITOR_ERROR_MASK         = 0xffff0000
+	
+};
+
+/* return values from callback function */
+enum {
+	EDITOR_CB_CONTINUE = 0, /* continue multiple editor execution on remaining files*/
+	EDITOR_CB_SKIP,         /* skip the remaining files */
+	EDITOR_CB_SUSPEND       /* suspend execution, one of editor_resume or editor_skip
+	                           must be called later */
+};
+
+
+/*
+Callback is called even on skipped files, with the EDITOR_ERROR_SKIPPED flag set. 
+It is a good place to call file_data_change_info_free().
+
+ed - pointer that can be used for editor_resume/editor_skip or NULL if all files were already processed
+flags - flags above
+list - list of procesed FileData structures, typically single file or whole list passed to start_editor_*
+data - generic pointer
+*/
+typedef gint (*EditorCallback) (gpointer ed, gint flags, GList *list, gpointer data);
+
+
+
+void editor_resume(gpointer ed);
+void editor_skip(gpointer ed);
+
+
+gint editor_command_parse(const gchar *template, GList *list, gchar **output);
 
 void editor_reset_defaults(void);
 gint start_editor_from_file(gint n, FileData *fd);
 gint start_editor_from_file_list(gint n, GList *list);
-
+gint start_editor_from_file_full(gint n, FileData *fd, EditorCallback cb, gpointer data);
+gint start_editor_from_file_list_full(gint n, GList *list, EditorCallback cb, gpointer data);
 gint editor_window_flag_set(gint n);
-
+const gchar *editor_get_error_str(gint flags);
 
 #endif
 

--- a/src/utilops.c	Wed Sep 12 21:17:31 2007 +0000
+++ b/src/utilops.c	Sun Sep 30 21:10:54 2007 +0000
@@ -28,6 +28,7 @@
 #include "ui_fileops.h"
 #include "ui_misc.h"
 #include "ui_tabcomp.h"
+#include "editors.h"
 
 /*
  *--------------------------------------------------------------------------
@@ -333,35 +334,89 @@
  *--------------------------------------------------------------------------
  */
 
+static gint copy_file_ext_cb(gpointer ed, gint flags, GList *list, gpointer data)
+{
+	if ((flags & EDITOR_ERROR_MASK) && !(flags & EDITOR_ERROR_SKIPPED))
+		{
+		FileData *fd = list->data;
+		gchar *title = _("Error copying file");
+		gchar *text = g_strdup_printf(_("%s\nUnable to copy file:\n%s\nto:\n%s"), editor_get_error_str(flags), fd->name, fd->change->dest); 
+		file_util_warning_dialog(title, text, GTK_STOCK_DIALOG_ERROR, NULL);
+		g_free(text);
+		}
+	while (list)
+		{
+		FileData *fd = list->data;
+		if (!(flags & EDITOR_ERROR_MASK))
+			file_maint_copied(fd);
+		file_data_change_info_free(NULL, fd);
+		list = list->next;
+		}
+	return EDITOR_CB_CONTINUE;
+}
+
+
 gint copy_file_ext(FileData *fd)
 {
-	gint ret;
+	gint ok;
 	g_assert(fd->change);
 	if (editor_command[CMD_COPY])
-		ret = start_editor_from_file(CMD_COPY, fd);
+		{
+		ok = !start_editor_from_file_full(CMD_COPY, fd, copy_file_ext_cb, NULL);
+		if (ok) return ok; /* that's all for now, let's continue in callback */
+		}
 	else
-		ret = copy_file(fd->change->source, fd->change->dest);
-
-	if (ret)
+		ok = copy_file(fd->change->source, fd->change->dest);
+
+	if (ok)
 		{
 		file_maint_copied(fd);
 		}
 
 	file_data_change_info_free(NULL, fd);
 		
-	return ret;
+	return ok;
 }
 
+static gint move_file_ext_cb(gpointer ed, gint flags, GList *list, gpointer data)
+{
+	if ((flags & EDITOR_ERROR_MASK) && !(flags & EDITOR_ERROR_SKIPPED))
+		{
+		FileData *fd = list->data;
+		gchar *title = _("Error moving file");
+		gchar *text = g_strdup_printf(_("%s\nUnable to move file:\n%s\nto:\n%s"), editor_get_error_str(flags), fd->name, fd->change->dest); 
+		file_util_warning_dialog(title, text, GTK_STOCK_DIALOG_ERROR, NULL);
+		g_free(text);
+		}
+	while (list)
+		{
+		FileData *fd = list->data;
+		if (!(flags & EDITOR_ERROR_MASK))
+			{
+			file_data_do_change(fd);
+			file_maint_moved(fd, NULL);
+			}
+		file_data_change_info_free(NULL, fd);
+		list = list->next;
+		}
+	return EDITOR_CB_CONTINUE;
+
+}
+
+
 gint move_file_ext(FileData *fd)
 {
-	gint ret;
+	gint ok;
 	g_assert(fd->change);
 	if (editor_command[CMD_MOVE])
-		ret = start_editor_from_file(CMD_MOVE, fd);
+		{
+		ok = !start_editor_from_file_full(CMD_MOVE, fd, move_file_ext_cb, NULL); 
+		if (ok) return ok; /* that's all for now, let's continue in callback */ 
+		}
 	else
-		ret = move_file(fd->change->source, fd->change->dest);
-
-	if (ret)
+		ok = move_file(fd->change->source, fd->change->dest);
+
+	if (ok)
 		{
 		file_data_do_change(fd);
 		file_maint_moved(fd, NULL);
@@ -369,19 +424,46 @@
 
 	file_data_change_info_free(NULL, fd);
 		
-	return ret;
+	return ok;
+}
+
+static gint rename_file_ext_cb(gpointer ed, gint flags, GList *list, gpointer data)
+{
+	if ((flags & EDITOR_ERROR_MASK) && !(flags & EDITOR_ERROR_SKIPPED))
+		{
+		FileData *fd = list->data;
+		gchar *title = _("Error renaming file");
+		gchar *text = g_strdup_printf(_("%s\nUnable to rename file:\n%s\nto:\n%s"), editor_get_error_str(flags), fd->name, fd->change->dest); 
+		file_util_warning_dialog(title, text, GTK_STOCK_DIALOG_ERROR, NULL);
+		g_free(text);
+		}
+	while (list)
+		{
+		FileData *fd = list->data;
+		if (!(flags & EDITOR_ERROR_MASK))
+			{
+			file_data_do_change(fd);
+			file_maint_renamed(fd);
+			}
+		file_data_change_info_free(NULL, fd);
+		list = list->next;
+		}
+	return EDITOR_CB_CONTINUE;
 }
 
 gint rename_file_ext(FileData *fd)
 {
-	gint ret;
+	gint ok;
 	g_assert(fd->change);
 	if (editor_command[CMD_RENAME])
-		ret = start_editor_from_file(CMD_RENAME, fd);
+		{
+		ok = !start_editor_from_file_full(CMD_RENAME, fd, rename_file_ext_cb, NULL);
+		if (ok) return ok; /* that's all for now, let's continue in callback */
+		}
 	else
-		ret = rename_file(fd->change->source, fd->change->dest);
+		ok = rename_file(fd->change->source, fd->change->dest);
 			
-	if (ret)
+	if (ok)
 		{
 		file_data_do_change(fd);
 		file_maint_renamed(fd);
@@ -389,7 +471,7 @@
 		
 	file_data_change_info_free(NULL, fd);
 		
-	return ret;
+	return ok;
 }
 
 
@@ -1386,26 +1468,81 @@
 static void file_util_delete_multiple_ok_cb(GenericDialog *gd, gpointer data);
 static void file_util_delete_multiple_cancel_cb(GenericDialog *gd, gpointer data);
 
+static void file_util_delete_ext_ok_cb(GenericDialog *gd, gpointer data)
+{
+	editor_resume(data);
+}
+
+static void file_util_delete_ext_cancel_cb(GenericDialog *gd, gpointer data)
+{
+	editor_skip(data);
+}
+
+
+static gint file_util_delete_ext_cb(gpointer resume_data, gint flags, GList *list, gpointer data)
+{
+	gint ret = EDITOR_CB_CONTINUE;
+	if ((flags & EDITOR_ERROR_MASK) && !(flags & EDITOR_ERROR_SKIPPED))
+		{
+			GString *msg = g_string_new(editor_get_error_str(flags));
+			g_string_append(msg,_("\nUnable to delete file by external command:\n")); 
+			GenericDialog *d;
+			while (list)
+				{
+				FileData *fd = list->data;
+				
+				g_string_append(msg, fd->path);
+				g_string_append(msg, "\n");
+				list = list->next;
+				}
+			if (resume_data)
+				{
+				g_string_append(msg, _("\n Continue multiple delete operation?"));
+				d = file_util_gen_dlg(_("Delete failed"), "GQview", "dlg_confirm",
+						      NULL, TRUE,
+						      file_util_delete_ext_cancel_cb, resume_data);
+
+				generic_dialog_add_message(d, GTK_STOCK_DIALOG_WARNING, NULL, msg->str);
+
+				generic_dialog_add_button(d, GTK_STOCK_GO_FORWARD, _("Co_ntinue"),
+							  file_util_delete_ext_ok_cb, TRUE);
+				gtk_widget_show(d->dialog);
+				ret = EDITOR_CB_SUSPEND;
+				}
+			else
+				{
+				file_util_warning_dialog(_("Delete failed"), msg->str, GTK_STOCK_DIALOG_ERROR, NULL);
+				}
+			g_string_free(msg, TRUE);
+		}
+	
+	if (!(flags & EDITOR_ERROR_MASK))
+		{
+		/* files were successfully deleted, call the maint functions */
+		while (list)
+			{
+			FileData *fd = list->data;
+			file_maint_removed(fd, list);
+			list = list->next;
+			}
+		}
+	return ret;
+}
+
 static void file_util_delete_multiple_ok_cb(GenericDialog *gd, gpointer data)
 {
 	GList *source_list = data;
 
 	if (editor_command[CMD_DELETE])
 		{
-		if (!start_editor_from_filelist(CMD_DELETE, source_list))
-			{
-			file_util_warning_dialog(_("File deletion failed"), _("Unable to delete files by external command\n"), GTK_STOCK_DIALOG_ERROR, NULL);
-			}
-		else
+		gint flags;
+		if ((flags = start_editor_from_filelist_full(CMD_DELETE, source_list, file_util_delete_ext_cb, NULL)))
 			{
-			while (source_list)
-				{
-				FileData *fd = source_list->data;
-				source_list = g_list_remove(source_list, fd);
-				file_maint_removed(fd, source_list);
-				file_data_unref(fd);
-				}
+			gchar *text = g_strdup_printf(_("%s\nUnable to delete files by external command.\n"), editor_get_error_str(flags)); 
+			file_util_warning_dialog(_("File deletion failed"), text, GTK_STOCK_DIALOG_ERROR, NULL);
+			g_free(text);
 			}
+		filelist_free(source_list);
 		return;
 		}
 
@@ -1612,16 +1749,13 @@
 
 	if (editor_command[CMD_DELETE])
 		{
-		if (!start_editor_from_file(CMD_DELETE, fd))
+		gint flags;
+		if ((flags = start_editor_from_file_full(CMD_DELETE, fd, file_util_delete_ext_cb, NULL)))
 			{
-			gchar *text = g_strdup_printf(_("Unable to delete file by external command:\n%s"), fd->path);
+			gchar *text = g_strdup_printf(_("%s\nUnable to delete file by external command:\n%s"), editor_get_error_str(flags), fd->path);
 			file_util_warning_dialog(_("File deletion failed"), text, GTK_STOCK_DIALOG_ERROR, NULL);
 			g_free(text);
 			}
-		else
-			{
-			file_maint_removed(fd, NULL);
-			}
 		}
 	else if (!file_util_unlink(fd))
 		{

--- a/src/view_file_icon.c	Wed Sep 12 21:17:31 2007 +0000
+++ b/src/view_file_icon.c	Sun Sep 30 21:10:54 2007 +0000
@@ -2629,10 +2629,9 @@
 			}
 		if (new_row >= 0)
 			{
-			FileData *fdn;
+			FileData *fdn = g_list_nth_data(vfi->list, new_row);
 			IconData *idn = vficon_icon_data(vfi,fdn);
 
-			fdn = g_list_nth_data(vfi->list, new_row);
 			vficon_select(vfi, idn);
 			vficon_send_layout_select(vfi, idn);
 			}