Mercurial > geeqie.yaz

changeset 968:065a129fd42b

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Simplify things related to ViewFileInfoList * and ViewFileInfoIcon * type casting: - drop VFICON_INFO() and VFLIST_INFO() macros - rename VFLIST_INFO_POINTER() and VFICON_INFO_POINTER() to shorter VFLIST() and VFICON(). - replace all related occurrences by shorter expressions. For example: VFICON_INFO(vf, columns) becomes VFICON(vf)->columns VFLIST_INFO(vf, click_fd) becomes VFLIST(vf)->click_fd
author zas_
date Sun, 17 Aug 2008 09:30:45 +0000
parents eb5dbdff14f6
children daabcea563b2
files src/view_file.c src/view_file.h src/view_file_icon.c src/view_file_list.c
diffstat 4 files changed, 225 insertions(+), 228 deletions(-) [+]
line wrap: on
line diff
--- a/src/view_file.c	Sun Aug 17 09:00:33 2008 +0000
+++ b/src/view_file.c	Sun Aug 17 09:30:45 2008 +0000
@@ -496,11 +496,11 @@
 	switch(vf->type)
 	{
 	case FILEVIEW_LIST:
-		vflist_color_set(vf, VFLIST_INFO(vf, click_fd), TRUE);
-		active = (VFLIST_INFO(vf, click_fd) != NULL);
+		vflist_color_set(vf, VFLIST(vf)->click_fd, TRUE);
+		active = (VFLIST(vf)->click_fd != NULL);
 		break;
 	case FILEVIEW_ICON:
-		active = (VFICON_INFO(vf, click_id) != NULL);
+		active = (VFICON(vf)->click_id != NULL);
 		break;
 	}
 
@@ -594,11 +594,11 @@
 	switch(vf->type)
 	{
 	case FILEVIEW_LIST:
-		menu_item_add_check(menu, _("Show _thumbnails"), VFLIST_INFO(vf, thumbs_enabled),
+		menu_item_add_check(menu, _("Show _thumbnails"), VFLIST(vf)->thumbs_enabled,
 				    G_CALLBACK(vflist_pop_menu_thumbs_cb), vf);
 		break;
 	case FILEVIEW_ICON:
-		menu_item_add_check(menu, _("Show filename _text"), VFICON_INFO(vf, show_text),
+		menu_item_add_check(menu, _("Show filename _text"), VFICON(vf)->show_text,
 				    G_CALLBACK(vficon_pop_menu_show_names_cb), vf);
 		break;
 	}
--- a/src/view_file.h	Sun Aug 17 09:00:33 2008 +0000
+++ b/src/view_file.h	Sun Aug 17 09:30:45 2008 +0000
@@ -14,11 +14,8 @@
 
 #define VIEW_FILE_TYPES_COUNT 2
 
-#define VFLIST_INFO_POINTER(_vf_) ((ViewFileInfoList *)(_vf_->info))
-#define VFLIST_INFO(_vf_, _part_) (VFLIST_INFO_POINTER(_vf_)->_part_)
-
-#define VFICON_INFO_POINTER(_vf_) ((ViewFileInfoIcon *)(_vf_->info))
-#define VFICON_INFO(_vf_, _part_) (VFICON_INFO_POINTER(_vf_)->_part_)
+#define VFLIST(_vf_) ((ViewFileInfoList *)(_vf_->info))
+#define VFICON(_vf_) ((ViewFileInfoIcon *)(_vf_->info))
 
 void vf_send_update(ViewFile *vf);
 
--- a/src/view_file_icon.c	Sun Aug 17 09:00:33 2008 +0000
+++ b/src/view_file_icon.c	Sun Aug 17 09:30:45 2008 +0000
@@ -165,23 +165,23 @@
 
 GList *vficon_pop_menu_file_list(ViewFile *vf)
 {
-	if (!VFICON_INFO(vf, click_id)) return NULL;
-
-	if (VFICON_INFO(vf, click_id)->selected & SELECTION_SELECTED)
+	if (!VFICON(vf)->click_id) return NULL;
+
+	if (VFICON(vf)->click_id->selected & SELECTION_SELECTED)
 		{
 		return vf_selection_get_list(vf);
 		}
 
-	return g_list_append(NULL, file_data_ref(VFICON_INFO(vf, click_id)->fd));
+	return g_list_append(NULL, file_data_ref(VFICON(vf)->click_id->fd));
 }
 
 void vficon_pop_menu_view_cb(GtkWidget *widget, gpointer data)
 {
 	ViewFile *vf = data;
 
-	if (!VFICON_INFO(vf, click_id)) return;
-
-	if (VFICON_INFO(vf, click_id)->selected & SELECTION_SELECTED)
+	if (!VFICON(vf)->click_id) return;
+
+	if (VFICON(vf)->click_id->selected & SELECTION_SELECTED)
 		{
 		GList *list;
 
@@ -191,7 +191,7 @@
 		}
 	else
 		{
-		view_window_new(VFICON_INFO(vf, click_id)->fd);
+		view_window_new(VFICON(vf)->click_id->fd);
 		}
 }
 
@@ -219,8 +219,8 @@
 void vficon_popup_destroy_cb(GtkWidget *widget, gpointer data)
 {
 	ViewFile *vf = data;
-	vficon_selection_remove(vf, VFICON_INFO(vf, click_id), SELECTION_PRELIGHT, NULL);
-	VFICON_INFO(vf, click_id) = NULL;
+	vficon_selection_remove(vf, VFICON(vf)->click_id, SELECTION_PRELIGHT, NULL);
+	VFICON(vf)->click_id = NULL;
 	vf->popup = NULL;
 }
 
@@ -264,8 +264,8 @@
 
 static void vficon_toggle_filenames(ViewFile *vf)
 {
-	VFICON_INFO(vf, show_text) = !VFICON_INFO(vf, show_text);
-	options->show_icon_names = VFICON_INFO(vf, show_text);
+	VFICON(vf)->show_text = !VFICON(vf)->show_text;
+	options->show_icon_names = VFICON(vf)->show_text;
 
 	vficon_populate_at_new_size(vf, vf->listview->allocation.width, vf->listview->allocation.height, TRUE);
 }
@@ -274,7 +274,7 @@
 {
 	gint width;
 
-	if (!VFICON_INFO(vf, show_text)) return options->thumbnails.max_width;
+	if (!VFICON(vf)->show_text) return options->thumbnails.max_width;
 
 	width = options->thumbnails.max_width + options->thumbnails.max_width / 2;
 	if (width < THUMB_MIN_ICON_WIDTH) width = THUMB_MIN_ICON_WIDTH;
@@ -297,8 +297,8 @@
 
 	if (n < 0) return FALSE;
 
-	*row = n / VFICON_INFO(vf, columns);
-	*col = n - (*row * VFICON_INFO(vf, columns));
+	*row = n / VFICON(vf)->columns;
+	*col = n - (*row * VFICON(vf)->columns);
 
 	return TRUE;
 }
@@ -409,34 +409,34 @@
 	GtkWidget *label;
 	gint x, y;
 
-	if (VFICON_INFO(vf, tip_window)) return;
+	if (VFICON(vf)->tip_window) return;
 
 	gdk_window_get_pointer(gtk_tree_view_get_bin_window(GTK_TREE_VIEW(vf->listview)), &x, &y, NULL);
 
-	VFICON_INFO(vf, tip_id) = vficon_find_data_by_coord(vf, x, y, NULL);
-	if (!VFICON_INFO(vf, tip_id)) return;
-
-	VFICON_INFO(vf, tip_window) = gtk_window_new(GTK_WINDOW_POPUP);
-	gtk_window_set_resizable(GTK_WINDOW(VFICON_INFO(vf, tip_window)), FALSE);
-	gtk_container_set_border_width(GTK_CONTAINER(VFICON_INFO(vf, tip_window)), 2);
-
-	label = gtk_label_new(VFICON_INFO(vf, tip_id)->fd->name);
-
-	g_object_set_data(G_OBJECT(VFICON_INFO(vf, tip_window)), "tip_label", label);
-	gtk_container_add(GTK_CONTAINER(VFICON_INFO(vf, tip_window)), label);
+	VFICON(vf)->tip_id = vficon_find_data_by_coord(vf, x, y, NULL);
+	if (!VFICON(vf)->tip_id) return;
+
+	VFICON(vf)->tip_window = gtk_window_new(GTK_WINDOW_POPUP);
+	gtk_window_set_resizable(GTK_WINDOW(VFICON(vf)->tip_window), FALSE);
+	gtk_container_set_border_width(GTK_CONTAINER(VFICON(vf)->tip_window), 2);
+
+	label = gtk_label_new(VFICON(vf)->tip_id->fd->name);
+
+	g_object_set_data(G_OBJECT(VFICON(vf)->tip_window), "tip_label", label);
+	gtk_container_add(GTK_CONTAINER(VFICON(vf)->tip_window), label);
 	gtk_widget_show(label);
 
 	gdk_window_get_pointer(NULL, &x, &y, NULL);
 
-	if (!GTK_WIDGET_REALIZED(VFICON_INFO(vf, tip_window))) gtk_widget_realize(VFICON_INFO(vf, tip_window));
-	gtk_window_move(GTK_WINDOW(VFICON_INFO(vf, tip_window)), x + 16, y + 16);
-	gtk_widget_show(VFICON_INFO(vf, tip_window));
+	if (!GTK_WIDGET_REALIZED(VFICON(vf)->tip_window)) gtk_widget_realize(VFICON(vf)->tip_window);
+	gtk_window_move(GTK_WINDOW(VFICON(vf)->tip_window), x + 16, y + 16);
+	gtk_widget_show(VFICON(vf)->tip_window);
 }
 
 static void tip_hide(ViewFile *vf)
 {
-	if (VFICON_INFO(vf, tip_window)) gtk_widget_destroy(VFICON_INFO(vf, tip_window));
-	VFICON_INFO(vf, tip_window) = NULL;
+	if (VFICON(vf)->tip_window) gtk_widget_destroy(VFICON(vf)->tip_window);
+	VFICON(vf)->tip_window = NULL;
 }
 
 static gint tip_schedule_cb(gpointer data)
@@ -444,7 +444,7 @@
 	ViewFile *vf = data;
 	GtkWidget *window;
 
-	if (VFICON_INFO(vf, tip_delay_id) == -1) return FALSE;
+	if (VFICON(vf)->tip_delay_id == -1) return FALSE;
 
 	window = gtk_widget_get_toplevel(vf->listview);
 
@@ -454,7 +454,7 @@
 		tip_show(vf);
 		}
 
-	VFICON_INFO(vf, tip_delay_id) = -1;
+	VFICON(vf)->tip_delay_id = -1;
 	return FALSE;
 }
 
@@ -462,15 +462,15 @@
 {
 	tip_hide(vf);
 
-	if (VFICON_INFO(vf, tip_delay_id) != -1)
+	if (VFICON(vf)->tip_delay_id != -1)
 		{
-		g_source_remove(VFICON_INFO(vf, tip_delay_id));
-		VFICON_INFO(vf, tip_delay_id) = -1;
+		g_source_remove(VFICON(vf)->tip_delay_id);
+		VFICON(vf)->tip_delay_id = -1;
 		}
 
-	if (!VFICON_INFO(vf, show_text))
+	if (!VFICON(vf)->show_text)
 		{
-		VFICON_INFO(vf, tip_delay_id) = g_timeout_add(VFICON_TIP_DELAY, tip_schedule_cb, vf);
+		VFICON(vf)->tip_delay_id = g_timeout_add(VFICON_TIP_DELAY, tip_schedule_cb, vf);
 		}
 }
 
@@ -478,34 +478,34 @@
 {
 	tip_hide(vf);
 
-	if (VFICON_INFO(vf, tip_delay_id) != -1) g_source_remove(VFICON_INFO(vf, tip_delay_id));
-	VFICON_INFO(vf, tip_delay_id) = -1;
+	if (VFICON(vf)->tip_delay_id != -1) g_source_remove(VFICON(vf)->tip_delay_id);
+	VFICON(vf)->tip_delay_id = -1;
 }
 
 static void tip_update(ViewFile *vf, IconData *id)
 {
-	if (VFICON_INFO(vf, tip_window))
+	if (VFICON(vf)->tip_window)
 		{
 		gint x, y;
 
 		gdk_window_get_pointer(NULL, &x, &y, NULL);
-		gtk_window_move(GTK_WINDOW(VFICON_INFO(vf, tip_window)), x + 16, y + 16);
-
-		if (id != VFICON_INFO(vf, tip_id))
+		gtk_window_move(GTK_WINDOW(VFICON(vf)->tip_window), x + 16, y + 16);
+
+		if (id != VFICON(vf)->tip_id)
 			{
 			GtkWidget *label;
 
-			VFICON_INFO(vf, tip_id) = id;
-
-			if (!VFICON_INFO(vf, tip_id))
+			VFICON(vf)->tip_id = id;
+
+			if (!VFICON(vf)->tip_id)
 				{
 				tip_hide(vf);
 				tip_schedule(vf);
 				return;
 				}
 
-			label = g_object_get_data(G_OBJECT(VFICON_INFO(vf, tip_window)), "tip_label");
-			gtk_label_set_text(GTK_LABEL(label), VFICON_INFO(vf, tip_id)->fd->name);
+			label = g_object_get_data(G_OBJECT(VFICON(vf)->tip_window), "tip_label");
+			gtk_label_set_text(GTK_LABEL(label), VFICON(vf)->tip_id->fd->name);
 			}
 		}
 	else
@@ -529,15 +529,15 @@
 	gchar *uri_text = NULL;
 	gint total;
 
-	if (!VFICON_INFO(vf, click_id)) return;
-
-	if (VFICON_INFO(vf, click_id)->selected & SELECTION_SELECTED)
+	if (!VFICON(vf)->click_id) return;
+
+	if (VFICON(vf)->click_id->selected & SELECTION_SELECTED)
 		{
 		list = vf_selection_get_list(vf);
 		}
 	else
 		{
-		list = g_list_append(NULL, file_data_ref(VFICON_INFO(vf, click_id)->fd));
+		list = g_list_append(NULL, file_data_ref(VFICON(vf)->click_id->fd));
 		}
 
 	if (!list) return;
@@ -557,16 +557,16 @@
 
 	tip_unschedule(vf);
 
-	if (VFICON_INFO(vf, click_id) && VFICON_INFO(vf, click_id)->fd->thumb_pixbuf)
+	if (VFICON(vf)->click_id && VFICON(vf)->click_id->fd->thumb_pixbuf)
 		{
 		gint items;
 
-		if (VFICON_INFO(vf, click_id)->selected & SELECTION_SELECTED)
-			items = g_list_length(VFICON_INFO(vf, selection));
+		if (VFICON(vf)->click_id->selected & SELECTION_SELECTED)
+			items = g_list_length(VFICON(vf)->selection);
 		else
 			items = 1;
 
-		dnd_set_drag_icon(widget, context, VFICON_INFO(vf, click_id)->fd->thumb_pixbuf, items);
+		dnd_set_drag_icon(widget, context, VFICON(vf)->click_id->fd->thumb_pixbuf, items);
 		}
 }
 
@@ -574,7 +574,7 @@
 {
 	ViewFile *vf = data;
 
-	vficon_selection_remove(vf, VFICON_INFO(vf, click_id), SELECTION_PRELIGHT, NULL);
+	vficon_selection_remove(vf, VFICON(vf)->click_id, SELECTION_PRELIGHT, NULL);
 
 	if (context->action == GDK_ACTION_MOVE)
 		{
@@ -661,7 +661,7 @@
 {
 	GList *work;
 
-	work = VFICON_INFO(vf, selection);
+	work = VFICON(vf)->selection;
 	while (work)
 		{
 		IconData *id = work->data;
@@ -669,7 +669,7 @@
 
 		if (vficon_index_by_id(vf, id) >= 0) continue;
 
-		VFICON_INFO(vf, selection) = g_list_remove(VFICON_INFO(vf, selection), id);
+		VFICON(vf)->selection = g_list_remove(VFICON(vf)->selection, id);
 		}
 }
 
@@ -677,8 +677,8 @@
 {
 	GList *work;
 
-	g_list_free(VFICON_INFO(vf, selection));
-	VFICON_INFO(vf, selection) = NULL;
+	g_list_free(VFICON(vf)->selection);
+	VFICON(vf)->selection = NULL;
 
 	work = vf->list;
 	while (work)
@@ -686,7 +686,7 @@
 		IconData *id = work->data;
 		work = work->next;
 		
-		VFICON_INFO(vf, selection) = g_list_append(VFICON_INFO(vf, selection), id);
+		VFICON(vf)->selection = g_list_append(VFICON(vf)->selection, id);
 		vficon_selection_add(vf, id, SELECTION_SELECTED, NULL);
 		}
 
@@ -697,7 +697,7 @@
 {
 	GList *work;
 
-	work = VFICON_INFO(vf, selection);
+	work = VFICON(vf)->selection;
 	while (work)
 		{
 		IconData *id = work->data;
@@ -706,8 +706,8 @@
 		vficon_selection_remove(vf, id, SELECTION_SELECTED, NULL);
 		}
 
-	g_list_free(VFICON_INFO(vf, selection));
-	VFICON_INFO(vf, selection) = NULL;
+	g_list_free(VFICON(vf)->selection);
+	VFICON(vf)->selection = NULL;
 
 	vf_send_update(vf);
 }
@@ -724,12 +724,12 @@
 
 		if (id->selected & SELECTION_SELECTED)
 			{
-			VFICON_INFO(vf, selection) = g_list_remove(VFICON_INFO(vf, selection), id);
+			VFICON(vf)->selection = g_list_remove(VFICON(vf)->selection, id);
 			vficon_selection_remove(vf, id, SELECTION_SELECTED, NULL);
 			}
 		else
 			{
-			VFICON_INFO(vf, selection) = g_list_append(VFICON_INFO(vf, selection), id);
+			VFICON(vf)->selection = g_list_append(VFICON(vf)->selection, id);
 			vficon_selection_add(vf, id, SELECTION_SELECTED, NULL);
 			}
 		}
@@ -739,11 +739,11 @@
 
 static void vficon_select(ViewFile *vf, IconData *id)
 {
-	VFICON_INFO(vf, prev_selection) = id;
+	VFICON(vf)->prev_selection = id;
 
 	if (!id || id->selected & SELECTION_SELECTED) return;
 
-	VFICON_INFO(vf, selection) = g_list_append(VFICON_INFO(vf, selection), id);
+	VFICON(vf)->selection = g_list_append(VFICON(vf)->selection, id);
 	vficon_selection_add(vf, id, SELECTION_SELECTED, NULL);
 
 	vf_send_update(vf);
@@ -751,11 +751,11 @@
 
 static void vficon_unselect(ViewFile *vf, IconData *id)
 {
-	VFICON_INFO(vf, prev_selection) = id;
+	VFICON(vf)->prev_selection = id;
 
 	if (!id || !(id->selected & SELECTION_SELECTED) ) return;
 
-	VFICON_INFO(vf, selection) = g_list_remove(VFICON_INFO(vf, selection), id);
+	VFICON(vf)->selection = g_list_remove(VFICON(vf)->selection, id);
 	vficon_selection_remove(vf, id, SELECTION_SELECTED, NULL);
 
 	vf_send_update(vf);
@@ -783,7 +783,7 @@
 	if (!vficon_find_position(vf, start, &row1, &col1) ||
 	    !vficon_find_position(vf, end, &row2, &col2) ) return;
 
-	VFICON_INFO(vf, prev_selection) = end;
+	VFICON(vf)->prev_selection = end;
 
 	if (!options->collections.rectangular_selection)
 		{
@@ -852,7 +852,7 @@
 		gint64 b = 0;
 		GList *work;
 
-		work = VFICON_INFO(vf, selection);
+		work = VFICON(vf)->selection;
 		while (work)
 			{
 			IconData *id = work->data;
@@ -866,7 +866,7 @@
 		*bytes = b;
 		}
 
-	return g_list_length(VFICON_INFO(vf, selection));
+	return g_list_length(VFICON(vf)->selection);
 }
 
 GList *vficon_selection_get_list(ViewFile *vf)
@@ -874,7 +874,7 @@
 	GList *list = NULL;
 	GList *work;
 
-	work = VFICON_INFO(vf, selection);
+	work = VFICON(vf)->selection;
 	while (work)
 		{
 		IconData *id = work->data;
@@ -896,7 +896,7 @@
 	GList *list = NULL;
 	GList *work;
 
-	work = VFICON_INFO(vf, selection);
+	work = VFICON(vf)->selection;
 	while (work)
 		{
 		list = g_list_prepend(list, GINT_TO_POINTER(g_list_index(vf->list, work->data)));
@@ -1041,12 +1041,12 @@
 
 	if (relative)
 		{
-		new_row = VFICON_INFO(vf, focus_row);
-		new_col = VFICON_INFO(vf, focus_column);
+		new_row = VFICON(vf)->focus_row;
+		new_col = VFICON(vf)->focus_column;
 
 		new_row += row;
 		if (new_row < 0) new_row = 0;
-		if (new_row >= VFICON_INFO(vf, rows)) new_row = VFICON_INFO(vf, rows) - 1;
+		if (new_row >= VFICON(vf)->rows) new_row = VFICON(vf)->rows - 1;
 
 		while (col != 0)
 			{
@@ -1066,23 +1066,23 @@
 				if (new_row > 0)
 					{
 					new_row--;
-					new_col = VFICON_INFO(vf, columns) - 1;
+					new_col = VFICON(vf)->columns - 1;
 					}
 				else
 					{
 					new_col = 0;
 					}
 				}
-			if (new_col >= VFICON_INFO(vf, columns))
+			if (new_col >= VFICON(vf)->columns)
 				{
-				if (new_row < VFICON_INFO(vf, rows) - 1)
+				if (new_row < VFICON(vf)->rows - 1)
 					{
 					new_row++;
 					new_col = 0;
 					}
 				else
 					{
-					new_col = VFICON_INFO(vf, columns) - 1;
+					new_col = VFICON(vf)->columns - 1;
 					}
 				}
 			}
@@ -1092,25 +1092,25 @@
 		new_row = row;
 		new_col = col;
 
-		if (new_row >= VFICON_INFO(vf, rows))
+		if (new_row >= VFICON(vf)->rows)
 			{
-			if (VFICON_INFO(vf, rows) > 0)
-				new_row = VFICON_INFO(vf, rows) - 1;
+			if (VFICON(vf)->rows > 0)
+				new_row = VFICON(vf)->rows - 1;
 			else
 				new_row = 0;
-			new_col = VFICON_INFO(vf, columns) - 1;
+			new_col = VFICON(vf)->columns - 1;
 			}
-		if (new_col >= VFICON_INFO(vf, columns)) new_col = VFICON_INFO(vf, columns) - 1;
+		if (new_col >= VFICON(vf)->columns) new_col = VFICON(vf)->columns - 1;
 		}
 
-	if (new_row == VFICON_INFO(vf, rows) - 1)
+	if (new_row == VFICON(vf)->rows - 1)
 		{
 		gint l;
 
 		/* if we moved beyond the last image, go to the last image */
 
 		l = g_list_length(vf->list);
-		if (VFICON_INFO(vf, rows) > 1) l -= (VFICON_INFO(vf, rows) - 1) * VFICON_INFO(vf, columns);
+		if (VFICON(vf)->rows > 1) l -= (VFICON(vf)->rows - 1) * VFICON(vf)->columns;
 		if (new_col >= l) new_col = l - 1;
 		}
 
@@ -1122,31 +1122,31 @@
 	GtkTreeIter iter;
 	gint row, col;
 
-	if (g_list_find(vf->list, VFICON_INFO(vf, focus_id)))
+	if (g_list_find(vf->list, VFICON(vf)->focus_id))
 		{
-		if (id == VFICON_INFO(vf, focus_id))
+		if (id == VFICON(vf)->focus_id)
 			{
 			/* ensure focus row col are correct */
-			vficon_find_position(vf, VFICON_INFO(vf, focus_id), &VFICON_INFO(vf, focus_row), &VFICON_INFO(vf, focus_column));
+			vficon_find_position(vf, VFICON(vf)->focus_id, &VFICON(vf)->focus_row, &VFICON(vf)->focus_column);
 			return;
 			}
-		vficon_selection_remove(vf, VFICON_INFO(vf, focus_id), SELECTION_FOCUS, NULL);
+		vficon_selection_remove(vf, VFICON(vf)->focus_id, SELECTION_FOCUS, NULL);
 		}
 
 	if (!vficon_find_position(vf, id, &row, &col))
 		{
-		VFICON_INFO(vf, focus_id) = NULL;
-		VFICON_INFO(vf, focus_row) = -1;
-		VFICON_INFO(vf, focus_column) = -1;
+		VFICON(vf)->focus_id = NULL;
+		VFICON(vf)->focus_row = -1;
+		VFICON(vf)->focus_column = -1;
 		return;
 		}
 
-	VFICON_INFO(vf, focus_id) = id;
-	VFICON_INFO(vf, focus_row) = row;
-	VFICON_INFO(vf, focus_column) = col;
-	vficon_selection_add(vf, VFICON_INFO(vf, focus_id), SELECTION_FOCUS, NULL);
-
-	if (vficon_find_iter(vf, VFICON_INFO(vf, focus_id), &iter, NULL))
+	VFICON(vf)->focus_id = id;
+	VFICON(vf)->focus_row = row;
+	VFICON(vf)->focus_column = col;
+	vficon_selection_add(vf, VFICON(vf)->focus_id, SELECTION_FOCUS, NULL);
+
+	if (vficon_find_iter(vf, VFICON(vf)->focus_id, &iter, NULL))
 		{
 		GtkTreePath *tpath;
 		GtkTreeViewColumn *column;
@@ -1168,15 +1168,15 @@
 	gint new_row = 0;
 	gint new_col = 0;
 
-	if (VFICON_INFO(vf, focus_id) && vficon_find_position(vf, VFICON_INFO(vf, focus_id), &new_row, &new_col))
+	if (VFICON(vf)->focus_id && vficon_find_position(vf, VFICON(vf)->focus_id, &new_row, &new_col))
 		{
 		/* first find the old focus, if it exists and is valid */
 		}
 	else
 		{
 		/* (try to) stay where we were */
-		new_row = VFICON_INFO(vf, focus_row);
-		new_col = VFICON_INFO(vf, focus_column);
+		new_row = VFICON(vf)->focus_row;
+		new_col = VFICON(vf)->focus_column;
 		}
 
 	vficon_move_focus(vf, new_row, new_col, FALSE);
@@ -1194,7 +1194,7 @@
 	page_size = (gint)adj->page_increment;
 
 	row_height = options->thumbnails.max_height + THUMB_BORDER_PADDING * 2;
-	if (VFICON_INFO(vf, show_text)) row_height += options->thumbnails.max_height / 3;
+	if (VFICON(vf)->show_text) row_height += options->thumbnails.max_height / 3;
 
 	ret = page_size / row_height;
 	if (ret < 1) ret = 1;
@@ -1217,7 +1217,7 @@
 	GtkTreePath *tpath;
 	gint cw, ch;
 
-	if (!vficon_find_iter(vf, VFICON_INFO(vf, click_id), &iter, &column)) return;
+	if (!vficon_find_iter(vf, VFICON(vf)->click_id, &iter, &column)) return;
 	store = gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview));
 	tpath = gtk_tree_model_get_path(store, &iter);
 	tree_view_get_cell_clamped(GTK_TREE_VIEW(vf->listview), tpath, column, FALSE, x, y, &cw, &ch);
@@ -1256,18 +1256,18 @@
 			focus_row = page_height(vf);
 			break;
 		case GDK_Home: case GDK_KP_Home:
-			focus_row = -VFICON_INFO(vf, focus_row);
-			focus_col = -VFICON_INFO(vf, focus_column);
+			focus_row = -VFICON(vf)->focus_row;
+			focus_col = -VFICON(vf)->focus_column;
 			break;
 		case GDK_End: case GDK_KP_End:
-			focus_row = VFICON_INFO(vf, rows) - 1 - VFICON_INFO(vf, focus_row);
-			focus_col = VFICON_INFO(vf, columns) - 1 - VFICON_INFO(vf, focus_column);
+			focus_row = VFICON(vf)->rows - 1 - VFICON(vf)->focus_row;
+			focus_col = VFICON(vf)->columns - 1 - VFICON(vf)->focus_column;
 			break;
 		case GDK_space:
-			id = vficon_find_data(vf, VFICON_INFO(vf, focus_row), VFICON_INFO(vf, focus_column), NULL);
+			id = vficon_find_data(vf, VFICON(vf)->focus_row, VFICON(vf)->focus_column, NULL);
 			if (id)
 				{
-				VFICON_INFO(vf, click_id) = id;
+				VFICON(vf)->click_id = id;
 				if (event->state & GDK_CONTROL_MASK)
 					{
 					gint selected;
@@ -1292,10 +1292,10 @@
 				}
 			break;
 		case GDK_Menu:
-			id = vficon_find_data(vf, VFICON_INFO(vf, focus_row), VFICON_INFO(vf, focus_column), NULL);
-			VFICON_INFO(vf, click_id) = id;
-
-			vficon_selection_add(vf, VFICON_INFO(vf, click_id), SELECTION_PRELIGHT, NULL);
+			id = vficon_find_data(vf, VFICON(vf)->focus_row, VFICON(vf)->focus_column, NULL);
+			VFICON(vf)->click_id = id;
+
+			vficon_selection_add(vf, VFICON(vf)->click_id, SELECTION_PRELIGHT, NULL);
 			tip_unschedule(vf);
 
 			vf->popup = vf_pop_menu(vf);
@@ -1311,9 +1311,9 @@
 		IconData *new_id;
 		IconData *old_id;
 
-		old_id = vficon_find_data(vf, VFICON_INFO(vf, focus_row), VFICON_INFO(vf, focus_column), NULL);
+		old_id = vficon_find_data(vf, VFICON(vf)->focus_row, VFICON(vf)->focus_column, NULL);
 		vficon_move_focus(vf, focus_row, focus_col, TRUE);
-		new_id = vficon_find_data(vf, VFICON_INFO(vf, focus_row), VFICON_INFO(vf, focus_column), NULL);
+		new_id = vficon_find_data(vf, VFICON(vf)->focus_row, VFICON(vf)->focus_column, NULL);
 
 		if (new_id != old_id)
 			{
@@ -1325,18 +1325,18 @@
 					}
 				else
 					{
-					vficon_select_region_util(vf, VFICON_INFO(vf, click_id), old_id, FALSE);
+					vficon_select_region_util(vf, VFICON(vf)->click_id, old_id, FALSE);
 					}
-				vficon_select_region_util(vf, VFICON_INFO(vf, click_id), new_id, TRUE);
+				vficon_select_region_util(vf, VFICON(vf)->click_id, new_id, TRUE);
 				vficon_send_layout_select(vf, new_id);
 				}
 			else if (event->state & GDK_CONTROL_MASK)
 				{
-				VFICON_INFO(vf, click_id) = new_id;
+				VFICON(vf)->click_id = new_id;
 				}
 			else
 				{
-				VFICON_INFO(vf, click_id) = new_id;
+				VFICON(vf)->click_id = new_id;
 				vf_select_none(vf);
 				vficon_select(vf, new_id);
 				vficon_send_layout_select(vf, new_id);
@@ -1382,8 +1382,8 @@
 
 	id = vficon_find_data_by_coord(vf, (gint)bevent->x, (gint)bevent->y, &iter);
 
-	VFICON_INFO(vf, click_id) = id;
-	vficon_selection_add(vf, VFICON_INFO(vf, click_id), SELECTION_PRELIGHT, &iter);
+	VFICON(vf)->click_id = id;
+	vficon_selection_add(vf, VFICON(vf)->click_id, SELECTION_PRELIGHT, &iter);
 
 	switch (bevent->button)
 		{
@@ -1396,7 +1396,7 @@
 			if (bevent->type == GDK_2BUTTON_PRESS &&
 			    vf->layout)
 				{
-				vficon_selection_remove(vf, VFICON_INFO(vf, click_id), SELECTION_PRELIGHT, &iter);
+				vficon_selection_remove(vf, VFICON(vf)->click_id, SELECTION_PRELIGHT, &iter);
 				layout_image_full_screen_start(vf->layout);
 				}
 #endif
@@ -1426,12 +1426,12 @@
 		id = vficon_find_data_by_coord(vf, (gint)bevent->x, (gint)bevent->y, &iter);
 		}
 
-	if (VFICON_INFO(vf, click_id))
+	if (VFICON(vf)->click_id)
 		{
-		vficon_selection_remove(vf, VFICON_INFO(vf, click_id), SELECTION_PRELIGHT, NULL);
+		vficon_selection_remove(vf, VFICON(vf)->click_id, SELECTION_PRELIGHT, NULL);
 		}
 
-	if (!id || VFICON_INFO(vf, click_id) != id) return TRUE;
+	if (!id || VFICON(vf)->click_id != id) return TRUE;
 
 	was_selected = (id->selected & SELECTION_SELECTED);
 
@@ -1446,9 +1446,9 @@
 				gint select;
 
 				select = !(id->selected & SELECTION_SELECTED);
-				if ((bevent->state & GDK_SHIFT_MASK) && VFICON_INFO(vf, prev_selection))
+				if ((bevent->state & GDK_SHIFT_MASK) && VFICON(vf)->prev_selection)
 					{
-					vficon_select_region_util(vf, VFICON_INFO(vf, prev_selection), id, select);
+					vficon_select_region_util(vf, VFICON(vf)->prev_selection, id, select);
 					}
 				else
 					{
@@ -1459,9 +1459,9 @@
 				{
 				vf_select_none(vf);
 
-				if ((bevent->state & GDK_SHIFT_MASK) && VFICON_INFO(vf, prev_selection))
+				if ((bevent->state & GDK_SHIFT_MASK) && VFICON(vf)->prev_selection)
 					{
-					vficon_select_region_util(vf, VFICON_INFO(vf, prev_selection), id, TRUE);
+					vficon_select_region_util(vf, VFICON(vf)->prev_selection, id, TRUE);
 					}
 				else
 					{
@@ -1542,7 +1542,7 @@
 	GList *list = NULL;
 	gint i;
 
-	for (i = 0; i < VFICON_INFO(vf, columns); i++) list = g_list_prepend(list, NULL);
+	for (i = 0; i < VFICON(vf)->columns; i++) list = g_list_prepend(list, NULL);
 
 	store = GTK_LIST_STORE(gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview)));
 	gtk_list_store_append(store, iter);
@@ -1595,7 +1595,7 @@
 			GList *list;
 
 			column = gtk_tree_view_get_column(GTK_TREE_VIEW(vf->listview), i);
-			gtk_tree_view_column_set_visible(column, (i < VFICON_INFO(vf, columns)));
+			gtk_tree_view_column_set_visible(column, (i < VFICON(vf)->columns));
 			gtk_tree_view_column_set_fixed_width(column, thumb_width + (THUMB_BORDER_PADDING * 6));
 
 			list = gtk_tree_view_column_get_cell_renderers(column);
@@ -1606,7 +1606,7 @@
 				{
 				g_object_set(G_OBJECT(cell), "fixed_width", thumb_width,
 							     "fixed_height", options->thumbnails.max_height,
-							     "show_text", VFICON_INFO(vf, show_text),
+							     "show_text", VFICON(vf)->show_text,
 							     "show_marks", vf->marks_enabled,
 							     "num_marks", FILEDATA_MARKS_SIZE,
 							     NULL);
@@ -1669,7 +1669,7 @@
 		g_list_free(list);
 		}
 
-	VFICON_INFO(vf, rows) = r;
+	VFICON(vf)->rows = r;
 
 	if (visible_id &&
 	    gtk_tree_view_get_path_at_pos(GTK_TREE_VIEW(vf->listview), 0, 0, &tpath, NULL, NULL, NULL))
@@ -1703,13 +1703,13 @@
 	new_cols = w / (thumb_width + (THUMB_BORDER_PADDING * 6));
 	if (new_cols < 1) new_cols = 1;
 
-	if (!force && new_cols == VFICON_INFO(vf, columns)) return;
-
-	VFICON_INFO(vf, columns) = new_cols;
+	if (!force && new_cols == VFICON(vf)->columns) return;
+
+	VFICON(vf)->columns = new_cols;
 
 	vficon_populate(vf, TRUE, TRUE);
 
-	DEBUG_1("col tab pop cols=%d rows=%d", VFICON_INFO(vf, columns), VFICON_INFO(vf, rows));
+	DEBUG_1("col tab pop cols=%d rows=%d", VFICON(vf)->columns, VFICON(vf)->rows);
 }
 
 #if 0
@@ -1721,7 +1721,7 @@
 	gint r, c;
 	gint valid;
 
-	if (VFICON_INFO(vf, rows) == 0) return;
+	if (VFICON(vf)->rows == 0) return;
 
 	store = gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview));
 
@@ -1779,7 +1779,7 @@
 		g_list_free(list);
 		}
 
-	VFICON_INFO(vf, rows) = r;
+	VFICON(vf)->rows = r;
 
 	vficon_update_focus(vf);
 }
@@ -1788,12 +1788,12 @@
 #if 0
 static void vficon_sync_idle(ViewFile *vf)
 {
-	if (VFICON_INFO(vf, sync_idle_id) == -1)
+	if (VFICON(vf)->sync_idle_id == -1)
 		{
 		/* high priority, the view needs to be resynced before a redraw
 		 * may contain invalid pointers at this time
 		 */
-		VFICON_INFO(vf, sync_idle_id) = g_idle_add_full(G_PRIORITY_HIGH, vficon_sync_idle_cb, vf, NULL);
+		VFICON(vf)->sync_idle_id = g_idle_add_full(G_PRIORITY_HIGH, vficon_sync_idle_cb, vf, NULL);
 		}
 }
 #endif
@@ -2131,7 +2131,7 @@
 	gint num_selected = 0;
 	GList *new_iconlist = NULL;
 
-	focus_id = VFICON_INFO(vf, focus_id);
+	focus_id = VFICON(vf)->focus_id;
 
 	if (vf->dir_fd)
 		{
@@ -2190,8 +2190,8 @@
 			/* file no longer exists, delete from vf->list */
 			GList *to_delete = work;
 			work = work->next;
-			if (id == VFICON_INFO(vf, prev_selection)) VFICON_INFO(vf, prev_selection) = NULL;
-			if (id == VFICON_INFO(vf, click_id)) VFICON_INFO(vf, click_id) = NULL;
+			if (id == VFICON(vf)->prev_selection) VFICON(vf)->prev_selection = NULL;
+			if (id == VFICON(vf)->click_id) VFICON(vf)->click_id = NULL;
 			file_data_unref(fd);
 			g_free(id);
 			vf->list = g_list_delete_link(vf->list, to_delete);
@@ -2318,7 +2318,7 @@
 						"cell-background-set", TRUE,
 						"foreground-gdk", &color_fg,
 						"foreground-set", TRUE,
-						"has-focus", (VFICON_INFO(vf, focus_id) == id), NULL);
+						"has-focus", (VFICON(vf)->focus_id == id), NULL);
 			if (sidecars)
 				{
 				g_free(sidecars);
@@ -2384,8 +2384,8 @@
 	file_data_unref(vf->dir_fd);
 	vf->dir_fd = file_data_ref(dir_fd);
 
-	g_list_free(VFICON_INFO(vf, selection));
-	VFICON_INFO(vf, selection) = NULL;
+	g_list_free(VFICON(vf)->selection);
+	VFICON(vf)->selection = NULL;
 
 	iconlist_free(vf->list);
 	vf->list = NULL;
@@ -2393,7 +2393,7 @@
 	/* NOTE: populate will clear the store for us */
 	ret = vficon_refresh_real(vf, FALSE);
 
-	VFICON_INFO(vf, focus_id) = NULL;
+	VFICON(vf)->focus_id = NULL;
 	vficon_move_focus(vf, 0, 0, FALSE);
 
 	return ret;
@@ -2412,7 +2412,7 @@
 	vficon_thumb_cleanup(vf);
 
 	iconlist_free(vf->list);
-	g_list_free(VFICON_INFO(vf, selection));
+	g_list_free(VFICON(vf)->selection);
 }
 
 ViewFile *vficon_new(ViewFile *vf, FileData *dir_fd)
@@ -2423,17 +2423,17 @@
 
 	vf->info = g_new0(ViewFileInfoIcon, 1);
 
-	VFICON_INFO(vf, selection) = NULL;
-	VFICON_INFO(vf, prev_selection) = NULL;
-
-	VFICON_INFO(vf, tip_window) = NULL;
-	VFICON_INFO(vf, tip_delay_id) = -1;
-
-	VFICON_INFO(vf, focus_row) = 0;
-	VFICON_INFO(vf, focus_column) = 0;
-	VFICON_INFO(vf, focus_id) = NULL;
-
-	VFICON_INFO(vf, show_text) = options->show_icon_names;
+	VFICON(vf)->selection = NULL;
+	VFICON(vf)->prev_selection = NULL;
+
+	VFICON(vf)->tip_window = NULL;
+	VFICON(vf)->tip_delay_id = -1;
+
+	VFICON(vf)->focus_row = 0;
+	VFICON(vf)->focus_column = 0;
+	VFICON(vf)->focus_id = NULL;
+
+	VFICON(vf)->show_text = options->show_icon_names;
 
 	store = gtk_list_store_new(1, G_TYPE_POINTER);
 	vf->listview = gtk_tree_view_new_with_model(GTK_TREE_MODEL(store));
@@ -2466,7 +2466,7 @@
 	g_signal_connect(G_OBJECT(vf->listview), "leave_notify_event",
 			 G_CALLBACK(vficon_leave_cb), vf);
 
-	/* force VFICON_INFO(vf, columns) to be at least 1 (sane) - this will be corrected in the size_cb */
+	/* force VFICON(vf)->columns to be at least 1 (sane) - this will be corrected in the size_cb */
 	vficon_populate_at_new_size(vf, 1, 1, FALSE);
 
 	file_data_register_notify_func(vf_notify_cb, vf, NOTIFY_PRIORITY_MEDIUM);
--- a/src/view_file_list.c	Sun Aug 17 09:00:33 2008 +0000
+++ b/src/view_file_list.c	Sun Aug 17 09:30:45 2008 +0000
@@ -230,15 +230,15 @@
 	gchar *uri_text = NULL;
 	gint total;
 
-	if (!VFLIST_INFO(vf, click_fd)) return;
+	if (!VFLIST(vf)->click_fd) return;
 
-	if (vflist_row_is_selected(vf, VFLIST_INFO(vf, click_fd)))
+	if (vflist_row_is_selected(vf, VFLIST(vf)->click_fd))
 		{
 		list = vf_selection_get_list(vf);
 		}
 	else
 		{
-		list = g_list_append(NULL, file_data_ref(VFLIST_INFO(vf, click_fd)));
+		list = g_list_append(NULL, file_data_ref(VFLIST(vf)->click_fd));
 		}
 
 	if (!list) return;
@@ -257,19 +257,19 @@
 {
 	ViewFile *vf = data;
 
-	vflist_color_set(vf, VFLIST_INFO(vf, click_fd), TRUE);
+	vflist_color_set(vf, VFLIST(vf)->click_fd, TRUE);
 
-	if (VFLIST_INFO(vf, thumbs_enabled) &&
-	    VFLIST_INFO(vf, click_fd) && VFLIST_INFO(vf, click_fd)->thumb_pixbuf)
+	if (VFLIST(vf)->thumbs_enabled &&
+	    VFLIST(vf)->click_fd && VFLIST(vf)->click_fd->thumb_pixbuf)
 		{
 		guint items;
 
-		if (vflist_row_is_selected(vf, VFLIST_INFO(vf, click_fd)))
+		if (vflist_row_is_selected(vf, VFLIST(vf)->click_fd))
 			items = vf_selection_count(vf, NULL);
 		else
 			items = 1;
 
-		dnd_set_drag_icon(widget, context, VFLIST_INFO(vf, click_fd)->thumb_pixbuf, items);
+		dnd_set_drag_icon(widget, context, VFLIST(vf)->click_fd->thumb_pixbuf, items);
 		}
 }
 
@@ -277,7 +277,7 @@
 {
 	ViewFile *vf = data;
 
-	vflist_color_set(vf, VFLIST_INFO(vf, click_fd), FALSE);
+	vflist_color_set(vf, VFLIST(vf)->click_fd, FALSE);
 
 	if (context->action == GDK_ACTION_MOVE)
 		{
@@ -306,21 +306,21 @@
 
 GList *vflist_pop_menu_file_list(ViewFile *vf)
 {
-	if (!VFLIST_INFO(vf, click_fd)) return NULL;
+	if (!VFLIST(vf)->click_fd) return NULL;
 
-	if (vflist_row_is_selected(vf, VFLIST_INFO(vf, click_fd)))
+	if (vflist_row_is_selected(vf, VFLIST(vf)->click_fd))
 		{
 		return vf_selection_get_list(vf);
 		}
 
-	return g_list_append(NULL, file_data_ref(VFLIST_INFO(vf, click_fd)));
+	return g_list_append(NULL, file_data_ref(VFLIST(vf)->click_fd));
 }
 
 void vflist_pop_menu_view_cb(GtkWidget *widget, gpointer data)
 {
 	ViewFile *vf = data;
 
-	if (vflist_row_is_selected(vf, VFLIST_INFO(vf, click_fd)))
+	if (vflist_row_is_selected(vf, VFLIST(vf)->click_fd))
 		{
 		GList *list;
 
@@ -330,7 +330,7 @@
 		}
 	else
 		{
-		view_window_new(VFLIST_INFO(vf, click_fd));
+		view_window_new(VFLIST(vf)->click_fd);
 		}
 }
 
@@ -341,7 +341,7 @@
 
 	list = vf_pop_menu_file_list(vf);
 	if (options->file_ops.enable_in_place_rename &&
-	    list && !list->next && VFLIST_INFO(vf, click_fd))
+	    list && !list->next && VFLIST(vf)->click_fd)
 		{
 		GtkTreeModel *store;
 		GtkTreeIter iter;
@@ -349,13 +349,13 @@
 		filelist_free(list);
 
 		store = gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview));
-		if (vflist_find_row(vf, VFLIST_INFO(vf, click_fd), &iter) >= 0)
+		if (vflist_find_row(vf, VFLIST(vf)->click_fd, &iter) >= 0)
 			{
 			GtkTreePath *tpath;
 
 			tpath = gtk_tree_model_get_path(store, &iter);
 			tree_edit_by_path(GTK_TREE_VIEW(vf->listview), tpath,
-					  vflist_column_idx(vf, FILE_COLUMN_NAME), VFLIST_INFO(vf, click_fd)->name,
+					  vflist_column_idx(vf, FILE_COLUMN_NAME), VFLIST(vf)->click_fd->name,
 					  vflist_row_rename_cb, vf);
 			gtk_tree_path_free(tpath);
 			}
@@ -369,14 +369,14 @@
 {
 	ViewFile *vf = data;
 
-	vflist_color_set(vf, VFLIST_INFO(vf, click_fd), FALSE);
+	vflist_color_set(vf, VFLIST(vf)->click_fd, FALSE);
 	if (vf->layout)
 		{
-		layout_thumb_set(vf->layout, !VFLIST_INFO(vf, thumbs_enabled));
+		layout_thumb_set(vf->layout, !VFLIST(vf)->thumbs_enabled);
 		}
 	else
 		{
-		vflist_thumb_set(vf, !VFLIST_INFO(vf, thumbs_enabled));
+		vflist_thumb_set(vf, !VFLIST(vf)->thumbs_enabled);
 		}
 }
 
@@ -384,15 +384,15 @@
 {
 	ViewFile *vf = data;
 
-	vflist_color_set(vf, VFLIST_INFO(vf, click_fd), FALSE);
+	vflist_color_set(vf, VFLIST(vf)->click_fd, FALSE);
 	vf_refresh(vf);
 }
 
 void vflist_popup_destroy_cb(GtkWidget *widget, gpointer data)
 {
 	ViewFile *vf = data;
-	vflist_color_set(vf, VFLIST_INFO(vf, click_fd), FALSE);
-	VFLIST_INFO(vf, click_fd) = NULL;
+	vflist_color_set(vf, VFLIST(vf)->click_fd, FALSE);
+	VFLIST(vf)->click_fd = NULL;
 	vf->popup = NULL;
 }
 
@@ -440,7 +440,7 @@
 	GtkTreePath *tpath;
 	gint cw, ch;
 
-	if (vflist_find_row(vf, VFLIST_INFO(vf, click_fd), &iter) < 0) return;
+	if (vflist_find_row(vf, VFLIST(vf)->click_fd, &iter) < 0) return;
 	store = gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview));
 	tpath = gtk_tree_model_get_path(store, &iter);
 	tree_view_get_cell_clamped(GTK_TREE_VIEW(vf->listview), tpath, FILE_COLUMN_NAME - 1, TRUE, x, y, &cw, &ch);
@@ -464,12 +464,12 @@
 
 		store = gtk_tree_view_get_model(GTK_TREE_VIEW(widget));
 		gtk_tree_model_get_iter(store, &iter, tpath);
-		gtk_tree_model_get(store, &iter, FILE_COLUMN_POINTER, &VFLIST_INFO(vf, click_fd), -1);
+		gtk_tree_model_get(store, &iter, FILE_COLUMN_POINTER, &VFLIST(vf)->click_fd, -1);
 		gtk_tree_path_free(tpath);
 		}
 	else
 		{
-		VFLIST_INFO(vf, click_fd) = NULL;
+		VFLIST(vf)->click_fd = NULL;
 		}
 
 	vf->popup = vf_pop_menu(vf);
@@ -511,7 +511,7 @@
 		gtk_tree_path_free(tpath);
 		}
 
-	VFLIST_INFO(vf, click_fd) = fd;
+	VFLIST(vf)->click_fd = fd;
 
 	if (bevent->button == MOUSE_BUTTON_RIGHT)
 		{
@@ -570,7 +570,7 @@
 
 	if (bevent->button == MOUSE_BUTTON_MIDDLE)
 		{
-		vflist_color_set(vf, VFLIST_INFO(vf, click_fd), FALSE);
+		vflist_color_set(vf, VFLIST(vf)->click_fd, FALSE);
 		}
 
 	if (bevent->button != MOUSE_BUTTON_LEFT && bevent->button != MOUSE_BUTTON_MIDDLE)
@@ -592,7 +592,7 @@
 
 	if (bevent->button == MOUSE_BUTTON_MIDDLE)
 		{
-		if (fd && VFLIST_INFO(vf, click_fd) == fd)
+		if (fd && VFLIST(vf)->click_fd == fd)
 			{
 			GtkTreeSelection *selection;
 
@@ -609,7 +609,7 @@
 		return TRUE;
 		}
 
-	if (fd && VFLIST_INFO(vf, click_fd) == fd &&
+	if (fd && VFLIST(vf)->click_fd == fd &&
 	    !(bevent->state & GDK_SHIFT_MASK ) &&
 	    !(bevent->state & GDK_CONTROL_MASK ) &&
 	    vflist_row_is_selected(vf, fd))
@@ -661,26 +661,26 @@
 
 	if (!vf->layout)
 		{
-		VFLIST_INFO(vf, select_idle_id) = -1;
+		VFLIST(vf)->select_idle_id = -1;
 		return FALSE;
 		}
 
 	vf_send_update(vf);
 
-	if (VFLIST_INFO(vf, select_fd))
+	if (VFLIST(vf)->select_fd)
 		{
-		vflist_select_image(vf, VFLIST_INFO(vf, select_fd));
-		VFLIST_INFO(vf, select_fd) = NULL;
+		vflist_select_image(vf, VFLIST(vf)->select_fd);
+		VFLIST(vf)->select_fd = NULL;
 		}
 
-	VFLIST_INFO(vf, select_idle_id) = -1;
+	VFLIST(vf)->select_idle_id = -1;
 	return FALSE;
 }
 
 static void vflist_select_idle_cancel(ViewFile *vf)
 {
-	if (VFLIST_INFO(vf, select_idle_id) != -1) g_source_remove(VFLIST_INFO(vf, select_idle_id));
-	VFLIST_INFO(vf, select_idle_id) = -1;
+	if (VFLIST(vf)->select_idle_id != -1) g_source_remove(VFLIST(vf)->select_idle_id);
+	VFLIST(vf)->select_idle_id = -1;
 }
 
 static gboolean vflist_select_cb(GtkTreeSelection *selection, GtkTreeModel *store, GtkTreePath *tpath,
@@ -692,17 +692,17 @@
 	if (!path_currently_selected &&
 	    gtk_tree_model_get_iter(store, &iter, tpath))
 		{
-		gtk_tree_model_get(store, &iter, FILE_COLUMN_POINTER, &VFLIST_INFO(vf, select_fd), -1);
+		gtk_tree_model_get(store, &iter, FILE_COLUMN_POINTER, &VFLIST(vf)->select_fd, -1);
 		}
 	else
 		{
-		VFLIST_INFO(vf, select_fd) = NULL;
+		VFLIST(vf)->select_fd = NULL;
 		}
 
 	if (vf->layout &&
-	    VFLIST_INFO(vf, select_idle_id) == -1)
+	    VFLIST(vf)->select_idle_id == -1)
 		{
-		VFLIST_INFO(vf, select_idle_id) = g_idle_add(vflist_select_idle_cb, vf);
+		VFLIST(vf)->select_idle_id = g_idle_add(vflist_select_idle_cb, vf);
 		}
 
 	return TRUE;
@@ -1139,7 +1139,7 @@
 static void vflist_thumb_update(ViewFile *vf)
 {
 	vflist_thumb_stop(vf);
-	if (!VFLIST_INFO(vf, thumbs_enabled)) return;
+	if (!VFLIST(vf)->thumbs_enabled) return;
 
 	vflist_thumb_status(vf, 0.0, _("Loading thumbs..."));
 	vf->thumbs_running = TRUE;
@@ -1364,7 +1364,7 @@
 	selection = gtk_tree_view_get_selection(GTK_TREE_VIEW(vf->listview));
 	gtk_tree_selection_select_all(selection);
 
-	VFLIST_INFO(vf, select_fd) = NULL;
+	VFLIST(vf)->select_fd = NULL;
 }
 
 void vflist_select_none(ViewFile *vf)
@@ -1638,7 +1638,7 @@
 	GList *selected;
 
 	store = GTK_TREE_STORE(gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview)));
-	thumbs = VFLIST_INFO(vf, thumbs_enabled);
+	thumbs = VFLIST(vf)->thumbs_enabled;
 
 	vflist_thumb_stop(vf);
 
@@ -1907,11 +1907,11 @@
 
 	vf->info = g_new0(ViewFileInfoList, 1);
 	
-	VFLIST_INFO(vf, click_fd) = NULL;
-	VFLIST_INFO(vf, select_fd) = NULL;
-	VFLIST_INFO(vf, thumbs_enabled) = FALSE;
+	VFLIST(vf)->click_fd = NULL;
+	VFLIST(vf)->select_fd = NULL;
+	VFLIST(vf)->thumbs_enabled = FALSE;
 
-	VFLIST_INFO(vf, select_idle_id) = -1;
+	VFLIST(vf)->select_idle_id = -1;
 
 	flist_types[FILE_COLUMN_POINTER] = G_TYPE_POINTER;
 	flist_types[FILE_COLUMN_VERSION] = G_TYPE_INT;
@@ -1971,9 +1971,9 @@
 
 void vflist_thumb_set(ViewFile *vf, gint enable)
 {
-	if (VFLIST_INFO(vf, thumbs_enabled) == enable) return;
+	if (VFLIST(vf)->thumbs_enabled == enable) return;
 
-	VFLIST_INFO(vf, thumbs_enabled) = enable;
+	VFLIST(vf)->thumbs_enabled = enable;
 	if (vf->layout) vf_refresh(vf);
 }