libcaja-private: reduce the scope of some variables

author: Pablo Barciela <[email protected]> 2019-05-02 01:09:20 +0200
committer: raveit65 <[email protected]> 2019-05-06 13:53:37 +0200
commit: a110c542bd7a16625f03c97e2fefa3be8da50c85 (patch)
tree: 7af078f9084b33eb0d5df7f3657a1f8049ff6b83 /libcaja-private
parent: 476f56a25be636970b336d525a7766b6d1eb3fff (diff)
download: caja-a110c542bd7a16625f03c97e2fefa3be8da50c85.tar.bz2
caja-a110c542bd7a16625f03c97e2fefa3be8da50c85.tar.xz
44 files changed, 794 insertions, 476 deletions
diff --git a/libcaja-private/caja-autorun.c b/libcaja-private/caja-autorun.c
index 04033445..15a4f394 100644
--- a/libcaja-private/caja-autorun.c
+++ b/libcaja-private/caja-autorun.c
@@ -475,7 +475,6 @@ caja_autorun_prepare_combo_box (GtkWidget *combo_box,
 cairo_surface_t *surface;
 int icon_size, icon_scale;
 int set_active;
- int n;
 int num_apps;
 gboolean pref_ask;
 gboolean pref_start_app;
@@ -594,6 +593,8 @@ caja_autorun_prepare_combo_box (GtkWidget *combo_box,
 COLUMN_AUTORUN_ITEM_TYPE, AUTORUN_SEP,
 -1);
 
+ int n;
+
 for (l = app_info_list, n = include_ask ? 4 : 3; l != NULL; l = l->next, n++)
 {
 GIcon *icon;
diff --git a/libcaja-private/caja-bookmark.c b/libcaja-private/caja-bookmark.c
index ea0eb253..d7e72fb4 100644
--- a/libcaja-private/caja-bookmark.c
+++ b/libcaja-private/caja-bookmark.c
@@ -467,8 +467,7 @@ bookmark_file_changed_callback (CajaFile *file, CajaBookmark *bookmark)
 static void
 caja_bookmark_set_icon_to_default (CajaBookmark *bookmark)
 {
- GIcon *icon, *emblemed_icon, *folder;
- GEmblem *emblem;
+ GIcon *emblemed_icon, *folder;
 
 if (bookmark->details->icon)
 {
@@ -479,6 +478,9 @@ caja_bookmark_set_icon_to_default (CajaBookmark *bookmark)
 
 if (caja_bookmark_uri_known_not_to_exist (bookmark))
 {
+ GIcon *icon;
+ GEmblem *emblem;
+
 icon = g_themed_icon_new ("dialog-warning");
 emblem = g_emblem_new (icon);
 
@@ -644,13 +646,14 @@ bookmark_image_menu_item_new_from_surface (cairo_surface_t *icon_surface,
 GtkWidget *
 caja_bookmark_menu_item_new (CajaBookmark *bookmark)
 {
- GtkWidget *menu_item;
 cairo_surface_t *image_cairo;
 
 image_cairo = create_image_cairo_for_bookmark (bookmark);
 
 if (strlen (bookmark->details->name) > 0)
 {
+ GtkWidget *menu_item;
+
 menu_item = bookmark_image_menu_item_new_from_surface (image_cairo, bookmark->details->name);
 
 return menu_item;
diff --git a/libcaja-private/caja-clipboard-monitor.c b/libcaja-private/caja-clipboard-monitor.c
index ce8069f7..73c1fd08 100644
--- a/libcaja-private/caja-clipboard-monitor.c
+++ b/libcaja-private/caja-clipboard-monitor.c
@@ -78,10 +78,10 @@ destroy_clipboard_monitor (void)
 CajaClipboardMonitor *
 caja_clipboard_monitor_get (void)
 {
- GtkClipboard *clipboard;
-
 if (clipboard_monitor == NULL)
 {
+ GtkClipboard *clipboard;
+
 clipboard_monitor = CAJA_CLIPBOARD_MONITOR (g_object_new (CAJA_TYPE_CLIPBOARD_MONITOR, NULL));
 eel_debug_call_at_shutdown (destroy_clipboard_monitor);
 
@@ -227,7 +227,7 @@ convert_file_list_to_string (CajaClipboardInfo *info,
 gsize *len)
 {
 GString *uris;
- char *uri, *tmp;
+ char *tmp;
 GFile *f;
 guint i;
 GList *l;
@@ -243,6 +243,8 @@ convert_file_list_to_string (CajaClipboardInfo *info,
 
 for (i = 0, l = info->files; l != NULL; l = l->next, i++)
 {
+ char *uri;
+
 uri = caja_file_get_uri (l->data);
 
 if (format_for_text)
@@ -286,9 +288,7 @@ caja_get_clipboard_callback (GtkClipboard *clipboard,
 guint info,
 gpointer user_data)
 {
- char **uris;
 GList *l;
- int i;
 CajaClipboardInfo *clipboard_info;
 GdkAtom target;
 
@@ -299,6 +299,9 @@ caja_get_clipboard_callback (GtkClipboard *clipboard,
 
 if (gtk_targets_include_uri (&target, 1))
 {
+ char **uris;
+ int i;
+
 uris = g_malloc ((g_list_length (clipboard_info->files) + 1) * sizeof (char *));
 i = 0;
 
diff --git a/libcaja-private/caja-clipboard.c b/libcaja-private/caja-clipboard.c
index 88f2a808..bc876a28 100644
--- a/libcaja-private/caja-clipboard.c
+++ b/libcaja-private/caja-clipboard.c
@@ -162,7 +162,6 @@ received_clipboard_contents (GtkClipboard *clipboard,
 static void
 set_paste_sensitive_if_clipboard_contains_data (GtkActionGroup *action_group)
 {
- GtkAction *action;
 if (gdk_display_supports_selection_notification (gdk_display_get_default ()))
 {
 gtk_clipboard_request_contents (gtk_clipboard_get (GDK_SELECTION_CLIPBOARD),
@@ -172,6 +171,8 @@ set_paste_sensitive_if_clipboard_contains_data (GtkActionGroup *action_group)
 }
 else
 {
+ GtkAction *action;
+
 /* If selection notification isn't supported, always activate Paste */
 action = gtk_action_group_get_action (action_group,
 "Paste");
@@ -614,7 +615,6 @@ caja_clipboard_get_uri_list_from_selection_data (GtkSelectionData *selection_dat
 GdkAtom copied_files_atom)
 {
 GList *items;
- char **lines;
 
 if (gtk_selection_data_get_data_type (selection_data) != copied_files_atom
 || gtk_selection_data_get_length (selection_data) <= 0)
@@ -623,6 +623,7 @@ caja_clipboard_get_uri_list_from_selection_data (GtkSelectionData *selection_dat
 }
 else
 {
+ char **lines;
 guchar *data;
 /* Not sure why it's legal to assume there's an extra byte
 * past the end of the selection data that it's safe to write
diff --git a/libcaja-private/caja-customization-data.c b/libcaja-private/caja-customization-data.c
index de516d5a..ebd0b029 100644
--- a/libcaja-private/caja-customization-data.c
+++ b/libcaja-private/caja-customization-data.c
@@ -124,8 +124,7 @@ caja_customization_data_new (const char *customization_name,
 int maximum_icon_width)
 {
 CajaCustomizationData *data;
- char *public_directory_path, *private_directory_path;
- char *temp_str;
+ char *private_directory_path;
 gboolean public_result, private_result;
 
 data = g_new0 (CajaCustomizationData, 1);
@@ -134,6 +133,8 @@ caja_customization_data_new (const char *customization_name,
 
 if (show_public_customizations)
 {
+ char *public_directory_path;
+
 public_directory_path = get_global_customization_path (customization_name);
 
 public_result = read_all_children (public_directory_path,
@@ -171,6 +172,8 @@ caja_customization_data_new (const char *customization_name,
 /* load the frame if necessary */
 if (strcmp (customization_name, "patterns") == 0)
 {
+ char *temp_str;
+
 temp_str = caja_pixmap_file ("chit_frame.png");
 if (temp_str != NULL)
 {
@@ -425,7 +428,7 @@ caja_customization_make_pattern_chit (GdkPixbuf *pattern_tile, GdkPixbuf *frame,
 static char*
 format_name_for_display (CajaCustomizationData *data, const char* name)
 {
- char *formatted_str, *mapped_name;
+ char *formatted_str;
 
 if (!g_strcmp0(name, RESET_IMAGE_NAME))
 {
@@ -437,6 +440,8 @@ format_name_for_display (CajaCustomizationData *data, const char* name)
 formatted_str = eel_filename_strip_extension (name);
 if (data->name_map_hash != NULL)
 {
+ char *mapped_name;
+
 mapped_name = g_hash_table_lookup (data->name_map_hash, formatted_str);
 if (mapped_name)
 {
@@ -455,7 +460,6 @@ static void
 load_name_map_hash_table (CajaCustomizationData *data)
 {
 char *xml_path;
- char *filename, *display_name;
 
 xmlDocPtr browser_data;
 xmlNodePtr category_node, current_node;
@@ -479,6 +483,8 @@ load_name_map_hash_table (CajaCustomizationData *data)
 /* loop through the entries, adding a mapping to the hash table */
 while (current_node != NULL)
 {
+ char *filename, *display_name;
+
 display_name = eel_xml_get_property_translated (current_node, "display_name");
 filename = xmlGetProp (current_node, "filename");
 if (display_name && filename)
diff --git a/libcaja-private/caja-debug-log.c b/libcaja-private/caja-debug-log.c
index 19462d34..646e7f66 100644
--- a/libcaja-private/caja-debug-log.c
+++ b/libcaja-private/caja-debug-log.c
@@ -396,8 +396,6 @@ caja_debug_log_enable_domains (const char **domains, int n_domains)
 void
 caja_debug_log_disable_domains (const char **domains, int n_domains)
 {
- int i;
-
 g_assert (domains != NULL);
 g_assert (n_domains >= 0);
 
@@ -405,6 +403,8 @@ caja_debug_log_disable_domains (const char **domains, int n_domains)
 
 if (domains_hash)
 {
+ int i;
+
 for (i = 0; i < n_domains; i++)
 {
 char *domain;
@@ -464,7 +464,6 @@ make_key_file_from_configuration (void)
 {
 GKeyFile *key_file;
 struct domains_dump_closure closure;
- int num_domains;
 
 key_file = g_key_file_new ();
 
@@ -472,6 +471,8 @@ make_key_file_from_configuration (void)
 
 if (domains_hash)
 {
+ int num_domains;
+
 num_domains = g_hash_table_size (domains_hash);
 if (num_domains != 0)
 {
diff --git a/libcaja-private/caja-desktop-icon-file.c b/libcaja-private/caja-desktop-icon-file.c
index c9eccc91..082fdca3 100644
--- a/libcaja-private/caja-desktop-icon-file.c
+++ b/libcaja-private/caja-desktop-icon-file.c
@@ -329,11 +329,12 @@ caja_desktop_icon_file_unmount (CajaFile *file,
 gpointer callback_data)
 {
 CajaDesktopIconFile *desktop_file;
- GMount *mount;
 
 desktop_file = CAJA_DESKTOP_ICON_FILE (file);
 if (desktop_file)
 {
+ GMount *mount;
+
 mount = caja_desktop_link_get_mount (desktop_file->details->link);
 if (mount != NULL)
 {
@@ -351,11 +352,12 @@ caja_desktop_icon_file_eject (CajaFile *file,
 gpointer callback_data)
 {
 CajaDesktopIconFile *desktop_file;
- GMount *mount;
 
 desktop_file = CAJA_DESKTOP_ICON_FILE (file);
 if (desktop_file)
 {
+ GMount *mount;
+
 mount = caja_desktop_link_get_mount (desktop_file->details->link);
 if (mount != NULL)
 {
diff --git a/libcaja-private/caja-desktop-link-monitor.c b/libcaja-private/caja-desktop-link-monitor.c
index a427ba5c..0d5da061 100644
--- a/libcaja-private/caja-desktop-link-monitor.c
+++ b/libcaja-private/caja-desktop-link-monitor.c
@@ -95,11 +95,12 @@ volume_file_name_used (CajaDesktopLinkMonitor *monitor,
 const char *name)
 {
 GList *l;
- char *other_name;
 gboolean same;
 
 for (l = monitor->details->mount_links; l != NULL; l = l->next)
 {
+ char *other_name;
+
 other_name = caja_desktop_link_get_file_name (l->data);
 same = strcmp (name, other_name) == 0;
 g_free (other_name);
@@ -135,8 +136,8 @@ has_mount (CajaDesktopLinkMonitor *monitor,
 GMount *mount)
 {
 gboolean ret;
- GMount *other_mount;
 GList *l;
+ GMount *other_mount = NULL;
 
 ret = FALSE;
 
@@ -159,14 +160,14 @@ static void
 create_mount_link (CajaDesktopLinkMonitor *monitor,
 GMount *mount)
 {
- CajaDesktopLink *link;
-
 if (has_mount (monitor, mount))
 return;
 
 if ((!g_mount_is_shadowed (mount)) &&
 g_settings_get_boolean (caja_desktop_preferences, CAJA_PREFERENCES_DESKTOP_VOLUMES_VISIBLE))
 {
+ CajaDesktopLink *link;
+
 link = caja_desktop_link_new_from_mount (mount);
 monitor->details->mount_links = g_list_prepend (monitor->details->mount_links, link);
 }
@@ -178,7 +179,7 @@ remove_mount_link (CajaDesktopLinkMonitor *monitor,
 {
 GList *l;
 CajaDesktopLink *link;
- GMount *other_mount;
+ GMount *other_mount = NULL;
 
 link = NULL;
 for (l = monitor->details->mount_links; l != NULL; l = l->next)
@@ -363,7 +364,7 @@ caja_desktop_link_monitor_init (gpointer object, gpointer klass)
 {
 CajaDesktopLinkMonitor *monitor;
 GList *l, *mounts;
- GMount *mount;
+ GMount *mount = NULL;
 
 monitor = CAJA_DESKTOP_LINK_MONITOR (object);
 
diff --git a/libcaja-private/caja-desktop-metadata.c b/libcaja-private/caja-desktop-metadata.c
index c9e136bc..41f0b021 100644
--- a/libcaja-private/caja-desktop-metadata.c
+++ b/libcaja-private/caja-desktop-metadata.c
@@ -235,8 +235,7 @@ caja_desktop_update_metadata_from_keyfile (CajaFile *file,
 {
 gchar **keys, **values;
 const gchar *actual_values[2];
- const gchar *key, *value;
- gchar *gio_key;
+ const gchar *value;
 gsize length, values_length;
 GKeyFile *keyfile;
 GFileInfo *info;
@@ -257,6 +256,9 @@ caja_desktop_update_metadata_from_keyfile (CajaFile *file,
 info = g_file_info_new ();
 
 for (idx = 0; idx < length; idx++) {
+ const gchar *key;
+ gchar *gio_key;
+
 key = keys[idx];
 values = g_key_file_get_string_list (keyfile,
 name,
diff --git a/libcaja-private/caja-directory-async.c b/libcaja-private/caja-directory-async.c
index 3915adcb..68d6edee 100644
--- a/libcaja-private/caja-directory-async.c
+++ b/libcaja-private/caja-directory-async.c
@@ -623,10 +623,11 @@ static void
 new_files_cancel (CajaDirectory *directory)
 {
 GList *l;
- NewFilesState *state;
 
 if (directory->details->new_files_in_progress != NULL)
 {
+ NewFilesState *state = NULL;
+
 for (l = directory->details->new_files_in_progress; l != NULL; l = l->next)
 {
 state = l->data;
@@ -688,10 +689,10 @@ static void
 remove_monitor_link (CajaDirectory *directory,
 GList *link)
 {
- Monitor *monitor;
-
 if (link != NULL)
 {
+ Monitor *monitor;
+
 monitor = link->data;
 request_counter_remove_request (directory->details->monitor_counters,
 monitor->request);
@@ -804,7 +805,6 @@ caja_directory_monitor_add_internal (CajaDirectory *directory,
 gpointer callback_data)
 {
 Monitor *monitor;
- GList *file_list;
 
 g_assert (CAJA_IS_DIRECTORY (directory));
 
@@ -829,6 +829,8 @@ caja_directory_monitor_add_internal (CajaDirectory *directory,
 
 if (callback != NULL)
 {
+ GList *file_list;
+
 file_list = caja_directory_get_file_list (directory);
 (* callback) (directory, file_list, callback_data);
 caja_file_list_free (file_list);
@@ -1128,12 +1130,13 @@ directory_load_one (CajaDirectory *directory,
 static void
 directory_load_cancel (CajaDirectory *directory)
 {
- CajaFile *file;
 DirectoryLoadState *state;
 
 state = directory->details->directory_load_in_progress;
 if (state != NULL)
 {
+ CajaFile *file;
+
 file = state->load_directory_file;
 file->details->loading_directory = FALSE;
 if (file->details->directory != directory)
@@ -1230,7 +1233,7 @@ caja_directory_remove_file_monitors (CajaDirectory *directory,
 CajaFile *file)
 {
 GList *result, **list, *node, *next;
- Monitor *monitor;
+ Monitor *monitor = NULL;
 
 g_assert (CAJA_IS_DIRECTORY (directory));
 g_assert (CAJA_IS_FILE (file));
@@ -1267,7 +1270,7 @@ caja_directory_add_file_monitors (CajaDirectory *directory,
 {
 GList **list;
 GList *l;
- Monitor *monitor;
+ Monitor *monitor = NULL;
 
 g_assert (CAJA_IS_DIRECTORY (directory));
 g_assert (CAJA_IS_FILE (file));
@@ -1618,8 +1621,8 @@ caja_directory_get_info_for_new_files (CajaDirectory *directory,
 GList *location_list)
 {
 NewFilesState *state;
- GFile *location;
 GList *l;
+ GFile *location = NULL;
 
 if (location_list == NULL)
 {
@@ -1656,8 +1659,8 @@ caja_async_destroying_file (CajaFile *file)
 CajaDirectory *directory;
 gboolean changed;
 GList *node, *next;
- ReadyCallback *callback;
- Monitor *monitor;
+ ReadyCallback *callback = NULL;
+ Monitor *monitor = NULL;
 
 directory = file->details->directory;
 changed = FALSE;
@@ -2064,7 +2067,7 @@ call_ready_callbacks (CajaDirectory *directory)
 {
 gboolean found_any;
 GList *node, *next;
- ReadyCallback *callback;
+ ReadyCallback *callback = NULL;
 
 found_any = FALSE;
 
@@ -2095,8 +2098,8 @@ caja_directory_has_active_request_for_file (CajaDirectory *directory,
 CajaFile *file)
 {
 GList *node;
- ReadyCallback *callback;
- Monitor *monitor;
+ ReadyCallback *callback = NULL;
+ Monitor *monitor = NULL;
 
 for (node = directory->details->call_when_ready_list;
 node != NULL; node = node->next)
@@ -2152,7 +2155,7 @@ static void
 mark_all_files_unconfirmed (CajaDirectory *directory)
 {
 GList *node;
- CajaFile *file;
+ CajaFile *file = NULL;
 
 for (node = directory->details->file_list; node != NULL; node = node->next)
 {
@@ -2192,7 +2195,7 @@ more_files_callback (GObject *source_object,
 CajaDirectory *directory;
 GError *error;
 GList *files, *l;
- GFileInfo *info;
+ GFileInfo *info = NULL;
 
 state = user_data;
 
@@ -2467,8 +2470,6 @@ is_needy (CajaFile *file,
 {
 CajaDirectory *directory;
 GList *node;
- ReadyCallback *callback;
- Monitor *monitor;
 
 if (!(* check_missing) (file))
 {
@@ -2478,6 +2479,8 @@ is_needy (CajaFile *file,
 directory = file->details->directory;
 if (directory->details->call_when_ready_counters[request_type_wanted] > 0)
 {
+ ReadyCallback *callback = NULL;
+
 for (node = directory->details->call_when_ready_list;
 node != NULL; node = node->next)
 {
@@ -2500,6 +2503,8 @@ is_needy (CajaFile *file,
 
 if (directory->details->monitor_counters[request_type_wanted] > 0)
 {
+ Monitor *monitor = NULL;
+
 for (node = directory->details->monitor_list;
 node != NULL; node = node->next)
 {
@@ -2519,10 +2524,10 @@ is_needy (CajaFile *file,
 static void
 directory_count_stop (CajaDirectory *directory)
 {
- CajaFile *file;
-
 if (directory->details->count_in_progress != NULL)
 {
+ CajaFile *file;
+
 file = directory->details->count_in_progress->count_file;
 if (file != NULL)
 {
@@ -2546,7 +2551,7 @@ count_non_skipped_files (GList *list)
 {
 guint count;
 GList *node;
- GFileInfo *info;
+ GFileInfo *info = NULL;
 
 count = 0;
 for (node = list; node != NULL; node = node->next)
@@ -2678,13 +2683,14 @@ count_children_callback (GObject *source_object,
 {
 DirectoryCountState *state;
 GFileEnumerator *enumerator;
- CajaDirectory *directory;
 GError *error;
 
 state = user_data;
 
 if (g_cancellable_is_cancelled (state->cancellable))
 {
+ CajaDirectory *directory;
+
 /* Operation was cancelled. Bail out */
 directory = state->directory;
 directory->details->count_in_progress = NULL;
@@ -2831,9 +2837,7 @@ deep_count_one (DeepCountState *state,
 GFileInfo *info)
 {
 CajaFile *file;
- GFile *subdir;
 gboolean is_seen_inode;
- const char *fs_id;
 
 if (should_skip_file (NULL, info))
 {
@@ -2850,6 +2854,8 @@ deep_count_one (DeepCountState *state,
 
 if (g_file_info_get_file_type (info) == G_FILE_TYPE_DIRECTORY)
 {
+ const char *fs_id;
+
 /* Count the directory. */
 file->details->deep_directory_count += 1;
 
@@ -2858,6 +2864,8 @@ deep_count_one (DeepCountState *state,
 fs_id = g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_ID_FILESYSTEM);
 if (g_strcmp0 (fs_id, state->fs_id) == 0)
 {
+ GFile *subdir;
+
 /* only if it is on the same filesystem */
 subdir = g_file_get_child (state->deep_count_location, g_file_info_get_name (info));
 state->deep_count_subdirectories = g_list_prepend (state->deep_count_subdirectories, subdir);
@@ -2908,7 +2916,6 @@ deep_count_state_free (DeepCountState *state)
 static void
 deep_count_next_dir (DeepCountState *state)
 {
- GFile *location;
 CajaFile *file;
 CajaDirectory *directory;
 gboolean done;
@@ -2923,6 +2930,8 @@ deep_count_next_dir (DeepCountState *state)
 
 if (state->deep_count_subdirectories != NULL)
 {
+ GFile *location;
+
 /* Work on a new directory. */
 location = state->deep_count_subdirectories->data;
 state->deep_count_subdirectories = g_list_remove
@@ -2957,7 +2966,7 @@ deep_count_more_files_callback (GObject *source_object,
 DeepCountState *state;
 CajaDirectory *directory;
 GList *files, *l;
- GFileInfo *info;
+ GFileInfo *info = NULL;
 
 state = user_data;
 
@@ -3074,10 +3083,10 @@ deep_count_load (DeepCountState *state, GFile *location)
 static void
 deep_count_stop (CajaDirectory *directory)
 {
- CajaFile *file;
-
 if (directory->details->deep_count_in_progress != NULL)
 {
+ CajaFile *file;
+
 file = directory->details->deep_count_file;
 if (file != NULL)
 {
@@ -3102,13 +3111,14 @@ deep_count_got_info (GObject *source_object,
 gpointer user_data)
 {
 GFileInfo *info;
- const char *id;
 GFile *file = (GFile *)source_object;
 DeepCountState *state = (DeepCountState *)user_data;
 
 info = g_file_query_info_finish (file, res, NULL);
 if (info != NULL)
 {
+ const char *id;
+
 id = g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_ID_FILESYSTEM);
 state->fs_id = g_strdup (id);
 g_object_unref (info);
@@ -3182,10 +3192,10 @@ deep_count_start (CajaDirectory *directory,
 static void
 mime_list_stop (CajaDirectory *directory)
 {
- CajaFile *file;
-
 if (directory->details->mime_list_in_progress != NULL)
 {
+ CajaFile *file;
+
 file = directory->details->mime_list_in_progress->mime_list_file;
 if (file != NULL)
 {
@@ -3287,7 +3297,7 @@ mime_list_callback (GObject *source_object,
 CajaDirectory *directory;
 GError *error;
 GList *files, *l;
- GFileInfo *info;
+ GFileInfo *info = NULL;
 
 state = user_data;
 directory = state->directory;
@@ -3349,13 +3359,14 @@ list_mime_enum_callback (GObject *source_object,
 {
 MimeListState *state;
 GFileEnumerator *enumerator;
- CajaDirectory *directory;
 GError *error;
 
 state = user_data;
 
 if (g_cancellable_is_cancelled (state->cancellable))
 {
+ CajaDirectory *directory;
+
 /* Operation was cancelled. Bail out */
 directory = state->directory;
 directory->details->mime_list_in_progress = NULL;
@@ -3464,10 +3475,10 @@ mime_list_start (CajaDirectory *directory,
 static void
 top_left_stop (CajaDirectory *directory)
 {
- CajaFile *file;
-
 if (directory->details->top_left_read_state != NULL)
 {
+ CajaFile *file;
+
 file = directory->details->top_left_read_state->file;
 if (file != NULL)
 {
@@ -3732,10 +3743,10 @@ query_info_callback (GObject *source_object,
 static void
 file_info_stop (CajaDirectory *directory)
 {
- CajaFile *file;
-
 if (directory->details->get_info_in_progress != NULL)
 {
+ CajaFile *file;
+
 file = directory->details->get_info_file;
 if (file != NULL)
 {
@@ -3843,7 +3854,6 @@ static gboolean
 is_link_trusted (CajaFile *file,
 gboolean is_launcher)
 {
- GFile *location;
 gboolean res;
 
 if (!is_launcher)
@@ -3860,6 +3870,8 @@ is_link_trusted (CajaFile *file,
 
 if (caja_file_is_local (file))
 {
+ GFile *location;
+
 location = caja_file_get_location (file);
 res = caja_is_in_system_dir (location);
 
@@ -3931,10 +3943,10 @@ should_read_link_info_sync (CajaFile *file)
 static void
 link_info_stop (CajaDirectory *directory)
 {
- CajaFile *file;
-
 if (directory->details->link_info_read_state != NULL)
 {
+ CajaFile *file;
+
 file = directory->details->link_info_read_state->file;
 
 if (file != NULL)
@@ -3961,7 +3973,7 @@ link_info_got_data (CajaDirectory *directory,
 goffset bytes_read,
 char *file_contents)
 {
- char *link_uri, *uri, *name, *icon;
+ char *uri, *name, *icon;
 gboolean is_launcher;
 gboolean is_foreign;
 
@@ -3976,6 +3988,8 @@ link_info_got_data (CajaDirectory *directory,
 /* Handle the case where we read the Caja link. */
 if (result)
 {
+ char *link_uri;
+
 link_uri = caja_file_get_uri (file);
 caja_link_get_link_info_given_file_contents (file_contents, bytes_read, link_uri,
 &uri, &name, &icon, &is_launcher, &is_foreign);
@@ -4117,9 +4131,6 @@ thumbnail_done (CajaDirectory *directory,
 GdkPixbuf *pixbuf,
 gboolean tried_original)
 {
- const char *thumb_mtime_str;
- time_t thumb_mtime = 0;
-
 file->details->thumbnail_is_up_to_date = TRUE;
 file->details->thumbnail_tried_original = tried_original;
 if (file->details->thumbnail)
@@ -4129,12 +4140,16 @@ thumbnail_done (CajaDirectory *directory,
 }
 if (pixbuf)
 {
+ time_t thumb_mtime = 0;
+
 if (tried_original)
 {
 thumb_mtime = file->details->mtime;
 }
 else
 {
+ const char *thumb_mtime_str;
+
 thumb_mtime_str = gdk_pixbuf_get_option (pixbuf, "tEXt::Thumb::MTime");
 if (thumb_mtime_str)
 {
@@ -4161,10 +4176,10 @@ thumbnail_done (CajaDirectory *directory,
 static void
 thumbnail_stop (CajaDirectory *directory)
 {
- CajaFile *file;
-
 if (directory->details->thumbnail_state != NULL)
 {
+ CajaFile *file;
+
 file = directory->details->thumbnail_state->file;
 
 if (file != NULL)
@@ -4300,7 +4315,6 @@ thumbnail_read_callback (GObject *source_object,
 gboolean result;
 CajaDirectory *directory;
 GdkPixbuf *pixbuf;
- GFile *location;
 
 state = user_data;
 
@@ -4327,6 +4341,8 @@ thumbnail_read_callback (GObject *source_object,
 
 if (pixbuf == NULL && state->trying_original)
 {
+ GFile *location;
+
 state->trying_original = FALSE;
 
 location = g_file_new_for_path (state->file->details->thumbnail_path);
@@ -4404,10 +4420,10 @@ thumbnail_start (CajaDirectory *directory,
 static void
 mount_stop (CajaDirectory *directory)
 {
- CajaFile *file;
-
 if (directory->details->mount_state != NULL)
 {
+ CajaFile *file;
+
 file = directory->details->mount_state->file;
 
 if (file != NULL)
@@ -4468,7 +4484,6 @@ find_enclosing_mount_callback (GObject *source_object,
 {
 GMount *mount;
 MountState *state;
- GFile *location, *root;
 
 state = user_data;
 if (state->directory == NULL)
@@ -4483,6 +4498,8 @@ find_enclosing_mount_callback (GObject *source_object,
 
 if (mount)
 {
+ GFile *location, *root;
+
 root = g_mount_get_root (mount);
 location = caja_file_get_location (state->file);
 if (!g_file_equal (location, root))
@@ -4621,10 +4638,10 @@ filesystem_info_cancel (CajaDirectory *directory)
 static void
 filesystem_info_stop (CajaDirectory *directory)
 {
- CajaFile *file;
-
 if (directory->details->filesystem_info_state != NULL)
 {
+ CajaFile *file;
+
 file = directory->details->filesystem_info_state->file;
 
 if (file != NULL)
@@ -5294,10 +5311,11 @@ static void
 add_all_files_to_work_queue (CajaDirectory *directory)
 {
 GList *node;
- CajaFile *file;
 
 for (node = directory->details->file_list; node != NULL; node = node->next)
 {
+ CajaFile *file;
+
 file = CAJA_FILE (node->data);
 
 caja_directory_add_file_to_work_queue (directory, file);
diff --git a/libcaja-private/caja-directory.c b/libcaja-private/caja-directory.c
index 8e05234c..00e03499 100644
--- a/libcaja-private/caja-directory.c
+++ b/libcaja-private/caja-directory.c
@@ -237,7 +237,7 @@ void
 emit_change_signals_for_all_files_in_all_directories (void)
 {
 GList *dirs, *l;
- CajaDirectory *directory;
+ CajaDirectory *directory = NULL;
 
 dirs = NULL;
 g_hash_table_foreach (directories,
@@ -361,11 +361,12 @@ CajaFile *
 caja_directory_get_corresponding_file (CajaDirectory *directory)
 {
 CajaFile *file;
- char *uri;
 
 file = caja_directory_get_existing_corresponding_file (directory);
 if (file == NULL)
 {
+ char *uri;
+
 uri = caja_directory_get_uri (directory);
 file = caja_file_get_by_uri (uri);
 g_free (uri);
@@ -719,12 +720,13 @@ caja_directory_emit_load_error (CajaDirectory *directory,
 static CajaDirectory *
 get_parent_directory (GFile *location)
 {
- CajaDirectory *directory;
 GFile *parent;
 
 parent = g_file_get_parent (location);
 if (parent)
 {
+ CajaDirectory *directory;
+
 directory = caja_directory_get_internal (parent, TRUE);
 g_object_unref (parent);
 return directory;
@@ -738,12 +740,13 @@ get_parent_directory (GFile *location)
 static CajaDirectory *
 get_parent_directory_if_exists (GFile *location)
 {
- CajaDirectory *directory;
 GFile *parent;
 
 parent = g_file_get_parent (location);
 if (parent)
 {
+ CajaDirectory *directory;
+
 directory = caja_directory_get_internal (parent, FALSE);
 g_object_unref (parent);
 return directory;
@@ -778,7 +781,7 @@ static void
 call_files_changed_common (CajaDirectory *directory, GList *file_list)
 {
 GList *node;
- CajaFile *file;
+ CajaFile *file = NULL;
 
 for (node = file_list; node != NULL; node = node->next)
 {
@@ -859,10 +862,11 @@ caja_directory_notify_files_added (GList *files)
 {
 GHashTable *added_lists;
 GList *p;
- CajaDirectory *directory;
 GHashTable *parent_directories;
 CajaFile *file;
- GFile *location, *parent;
+ GFile *parent;
+ CajaDirectory *directory = NULL;
+ GFile *location = NULL;
 
 /* Make a list of added files in each directory. */
 added_lists = g_hash_table_new (NULL, NULL);
@@ -948,8 +952,8 @@ caja_directory_notify_files_changed (GList *files)
 {
 GHashTable *changed_lists;
 GList *node;
- GFile *location;
- CajaFile *file;
+ GFile *location = NULL;
+ CajaFile *file = NULL;
 
 /* Make a list of changed files in each directory. */
 changed_lists = g_hash_table_new (NULL, NULL);
@@ -986,10 +990,10 @@ caja_directory_notify_files_removed (GList *files)
 {
 GHashTable *changed_lists;
 GList *p;
- CajaDirectory *directory;
 GHashTable *parent_directories;
- CajaFile *file;
- GFile *location;
+ CajaDirectory *directory = NULL;
+ CajaFile *file = NULL;
+ GFile *location = NULL;
 
 /* Make a list of changed files in each directory. */
 changed_lists = g_hash_table_new (NULL, NULL);
@@ -1095,10 +1099,10 @@ caja_directory_moved_internal (GFile *old_location,
 GFile *new_location)
 {
 CollectData collection;
- CajaDirectory *directory;
 GList *node, *affected_files;
- GFile *new_directory_location;
 char *relative_path;
+ CajaDirectory *directory = NULL;
+ GFile *new_directory_location = NULL;
 
 collection.container = old_location;
 collection.directories = NULL;
@@ -1161,9 +1165,9 @@ caja_directory_moved (const char *old_uri,
 {
 GList *list, *node;
 GHashTable *hash;
- CajaFile *file;
 GFile *old_location;
 GFile *new_location;
+ CajaFile *file = NULL;
 
 hash = g_hash_table_new (NULL, NULL);
 
@@ -1199,7 +1203,8 @@ caja_directory_notify_files_moved (GList *file_pairs)
 GHashTable *added_lists, *changed_lists;
 char *name;
 CajaFileAttributes cancel_attributes;
- GFile *to_location, *from_location;
+ GFile *to_location = NULL;
+ GFile *from_location = NULL;
 
 /* Make a list of added and changed files in each directory. */
 new_files_list = NULL;
@@ -1318,10 +1323,10 @@ void
 caja_directory_schedule_position_set (GList *position_setting_list)
 {
 GList *p;
- const CajaFileChangesQueuePosition *item;
- CajaFile *file;
 char str[64];
 time_t now;
+ const CajaFileChangesQueuePosition *item = NULL;
+ CajaFile *file = NULL;
 
 time (&now);
 
diff --git a/libcaja-private/caja-dnd.c b/libcaja-private/caja-dnd.c
index 83ae648e..dfea8023 100644
--- a/libcaja-private/caja-dnd.c
+++ b/libcaja-private/caja-dnd.c
@@ -125,9 +125,9 @@ caja_drag_destroy_selection_list (GList *list)
 GList *
 caja_drag_uri_list_from_selection_list (const GList *selection_list)
 {
- CajaDragSelectionItem *selection_item;
 GList *uri_list;
 const GList *l;
+ CajaDragSelectionItem *selection_item = NULL;
 
 uri_list = NULL;
 for (l = selection_list; l != NULL; l = l->next)
@@ -369,13 +369,14 @@ static gboolean
 check_same_fs (CajaFile *file1,
 CajaFile *file2)
 {
- char *id1, *id2;
 gboolean result;
 
 result = FALSE;
 
 if (file1 != NULL && file2 != NULL)
 {
+ char *id1, *id2;
+
 id1 = caja_file_get_filesystem_id (file1);
 id2 = caja_file_get_filesystem_id (file2);
 
@@ -637,7 +638,6 @@ static void
 add_one_compatible_uri (const char *uri, int x, int y, int w, int h, gpointer data)
 {
 GString *result;
- char *local_path;
 
 result = (GString *) data;
 
@@ -654,6 +654,8 @@ add_one_compatible_uri (const char *uri, int x, int y, int w, int h, gpointer da
 }
 else
 {
+ char *local_path;
+
 local_path = g_filename_from_uri (uri, NULL, NULL);
 
 /* Check for characters that confuse the old
@@ -1039,10 +1041,11 @@ gboolean
 caja_drag_selection_includes_special_link (GList *selection_list)
 {
 GList *node;
- char *uri;
 
 for (node = selection_list; node != NULL; node = node->next)
 {
+ char *uri;
+
 uri = ((CajaDragSelectionItem *) node->data)->uri;
 
 if (eel_uri_is_desktop (uri))
@@ -1226,7 +1229,6 @@ slot_proxy_handle_drop (GtkWidget *widget,
 CajaWindowSlotInfo *target_slot;
 CajaView *target_view;
 char *target_uri;
- GList *uri_list;
 
 if (!drag_info->have_data ||
 !drag_info->have_valid_data)
@@ -1268,6 +1270,8 @@ slot_proxy_handle_drop (GtkWidget *widget,
 {
 if (drag_info->info == CAJA_ICON_DND_MATE_ICON_LIST)
 {
+ GList *uri_list;
+
 uri_list = caja_drag_uri_list_from_selection_list (drag_info->data.selection_list);
 g_assert (uri_list != NULL);
 
diff --git a/libcaja-private/caja-entry.c b/libcaja-private/caja-entry.c
index d8e90438..7c1db0b9 100644
--- a/libcaja-private/caja-entry.c
+++ b/libcaja-private/caja-entry.c
@@ -96,7 +96,6 @@ caja_entry_key_press (GtkWidget *widget, GdkEventKey *event)
 {
 CajaEntry *entry;
 GtkEditable *editable;
- int position;
 gboolean old_has, new_has;
 gboolean result;
 
@@ -119,6 +118,8 @@ caja_entry_key_press (GtkWidget *widget, GdkEventKey *event)
 */
 if (entry->details->special_tab_handling && gtk_editable_get_selection_bounds (editable, NULL, NULL))
 {
+ int position;
+
 position = strlen (gtk_entry_get_text (GTK_ENTRY (editable)));
 gtk_editable_select_region (editable, position, position);
 return TRUE;
diff --git a/libcaja-private/caja-file-changes-queue.c b/libcaja-private/caja-file-changes-queue.c
index 58f11ad4..20363723 100644
--- a/libcaja-private/caja-file-changes-queue.c
+++ b/libcaja-private/caja-file-changes-queue.c
@@ -220,7 +220,7 @@ static void
 pairs_list_free (GList *pairs)
 {
 GList *p;
- GFilePair *pair;
+ GFilePair *pair = NULL;
 
 /* deep delete the list of pairs */
 
@@ -240,7 +240,7 @@ static void
 position_set_list_free (GList *list)
 {
 GList *p;
- CajaFileChangesQueuePosition *item;
+ CajaFileChangesQueuePosition *item = NULL;
 
 for (p = list; p != NULL; p = p->next)
 {
diff --git a/libcaja-private/caja-file-conflict-dialog.c b/libcaja-private/caja-file-conflict-dialog.c
index 163d877f..97266fb8 100644
--- a/libcaja-private/caja-file-conflict-dialog.c
+++ b/libcaja-private/caja-file-conflict-dialog.c
@@ -526,11 +526,12 @@ diff_button_clicked_cb (GtkButton *w,
 
 GError *error;
 char *command;
- char **argv;
 
 command = g_find_program_in_path ("meld");
 if (command)
 {
+ char **argv;
+
 argv = g_new (char *, 4);
 argv[0] = command;
 argv[1] = g_file_get_path (caja_file_get_location (details->source));
diff --git a/libcaja-private/caja-file-dnd.c b/libcaja-private/caja-file-dnd.c
index 061019e9..e5322486 100644
--- a/libcaja-private/caja-file-dnd.c
+++ b/libcaja-private/caja-file-dnd.c
@@ -35,10 +35,9 @@
 static gboolean
 caja_drag_can_accept_files (CajaFile *drop_target_item)
 {
- CajaDirectory *directory;
-
 if (caja_file_is_directory (drop_target_item))
 {
+ CajaDirectory *directory;
 gboolean res;
 
 /* target is a directory, accept if editable */
@@ -154,7 +153,7 @@ void
 caja_drag_file_receive_dropped_keyword (CajaFile *file,
 const char *keyword)
 {
- GList *keywords, *word;
+ GList *keywords;
 
 g_return_if_fail (CAJA_IS_FILE (file));
 g_return_if_fail (keyword != NULL);
@@ -166,6 +165,8 @@ caja_drag_file_receive_dropped_keyword (CajaFile *file,
 }
 else
 {
+ GList *word;
+
 keywords = caja_file_get_keywords (file);
 word = g_list_find_custom (keywords, keyword, (GCompareFunc) strcmp);
 if (word == NULL)
diff --git a/libcaja-private/caja-file-operations.c b/libcaja-private/caja-file-operations.c
index eb62c7b0..dac74145 100644
--- a/libcaja-private/caja-file-operations.c
+++ b/libcaja-private/caja-file-operations.c
@@ -296,7 +296,6 @@ format_time (int seconds)
 {
 int minutes;
 int hours;
- char *res;
 
 if (seconds < 0) {
 /* Just to make sure... */
@@ -316,6 +315,7 @@ format_time (int seconds)
 
 if (seconds < 60*60*4) {
 char *h, *m;
+ char *res;
 
 minutes = (seconds - hours * 60 * 60) / 60;
 
@@ -950,7 +950,6 @@ init_common (gsize job_size,
 gboolean should_start, gboolean can_pause)
 {
 CommonJob *common;
- GdkScreen *screen;
 
 /* expected warning with Clang static analyzer: *
 * "Cast a region whose size is not a multiple of the destination type size" *
@@ -972,6 +971,8 @@ init_common (gsize job_size,
 common->inhibit_cookie = -1;
 common->screen_num = 0;
 if (parent_window) {
+ GdkScreen *screen;
+
 screen = gtk_widget_get_screen (GTK_WIDGET (parent_window));
 common->screen_num = gdk_x11_screen_get_screen_number (screen);
 }
@@ -1095,7 +1096,6 @@ static gboolean
 do_run_simple_dialog (gpointer _data)
 {
 RunSimpleDialogData *data = _data;
- const char *button_title;
 GtkWidget *dialog;
 int result;
 int response_id;
@@ -1115,6 +1115,8 @@ do_run_simple_dialog (gpointer _data)
 for (response_id = 0;
 data->button_titles[response_id] != NULL;
 response_id++) {
+ const char *button_title;
+
 button_title = data->button_titles[response_id];
 if (!data->show_all && is_all_button_text (button_title)) {
 continue;
@@ -1501,8 +1503,7 @@ report_delete_progress (CommonJob *job,
 TransferInfo *transfer_info)
 {
 int files_left;
- double elapsed, transfer_rate;
- int remaining_time;
+ double elapsed;
 gint64 now;
 char *files_left_s;
 
@@ -1534,6 +1535,9 @@ report_delete_progress (CommonJob *job,
 caja_progress_info_set_details (job->progress, files_left_s);
 } else {
 char *details, *time_left_s;
+ int remaining_time;
+ double transfer_rate;
+
 transfer_rate = transfer_info->num_files / elapsed;
 remaining_time = files_left / transfer_rate;
 
@@ -1789,10 +1793,10 @@ static void
 delete_files (CommonJob *job, GList *files, int *files_skipped)
 {
 GList *l;
- GFile *file;
 SourceInfo source_info;
 TransferInfo transfer_info;
 gboolean skipped_file;
+ GFile *file = NULL;
 
 if (job_aborted (job)) {
 return;
@@ -1953,13 +1957,14 @@ static gboolean
 delete_job_done (gpointer user_data)
 {
 DeleteJob *job;
- GHashTable *debuting_uris;
 
 job = user_data;
 
 g_list_free_full (job->files, g_object_unref);
 
 if (job->done_callback) {
+ GHashTable *debuting_uris;
+
 debuting_uris = g_hash_table_new_full (g_file_hash, (GEqualFunc)g_file_equal, g_object_unref, NULL);
 job->done_callback (debuting_uris, job->user_cancel, job->done_callback_data);
 g_hash_table_unref (debuting_uris);
@@ -1981,13 +1986,13 @@ delete_job (GIOSchedulerJob *io_job,
 GList *to_trash_files;
 GList *to_delete_files;
 GList *l;
- GFile *file;
 gboolean confirmed;
 CommonJob *common;
 gboolean must_confirm_delete_in_trash;
 gboolean must_confirm_delete;
 gboolean must_confirm_trash;
 int files_skipped;
+ GFile *file = NULL;
 
 common = (CommonJob *)job;
 common->io_job = io_job;
@@ -2142,7 +2147,6 @@ unmount_mount_callback (GObject *source_object,
 {
 UnmountData *data = user_data;
 GError *error;
- char *primary;
 gboolean unmounted;
 
 error = NULL;
@@ -2160,6 +2164,8 @@ unmount_mount_callback (GObject *source_object,
 
 if (! unmounted) {
 if (error && error->code != G_IO_ERROR_FAILED_HANDLED) {
+ char *primary;
+
 if (data->eject) {
 primary = f (_("Unable to eject %V"), source_object);
 } else {
@@ -2217,7 +2223,6 @@ dir_has_files (GFile *dir)
 {
 GFileEnumerator *enumerator;
 gboolean res;
- GFileInfo *file_info;
 
 res = FALSE;
 
@@ -2226,6 +2231,8 @@ dir_has_files (GFile *dir)
 0,
 NULL, NULL);
 if (enumerator) {
+ GFileInfo *file_info;
+
 file_info = g_file_enumerator_next_file (enumerator, NULL, NULL);
 if (file_info != NULL) {
 res = TRUE;
@@ -2244,8 +2251,6 @@ static GList *
 get_trash_dirs_for_mount (GMount *mount)
 {
 GFile *root;
- GFile *trash;
- char *relpath;
 GList *list;
 
 root = g_mount_get_root (mount);
@@ -2256,6 +2261,9 @@ get_trash_dirs_for_mount (GMount *mount)
 list = NULL;
 
 if (g_file_is_native (root)) {
+ GFile *trash;
+ char *relpath;
+
 relpath = g_strdup_printf (".Trash/%d", getuid ());
 trash = g_file_resolve_relative_path (root, relpath);
 g_free (relpath);
@@ -2283,9 +2291,9 @@ get_trash_dirs_for_mount (GMount *mount)
 static gboolean
 has_trash_files (GMount *mount)
 {
- GList *dirs, *l;
- GFile *dir;
 gboolean res;
+ GList *dirs, *l;
+ GFile *dir = NULL;
 
 dirs = get_trash_dirs_for_mount (mount);
 
@@ -2367,7 +2375,6 @@ caja_file_operations_unmount_mount_full (GtkWindow *parent_
 gpointer callback_data)
 {
 UnmountData *data;
- int response;
 
 data = g_new0 (UnmountData, 1);
 data->callback = callback;
@@ -2381,6 +2388,8 @@ caja_file_operations_unmount_mount_full (GtkWindow *parent_
 data->mount = g_object_ref (mount);
 
 if (check_trash && has_trash_files (mount)) {
+ int response;
+
 response = prompt_empty_trash (parent_window);
 
 if (response == GTK_RESPONSE_ACCEPT) {
@@ -2441,13 +2450,14 @@ volume_mount_cb (GObject *source_object,
 GObject *mount_callback_data_object;
 GMountOperation *mount_op = user_data;
 GError *error;
- char *primary;
- char *name;
 
 error = NULL;
 caja_allow_autorun_for_volume_finish (G_VOLUME (source_object));
 if (!g_volume_mount_finish (G_VOLUME (source_object), res, &error)) {
 if (error->code != G_IO_ERROR_FAILED_HANDLED) {
+ char *name;
+ char *primary;
+
 name = g_volume_get_name (G_VOLUME (source_object));
 primary = g_strdup_printf (_("Unable to mount %s"), name);
 g_free (name);
@@ -2805,7 +2815,7 @@ scan_sources (GList *files,
 OpKind kind)
 {
 GList *l;
- GFile *file;
+ GFile *file = NULL;
 
 memset (source_info, 0, sizeof (SourceInfo));
 source_info->op = kind;
@@ -2989,7 +2999,6 @@ report_copy_progress (CopyMoveJob *copy_job,
 int files_left;
 goffset total_size;
 double elapsed, transfer_rate;
- int remaining_time;
 guint64 now;
 CommonJob *job;
 gboolean is_move;
@@ -3091,7 +3100,9 @@ report_copy_progress (CopyMoveJob *copy_job,
 s = f (_("%S of %S"), transfer_info->num_bytes, total_size);
 caja_progress_info_take_details (job->progress, s);
 } else {
+ int remaining_time;
 char *s;
+
 remaining_time = (total_size - transfer_info->num_bytes) / transfer_rate;
 
 /* To translators: %S will expand to a size like "2 bytes" or "3 MB", %T to a time duration like
@@ -3208,8 +3219,7 @@ get_unique_target_file (GFile *src,
 const char *dest_fs_type,
 int count)
 {
- const char *editname, *end;
- char *basename, *new_name;
+ char *new_name;
 GFileInfo *info;
 GFile *dest;
 int max_length;
@@ -3221,6 +3231,8 @@ get_unique_target_file (GFile *src,
 G_FILE_ATTRIBUTE_STANDARD_EDIT_NAME,
 0, NULL, NULL);
 if (info != NULL) {
+ const char *editname;
+
 editname = g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_STANDARD_EDIT_NAME);
 
 if (editname != NULL) {
@@ -3234,6 +3246,8 @@ get_unique_target_file (GFile *src,
 }
 
 if (dest == NULL) {
+ char *basename;
+
 basename = g_file_get_basename (src);
 
 if (g_utf8_validate (basename, -1, NULL)) {
@@ -3244,6 +3258,8 @@ get_unique_target_file (GFile *src,
 }
 
 if (dest == NULL) {
+ const char *end;
+
 end = strrchr (basename, '.');
 if (end != NULL) {
 count += atoi (end + 1);
@@ -3266,8 +3282,7 @@ get_target_file_for_link (GFile *src,
 const char *dest_fs_type,
 int count)
 {
- const char *editname;
- char *basename, *new_name;
+ char *new_name;
 GFileInfo *info;
 GFile *dest;
 int max_length;
@@ -3279,6 +3294,8 @@ get_target_file_for_link (GFile *src,
 G_FILE_ATTRIBUTE_STANDARD_EDIT_NAME,
 0, NULL, NULL);
 if (info != NULL) {
+ const char *editname;
+
 editname = g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_STANDARD_EDIT_NAME);
 
 if (editname != NULL) {
@@ -3292,6 +3309,8 @@ get_target_file_for_link (GFile *src,
 }
 
 if (dest == NULL) {
+ char *basename;
+
 basename = g_file_get_basename (src);
 make_file_name_valid_for_dest_fs (basename, dest_fs_type);
 
@@ -3325,18 +3344,19 @@ get_target_file (GFile *src,
 const char *dest_fs_type,
 gboolean same_fs)
 {
- char *basename;
 GFile *dest;
- GFileInfo *info;
- char *copyname;
 
 dest = NULL;
 if (!same_fs) {
+ GFileInfo *info;
+
 info = g_file_query_info (src,
 G_FILE_ATTRIBUTE_STANDARD_COPY_NAME,
 0, NULL, NULL);
 
 if (info) {
+ char *copyname;
+
 copyname = g_strdup (g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_STANDARD_COPY_NAME));
 
 if (copyname) {
@@ -3350,6 +3370,8 @@ get_target_file (GFile *src,
 }
 
 if (dest == NULL) {
+ char *basename;
+
 basename = g_file_get_basename (src);
 make_file_name_valid_for_dest_fs (basename, dest_fs_type);
 dest = g_file_get_child (dest_dir, basename);
@@ -3362,7 +3384,6 @@ get_target_file (GFile *src,
 static gboolean
 has_fs_id (GFile *file, const char *fs_id)
 {
- const char *id;
 GFileInfo *info;
 gboolean res;
 
@@ -3373,6 +3394,8 @@ has_fs_id (GFile *file, const char *fs_id)
 NULL, NULL);
 
 if (info) {
+ const char *id;
+
 id = g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_ID_FILESYSTEM);
 
 if (id && strcmp (id, fs_id) == 0) {
@@ -4514,7 +4537,6 @@ copy_files (CopyMoveJob *job,
 {
 CommonJob *common;
 GList *l;
- GFile *src;
 gboolean same_fs;
 int i;
 GdkPoint *point;
@@ -4523,8 +4545,8 @@ copy_files (CopyMoveJob *job,
 GFile *dest;
 GFile *source_dir;
 char *dest_fs_type;
- GFileInfo *inf;
 gboolean readonly_source_fs;
+ GFile *src = NULL;
 
 dest_fs_type = NULL;
 readonly_source_fs = FALSE;
@@ -4536,6 +4558,8 @@ copy_files (CopyMoveJob *job,
 /* Query the source dir, not the file because if its a symlink we'll follow it */
 source_dir = g_file_get_parent ((GFile *) job->files->data);
 if (source_dir) {
+ GFileInfo *inf;
+
 inf = g_file_query_filesystem_info (source_dir, "filesystem::readonly", NULL, NULL);
 if (inf != NULL) {
 readonly_source_fs = g_file_info_get_attribute_boolean (inf, "filesystem::readonly");
@@ -4792,8 +4816,8 @@ move_copy_file_callback_new (GFile *file,
 static GList *
 get_files_from_fallbacks (GList *fallbacks)
 {
- MoveFileCopyFallback *fallback;
 GList *res, *l;
+ MoveFileCopyFallback *fallback = NULL;
 
 res = NULL;
 for (l = fallbacks; l != NULL; l = l->next) {
@@ -5032,12 +5056,12 @@ move_files_prepare (CopyMoveJob *job,
 {
 CommonJob *common;
 GList *l;
- GFile *src;
 gboolean same_fs;
 gboolean last_item;
 int i;
 GdkPoint *point;
 int total, left;
+ GFile *src = NULL;
 
 common = &job->common;
 
@@ -5094,13 +5118,14 @@ move_files (CopyMoveJob *job,
 {
 CommonJob *common;
 GList *l;
- GFile *src;
 gboolean same_fs;
 int i;
 GdkPoint *point;
 gboolean skipped_file;
 MoveFileCopyFallback *fallback;
-common = &job->common;
+ GFile *src = NULL;
+
+ common = &job->common;
 
 report_copy_progress (job, source_info, transfer_info);
 
@@ -5108,7 +5133,7 @@ common = &job->common;
 for (l = fallbacks;
 l != NULL && !job_aborted (common);
 l = l->next) {
- caja_progress_info_get_ready (common->progress);
+ caja_progress_info_get_ready (common->progress);
 
 fallback = l->data;
 src = fallback->file;
@@ -5666,19 +5691,16 @@ set_permissions_file (SetPermissionsJob *job,
 GFileInfo *info)
 {
 CommonJob *common;
- GFileInfo *child_info;
 gboolean free_info;
 guint32 current;
 guint32 value;
 guint32 mask;
- GFileEnumerator *enumerator;
- GFile *child;
 
 common = (CommonJob *)job;
 
 caja_progress_info_pulse_progress (common->progress);
 
- caja_progress_info_get_ready (common->progress);
+ caja_progress_info_get_ready (common->progress);
 
 free_info = FALSE;
 if (info == NULL) {
@@ -5717,8 +5739,9 @@ set_permissions_file (SetPermissionsJob *job,
 common->cancellable, NULL);
 }
 
- if (!job_aborted (common) &&
- g_file_info_get_file_type (info) == G_FILE_TYPE_DIRECTORY) {
+ if (!job_aborted (common) && g_file_info_get_file_type (info) == G_FILE_TYPE_DIRECTORY) {
+ GFileEnumerator *enumerator;
+
 enumerator = g_file_enumerate_children (file,
 G_FILE_ATTRIBUTE_STANDARD_NAME","
 G_FILE_ATTRIBUTE_STANDARD_TYPE","
@@ -5727,6 +5750,9 @@ set_permissions_file (SetPermissionsJob *job,
 common->cancellable,
 NULL);
 if (enumerator) {
+ GFile *child = NULL;
+ GFileInfo *child_info = NULL;
+
 while (!job_aborted (common) &&
 (child_info = g_file_enumerator_next_file (enumerator, common->cancellable, NULL)) != NULL) {
 child = g_file_get_child (file,
@@ -5814,7 +5840,7 @@ location_list_from_uri_list (const GList *uris)
 {
 const GList *l;
 GList *files;
- GFile *f;
+ GFile *f = NULL;
 
 files = NULL;
 for (l = uris; l != NULL; l = l->next) {
@@ -5851,7 +5877,7 @@ caja_file_operations_copy_move (const GList *item_uris,
 {
 GList *locations;
 GList *p;
- GFile *dest, *src_dir;
+ GFile *dest;
 GtkWindow *parent_window;
 gboolean target_is_mapping;
 gboolean have_nonmapping_source;
@@ -5888,6 +5914,8 @@ caja_file_operations_copy_move (const GList *item_uris,
 }
 
 if (copy_action == GDK_ACTION_COPY) {
+ GFile *src_dir;
+
 src_dir = g_file_get_parent (locations->data);
 if (target_dir == NULL ||
 (src_dir != NULL &&
@@ -6370,17 +6398,15 @@ delete_trash_file (CommonJob *job,
 gboolean del_file,
 gboolean del_children)
 {
- GFileInfo *info;
- GFile *child;
- GFileEnumerator *enumerator;
-
- caja_progress_info_get_ready (job->progress);
+ caja_progress_info_get_ready (job->progress);
 
 if (job_aborted (job)) {
 return;
 }
 
 if (del_children) {
+ GFileEnumerator *enumerator;
+
 enumerator = g_file_enumerate_children (file,
 G_FILE_ATTRIBUTE_STANDARD_NAME ","
 G_FILE_ATTRIBUTE_STANDARD_TYPE,
@@ -6388,6 +6414,9 @@ delete_trash_file (CommonJob *job,
 job->cancellable,
 NULL);
 if (enumerator) {
+ GFileInfo *info = NULL;
+ GFile *child = NULL;
+
 while (!job_aborted (job) &&
 (info = g_file_enumerator_next_file (enumerator, job->cancellable, NULL)) != NULL) {
 child = g_file_get_child (file,
diff --git a/libcaja-private/caja-file-utilities.c b/libcaja-private/caja-file-utilities.c
index 925c08f2..84198bfa 100644
--- a/libcaja-private/caja-file-utilities.c
+++ b/libcaja-private/caja-file-utilities.c
@@ -55,7 +55,6 @@ static void desktop_dir_changed (void);
 char *
 caja_compute_title_for_location (GFile *location)
 {
- CajaFile *file;
 char *title;
 
 /* TODO-gio: This doesn't really work all that great if the
@@ -64,6 +63,8 @@ caja_compute_title_for_location (GFile *location)
 title = NULL;
 if (location)
 {
+ CajaFile *file;
+
 file = caja_file_get (location);
 title = caja_file_get_description (file);
 if (title == NULL)
@@ -195,10 +196,8 @@ parse_xdg_dirs (const char *config_file)
 XdgDirEntry dir;
 char *data;
 char **lines;
- char *p, *d;
- int i;
- char *type_start, *type_end;
- char *value, *unescaped;
+ char *p;
+ char *unescaped;
 gboolean relative;
 
 array = g_array_new (TRUE, TRUE, sizeof (XdgDirEntry));
@@ -212,10 +211,16 @@ parse_xdg_dirs (const char *config_file)
 
 if (g_file_get_contents (config_file, &data, NULL, NULL))
 {
+ int i;
+
 lines = g_strsplit (data, "\n", 0);
 g_free (data);
 for (i = 0; lines[i] != NULL; i++)
 {
+ char *d;
+ char *type_start, *type_end;
+ char *value;
+
 p = lines[i];
 while (g_ascii_isspace (*p))
 p++;
@@ -383,10 +388,10 @@ unschedule_user_dirs_changed (void)
 static void
 free_xdg_dir_cache (void)
 {
- int i;
-
 if (cached_xdg_dirs != NULL)
 {
+ int i;
+
 for (i = 0; cached_xdg_dirs[i].type != NULL; i++)
 {
 if (cached_xdg_dirs[i].file != NULL)
@@ -422,8 +427,7 @@ destroy_xdg_dir_cache (void)
 static void
 update_xdg_dir_cache (void)
 {
- GFile *file;
- char *config_file, *uri;
+ char *uri;
 int i;
 
 free_xdg_dir_cache ();
@@ -451,6 +455,9 @@ update_xdg_dir_cache (void)
 
 if (cached_xdg_dirs_monitor == NULL)
 {
+ GFile *file;
+ char *config_file;
+
 config_file = g_build_filename (g_get_user_config_dir (),
 "user-dirs.dirs", NULL);
 file = g_file_new_for_path (config_file);
@@ -672,12 +679,13 @@ gboolean
 caja_is_home_directory_file (GFile *dir,
 const char *filename)
 {
- char *dirname;
 static GFile *home_dir_dir = NULL;
 static char *home_dir_filename = NULL;
 
 if (home_dir_dir == NULL)
 {
+ char *dirname;
+
 dirname = g_path_get_dirname (g_get_home_dir ());
 home_dir_dir = g_file_new_for_path (dirname);
 g_free (dirname);
@@ -763,7 +771,7 @@ caja_get_mounted_mount_for_root (GFile *location)
 GVolumeMonitor *volume_monitor;
 GList *mounts;
 GList *l;
- GMount *mount;
+ GMount *mount = NULL;
 GMount *result = NULL;
 GFile *root = NULL;
 GFile *default_location = NULL;
@@ -917,8 +925,8 @@ caja_ensure_unique_file_name (const char *directory_uri,
 GFile *
 caja_find_existing_uri_in_hierarchy (GFile *location)
 {
- GFileInfo *info;
- GFile *tmp;
+ GFileInfo *info = NULL;
+ GFile *tmp = NULL;
 
 g_assert (location != NULL);
 
@@ -1131,8 +1139,10 @@ caja_trashed_files_get_original_directories (GList *files,
 GList **unhandled_files)
 {
 GHashTable *directories;
- CajaFile *file, *original_file, *original_dir;
 GList *l, *m;
+ CajaFile *file = NULL;
+ CajaFile *original_file = NULL;
+ CajaFile *original_dir = NULL;
 
 directories = NULL;
 
@@ -1192,8 +1202,8 @@ caja_trashed_files_get_original_directories (GList *files,
 static GList *
 locations_from_file_list (GList *file_list)
 {
- CajaFile *file;
 GList *l, *ret;
+ CajaFile *file = NULL;
 
 ret = NULL;
 
@@ -1210,17 +1220,17 @@ void
 caja_restore_files_from_trash (GList *files,
 GtkWindow *parent_window)
 {
- CajaFile *file, *original_dir;
 GHashTable *original_dirs_hash;
 GList *original_dirs, *unhandled_files;
- GFile *original_dir_location;
- GList *locations, *l;
- char *message, *file_name;
+ GList *l;
+ CajaFile *file = NULL;
 
 original_dirs_hash = caja_trashed_files_get_original_directories (files, &unhandled_files);
 
 for (l = unhandled_files; l != NULL; l = l->next)
 {
+ char *message, *file_name;
+
 file = CAJA_FILE (l->data);
 file_name = caja_file_get_display_name (file);
 message = g_strdup_printf (_("Could not determine original location of \"%s\" "), file_name);
@@ -1234,6 +1244,10 @@ caja_restore_files_from_trash (GList *files,
 
 if (original_dirs_hash != NULL)
 {
+ CajaFile *original_dir = NULL;
+ GFile *original_dir_location = NULL;
+ GList *locations = NULL;
+
 original_dirs = g_hash_table_get_keys (original_dirs_hash);
 for (l = original_dirs; l != NULL; l = l->next)
 {
diff --git a/libcaja-private/caja-file.c b/libcaja-private/caja-file.c
index 3be71387..6c2b829d 100644
--- a/libcaja-private/caja-file.c
+++ b/libcaja-private/caja-file.c
@@ -767,13 +767,14 @@ finalize (GObject *object)
 {
 CajaDirectory *directory;
 CajaFile *file;
- char *uri;
 
 file = CAJA_FILE (object);
 
 g_assert (file->details->operations_in_progress == NULL);
 
 if (file->details->is_thumbnailing) {
+ char *uri;
+
 uri = caja_file_get_uri (file);
 caja_thumbnail_remove_from_queue (uri);
 g_free (uri);
@@ -1703,12 +1704,6 @@ rename_get_info_callback (GObject *source_object,
 gpointer callback_data)
 {
 CajaFileOperation *op;
- CajaDirectory *directory;
- CajaFile *existing_file;
- char *old_name;
- char *old_uri;
- char *new_uri;
- const char *new_name;
 GFileInfo *new_info;
 GError *error;
 
@@ -1717,6 +1712,13 @@ rename_get_info_callback (GObject *source_object,
 error = NULL;
 new_info = g_file_query_info_finish (G_FILE (source_object), res, &error);
 if (new_info != NULL) {
+ CajaDirectory *directory;
+ CajaFile *existing_file;
+ char *old_name;
+ char *old_uri;
+ char *new_uri;
+ const char *new_name;
+
 directory = op->file->details->directory;
 
 new_name = g_file_info_get_name (new_info);
@@ -1806,7 +1808,6 @@ caja_file_rename (CajaFile *file,
 gpointer callback_data)
 {
 CajaFileOperation *op;
- char *uri;
 char *old_name;
 char *new_file_name;
 gboolean success, name_changed;
@@ -1905,6 +1906,8 @@ caja_file_rename (CajaFile *file,
 }
 
 if (is_renameable_desktop_file) {
+ char *uri;
+
 /* Don't actually change the name if the new name is the same.
 * This helps for the vfolder method where this can happen and
 * we want to minimize actual changes
@@ -1974,7 +1977,7 @@ gboolean
 caja_file_rename_in_progress (CajaFile *file)
 {
 GList *node;
- CajaFileOperation *op;
+ CajaFileOperation *op = NULL;
 
 for (node = file->details->operations_in_progress; node != NULL; node = node->next) {
 op = node->data;
@@ -1991,7 +1994,7 @@ caja_file_cancel (CajaFile *file,
 gpointer callback_data)
 {
 GList *node, *next;
- CajaFileOperation *op;
+ CajaFileOperation *op = NULL;
 
 for (node = file->details->operations_in_progress; node != NULL; node = next) {
 next = node->next;
@@ -2062,12 +2065,13 @@ update_link (CajaFile *link_file, CajaFile *target_file)
 static GList *
 get_link_files (CajaFile *target_file)
 {
- char *uri;
 GList **link_files;
 
 if (symbolic_links == NULL) {
 link_files = NULL;
 } else {
+ char *uri;
+
 uri = caja_file_get_uri (target_file);
 link_files = g_hash_table_lookup (symbolic_links, uri);
 g_free (uri);
@@ -2095,7 +2099,6 @@ update_info_internal (CajaFile *file,
 GFileInfo *info,
 gboolean update_name)
 {
- GList *node;
 gboolean changed;
 gboolean is_symlink, is_hidden, is_backup, is_mountpoint;
 gboolean has_permissions;
@@ -2112,11 +2115,9 @@ update_info_internal (CajaFile *file,
 time_t trash_time;
 GTimeVal g_trash_time;
 const char * time_string;
- const char *symlink_name, *mime_type, *selinux_context, *name, *thumbnail_path;
+ const char *symlink_name, *mime_type, *selinux_context, *thumbnail_path;
 GFileType file_type;
 GIcon *icon;
- char *old_activation_uri;
- const char *activation_uri;
 const char *description;
 const char *filesystem_id;
 const char *trash_orig_path;
@@ -2159,6 +2160,8 @@ update_info_internal (CajaFile *file,
 file->details->type = file_type;
 
 if (!file->details->got_custom_activation_uri) {
+ const char *activation_uri;
+
 activation_uri = g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_STANDARD_TARGET_URI);
 if (activation_uri == NULL) {
 if (file->details->activation_uri) {
@@ -2167,6 +2170,8 @@ update_info_internal (CajaFile *file,
 changed = TRUE;
 }
 } else {
+ char *old_activation_uri;
+
 old_activation_uri = file->details->activation_uri;
 file->details->activation_uri = g_strdup (activation_uri);
 
@@ -2503,9 +2508,13 @@ update_info_internal (CajaFile *file,
 caja_file_update_metadata_from_info (file, info);
 
 if (update_name) {
+ const char *name;
+
 name = g_file_info_get_name (info);
 if (file->details->name == NULL ||
 strcmp (file->details->name, name) != 0) {
+ GList *node;
+
 changed = TRUE;
 
 node = caja_directory_begin_file_name_change
@@ -3052,7 +3061,6 @@ static int
 compare_by_emblems (CajaFile *file_1, CajaFile *file_2)
 {
 const char *keyword_cache_1, *keyword_cache_2;
- size_t length;
 int compare_result;
 
 fill_emblem_cache_if_needed (file_1);
@@ -3063,6 +3071,8 @@ compare_by_emblems (CajaFile *file_1, CajaFile *file_2)
 keyword_cache_1 = file_1->details->compare_by_emblem_cache->emblem_keywords;
 keyword_cache_2 = file_2->details->compare_by_emblem_cache->emblem_keywords;
 for (; *keyword_cache_1 != '\0' && *keyword_cache_2 != '\0';) {
+ size_t length;
+
 compare_result = g_utf8_collate (keyword_cache_1, keyword_cache_2);
 if (compare_result != 0) {
 return compare_result;
@@ -3228,7 +3238,6 @@ is_valid_extension_segment (const char *segment, int segment_index)
 {
 gboolean result;
 gboolean has_letters;
- char c;
 int char_offset;
 switch (segment_index) {
 case 0:
@@ -3239,6 +3248,8 @@ is_valid_extension_segment (const char *segment, int segment_index)
 has_letters = FALSE;
 char_offset = 0;
 while (TRUE) {
+ char c;
+
 c = *(segment + char_offset);
 if (c == '\0') {
 result = has_letters;
@@ -3333,18 +3344,21 @@ compare_by_extension_segments (CajaFile *file_1, CajaFile *file_2)
 static gchar *
 caja_file_get_extension_as_string (CajaFile *file)
 {
- char *name;
 int rem_chars;
- int segment_index;
 char *segment;
- char *right_segment;
- char *result;
+
 if (!caja_file_is_directory (file)) {
+ char *name;
+
 name = caja_file_get_display_name (file);
 rem_chars = strlen (name);
 segment = prev_extension_segment (name + rem_chars, &rem_chars);
 
 if (rem_chars > 0 && is_valid_extension_segment (segment, 0)) {
+ int segment_index;
+ char *right_segment;
+ char *result;
+
 segment_index = 1;
 do {
 right_segment = segment;
@@ -3770,10 +3784,8 @@ GList *
 caja_file_get_metadata_list (CajaFile *file,
 const char *key)
 {
- GList *res;
 guint id;
 char **value;
- int i;
 
 g_return_val_if_fail (key != NULL, NULL);
 g_return_val_if_fail (key[0] != '\0', NULL);
@@ -3791,6 +3803,9 @@ caja_file_get_metadata_list (CajaFile *file,
 value = g_hash_table_lookup (file->details->metadata, GUINT_TO_POINTER (id));
 
 if (value) {
+ GList *res;
+ int i;
+
 res = NULL;
 for (i = 0; value[i] != NULL; i++) {
 res = g_list_prepend (res, g_strdup (value[i]));
@@ -4029,9 +4044,6 @@ caja_file_peek_display_name_collation_key (CajaFile *file)
 static const char *
 caja_file_peek_display_name (CajaFile *file)
 {
- const char *name;
- char *escaped_name;
-
 /*
 stefano-k: Imported 15_nautilus_file_peek_crash.patch from debian nautilus
 Date: Thu, 27 Jan 2011 10:22:10 +0000
@@ -4048,6 +4060,8 @@ caja_file_peek_display_name (CajaFile *file)
 /* Default to display name based on filename if its not set yet */
 
 if (file->details->display_name == NULL) {
+ const char *name;
+
 name = file->details->name;
 if (g_utf8_validate (name, -1, NULL)) {
 caja_file_set_display_name (file,
@@ -4055,6 +4069,8 @@ caja_file_peek_display_name (CajaFile *file)
 NULL,
 FALSE);
 } else {
+ char *escaped_name;
+
 escaped_name = g_uri_escape_string (name, G_URI_RESERVED_CHARS_ALLOWED_IN_PATH, TRUE);
 caja_file_set_display_name (file,
 escaped_name,
@@ -4191,11 +4207,12 @@ caja_file_get_drop_target_uri (CajaFile *file)
 {
 char *uri, *target_uri;
 GFile *location;
- CajaDesktopLink *link;
 
 g_return_val_if_fail (CAJA_IS_FILE (file), NULL);
 
 if (CAJA_IS_DESKTOP_ICON_FILE (file)) {
+ CajaDesktopLink *link;
+
 link = caja_desktop_icon_file_get_link (CAJA_DESKTOP_ICON_FILE (file));
 
 if (link != NULL) {
@@ -4245,12 +4262,13 @@ get_custom_icon_metadata_uri (CajaFile *file)
 {
 char *custom_icon_uri;
 char *uri;
- char *dir_uri;
 
 uri = caja_file_get_metadata (file, CAJA_METADATA_KEY_CUSTOM_ICON, NULL);
 if (uri != NULL &&
 caja_file_is_directory (file) &&
 is_uri_relative (uri)) {
+ char *dir_uri;
+
 dir_uri = caja_file_get_uri (file);
 custom_icon_uri = g_build_filename (dir_uri, uri, NULL);
 g_free (dir_uri);
@@ -4377,12 +4395,7 @@ caja_file_get_gicon (CajaFile *file,
 CajaFileIconFlags flags)
 {
 const char * const * names;
- const char *name;
- GPtrArray *prepend_array;
- GMount *mount;
 GIcon *icon, *mount_icon = NULL, *emblemed_icon;
- GEmblem *emblem;
- int i;
 gboolean is_folder = FALSE, is_preview = FALSE, is_inode_directory = FALSE;
 
 if (file == NULL) {
@@ -4400,6 +4413,8 @@ caja_file_get_gicon (CajaFile *file,
 /* fetch the mount icon here, we'll use it later */
 if (flags & CAJA_FILE_ICON_FLAGS_USE_MOUNT_ICON ||
 flags & CAJA_FILE_ICON_FLAGS_USE_MOUNT_ICON_AS_EMBLEM) {
+ GMount *mount;
+
 mount = caja_file_get_mount (file);
 
 if (mount != NULL) {
@@ -4416,10 +4431,15 @@ caja_file_get_gicon (CajaFile *file,
 ((flags & CAJA_FILE_ICON_FLAGS_IGNORE_VISITING) == 0 &&
 caja_file_has_open_window (file))) &&
 G_IS_THEMED_ICON (file->details->icon)) {
+ GPtrArray *prepend_array;
+ int i;
+
 names = g_themed_icon_get_names (G_THEMED_ICON (file->details->icon));
 prepend_array = g_ptr_array_new ();
 
 for (i = 0; names[i] != NULL; i++) {
+ const char *name;
+
 name = names[i];
 
 if (strcmp (name, "folder") == 0) {
@@ -4477,6 +4497,7 @@ caja_file_get_gicon (CajaFile *file,
 icon = mount_icon;
 } else if ((flags & CAJA_FILE_ICON_FLAGS_USE_MOUNT_ICON_AS_EMBLEM) &&
 mount_icon != NULL && !g_icon_equal (mount_icon, icon)) {
+ GEmblem *emblem;
 
 emblem = g_emblem_new (mount_icon);
 emblemed_icon = g_emblemed_icon_new (icon, emblem);
@@ -4521,8 +4542,7 @@ caja_file_get_icon (CajaFile *file,
 {
 CajaIconInfo *icon;
 GIcon *gicon;
- GdkPixbuf *raw_pixbuf, *scaled_pixbuf;
- int modified_size;
+ GdkPixbuf *scaled_pixbuf;
 
 if (file == NULL) {
 return NULL;
@@ -4551,6 +4571,7 @@ caja_file_get_icon (CajaFile *file,
 
 if (flags & CAJA_FILE_ICON_FLAGS_USE_THUMBNAILS &&
 caja_file_should_show_thumbnail (file)) {
+ int modified_size;
 
 if (flags & CAJA_FILE_ICON_FLAGS_FORCE_THUMBNAIL_SIZE) {
 modified_size = size * scale;
@@ -4561,6 +4582,7 @@ caja_file_get_icon (CajaFile *file,
 if (file->details->thumbnail) {
 int w, h, s;
 double thumb_scale;
+ GdkPixbuf *raw_pixbuf;
 
 raw_pixbuf = g_object_ref (file->details->thumbnail);
 
@@ -4807,7 +4829,6 @@ caja_file_fit_date_as_string (CajaFile *file,
 {
 time_t file_time_raw;
 const char **formats;
- const char *width_template;
 const char *format;
 char *date_string;
 gchar *result = NULL;
@@ -4857,6 +4878,8 @@ caja_file_fit_date_as_string (CajaFile *file,
 format = NULL;
 
 for (i = 0; ; i += 2) {
+ const char *width_template;
+
 width_template = (formats [i] ? _(formats [i]) : NULL);
 if (width_template == NULL) {
 /* no more formats left */
@@ -4928,11 +4951,11 @@ caja_file_fit_modified_date_as_string (CajaFile *file,
 static char *
 caja_file_get_trash_original_file_parent_as_string (CajaFile *file)
 {
- CajaFile *orig_file, *parent;
- GFile *location;
- char *filename;
-
 if (file->details->trash_orig_path != NULL) {
+ CajaFile *orig_file, *parent;
+ GFile *location;
+ char *filename;
+
 orig_file = caja_file_get_trash_original_file (file);
 parent = caja_file_get_parent (orig_file);
 location = caja_file_get_location (parent);
@@ -5418,9 +5441,10 @@ caja_file_set_permissions (CajaFile *file,
 gpointer callback_data)
 {
 GFileInfo *info;
- GError *error;
 
 if (!caja_file_can_set_permissions (file)) {
+ GError *error;
+
 /* Claim that something changed even if the permission change failed.
 * This makes it easier for some clients who see the "reverting"
 * to the old permissions as "changing back".
@@ -5783,7 +5807,7 @@ GList *
 caja_get_user_names (void)
 {
 GList *list;
- char *real_name, *name;
+ char *name;
 struct passwd *user;
 
 list = NULL;
@@ -5791,6 +5815,8 @@ caja_get_user_names (void)
 setpwent ();
 
 while ((user = getpwent ()) != NULL) {
+ char *real_name;
+
 real_name = get_real_name (user->pw_name, user->pw_gecos);
 if (real_name != NULL) {
 name = g_strconcat (user->pw_name, "\n", real_name, NULL);
@@ -5890,9 +5916,9 @@ static GList *
 caja_get_group_names_for_user (void)
 {
 GList *list;
- struct group *group;
 int count, i;
 gid_t gid_list[NGROUPS_MAX + 1];
+ struct group *group = NULL;
 
 
 list = NULL;
@@ -6739,9 +6765,9 @@ get_description (CajaFile *file)
 static char *
 update_description_for_link (CajaFile *file, char *string)
 {
- char *res;
-
 if (caja_file_is_symbolic_link (file)) {
+ char *res;
+
 g_assert (!caja_file_is_broken_symbolic_link (file));
 if (string == NULL) {
 return g_strdup (_("link"));
@@ -6936,9 +6962,9 @@ caja_file_get_emblem_pixbufs (CajaFile *file,
 {
 GList *icons, *l;
 GList *pixbufs;
- GIcon *icon;
 GdkPixbuf *pixbuf;
- CajaIconInfo *icon_info;
+ GIcon *icon = NULL;
+ CajaIconInfo *icon_info = NULL;
 
 icons = caja_file_get_emblem_icons (file, exclude);
 pixbufs = NULL;
@@ -6972,9 +6998,10 @@ static GList *
 sort_keyword_list_and_remove_duplicates (GList *keywords)
 {
 GList *p;
- GList *duplicate_link;
 
 if (keywords != NULL) {
+ GList *duplicate_link = NULL;
+
 keywords = eel_g_str_list_alphabetize (keywords);
 
 p = keywords;
@@ -7134,7 +7161,6 @@ get_fs_free_cb (GObject *source_object,
 gpointer user_data)
 {
 CajaDirectory *directory;
- CajaFile *file;
 guint64 free_space;
 GFileInfo *info;
 
@@ -7151,6 +7177,8 @@ get_fs_free_cb (GObject *source_object,
 }
 
 if (directory->details->free_space != free_space) {
+ CajaFile *file;
+
 directory->details->free_space = free_space;
 file = caja_directory_get_existing_corresponding_file (directory);
 if (file) {
@@ -7172,7 +7200,6 @@ char *
 caja_file_get_volume_free_space (CajaFile *file)
 {
 CajaDirectory *directory;
- GFile *location;
 char *res;
 time_t now;
 
@@ -7182,6 +7209,8 @@ caja_file_get_volume_free_space (CajaFile *file)
 /* Update first time and then every 2 seconds */
 if (directory->details->free_space_read == 0 ||
 (now - directory->details->free_space_read) > 2) {
+ GFile *location;
+
 directory->details->free_space_read = now;
 location = caja_file_get_location (file);
 g_file_query_filesystem_info_async (location,
@@ -7267,9 +7296,6 @@ caja_file_get_symbolic_link_target_path (CajaFile *file)
 char *
 caja_file_get_symbolic_link_target_uri (CajaFile *file)
 {
- GFile *location, *parent, *target;
- char *target_uri;
-
 if (!caja_file_is_symbolic_link (file)) {
 g_warning ("File has symlink target, but is not marked as symlink");
 }
@@ -7277,6 +7303,9 @@ caja_file_get_symbolic_link_target_uri (CajaFile *file)
 if (file->details->symlink_name == NULL) {
 return NULL;
 } else {
+ GFile *location, *parent, *target;
+ char *target_uri;
+
 target = NULL;
 
 location = caja_file_get_location (file);
@@ -7471,7 +7500,6 @@ caja_file_is_binary (CajaFile *file)
 }
 
 gboolean is_binary = FALSE;
- int c;
 int i = 0;
 FILE *fp;
 
@@ -7487,6 +7515,8 @@ caja_file_is_binary (CajaFile *file)
 }
 
 while (!feof (fp)) {
+ int c;
+
 if (i > 4096) {
 break;
 }
@@ -7594,12 +7624,13 @@ caja_file_get_filesystem_id (CajaFile *file)
 CajaFile *
 caja_file_get_trash_original_file (CajaFile *file)
 {
- GFile *location;
 CajaFile *original_file;
 
 original_file = NULL;
 
 if (file->details->trash_orig_path != NULL) {
+ GFile *location;
+
 location = g_file_new_for_path (file->details->trash_orig_path);
 original_file = caja_file_get (location);
 g_object_unref (location);
@@ -8300,7 +8331,7 @@ caja_file_list_call_when_ready (GList *file_list,
 {
 GList *l;
 FileListReadyData *data;
- CajaFile *file;
+ CajaFile *file = NULL;
 
 g_return_if_fail (file_list != NULL);
 
@@ -8330,7 +8361,6 @@ void
 caja_file_list_cancel_call_when_ready (CajaFileListHandle *handle)
 {
 GList *l;
- CajaFile *file;
 FileListReadyData *data;
 
 g_return_if_fail (handle != NULL);
@@ -8339,6 +8369,8 @@ caja_file_list_cancel_call_when_ready (CajaFileListHandle *handle)
 
 l = g_list_find (ready_data_list, data);
 if (l != NULL) {
+ CajaFile *file = NULL;
+
 for (l = data->remaining_files; l != NULL; l = l->next) {
 file = CAJA_FILE (l->data);
 
diff --git a/libcaja-private/caja-icon-canvas-item.c b/libcaja-private/caja-icon-canvas-item.c
index 2c49045b..e5273293 100644
--- a/libcaja-private/caja-icon-canvas-item.c
+++ b/libcaja-private/caja-icon-canvas-item.c
@@ -503,11 +503,12 @@ get_scaled_icon_size (CajaIconCanvasItem *item,
 gint *width,
 gint *height)
 {
- EelCanvas *canvas;
 GdkPixbuf *pixbuf = NULL;
 gint scale = 1;
 
 if (item != NULL) {
+ EelCanvas *canvas;
+
 canvas = EEL_CANVAS_ITEM (item)->canvas;
 scale = gtk_widget_get_scale_factor (GTK_WIDGET (canvas));
 pixbuf = item->details->pixbuf;
@@ -980,9 +981,7 @@ layout_get_size_for_layout (PangoLayout *layout,
 int height_for_entire_text,
 int *height_for_layout)
 {
- PangoLayoutIter *iter;
 PangoRectangle logical_rect;
- int i;
 
 /* only use the first max_layout_line_count lines for the gridded auto layout */
 if (pango_layout_get_line_count (layout) <= max_layout_line_count)
@@ -991,6 +990,9 @@ layout_get_size_for_layout (PangoLayout *layout,
 }
 else
 {
+ PangoLayoutIter *iter;
+ int i;
+
 *height_for_layout = 0;
 iter = pango_layout_get_iter (layout);
 /* VOODOO-TODO, determine number of lines based on the icon size for text besides icon.
@@ -1457,13 +1459,14 @@ static GdkPixbuf *
 get_knob_pixbuf (void)
 {
 GdkPixbuf *knob_pixbuf;
- char *knob_filename;
 
 knob_pixbuf = gtk_icon_theme_load_icon (gtk_icon_theme_get_default (),
 "stock-caja-knob",
 8, 0, NULL);
 if (!knob_pixbuf)
 {
+ char *knob_filename;
+
 knob_filename = caja_pixmap_file ("knob.png");
 knob_pixbuf = gdk_pixbuf_new_from_file (knob_filename, NULL);
 g_free (knob_filename);
@@ -1695,10 +1698,7 @@ static cairo_surface_t *
 real_map_surface (CajaIconCanvasItem *icon_item)
 {
 EelCanvas *canvas;
- char *audio_filename;
- GdkPixbuf *temp_pixbuf, *old_pixbuf, *audio_pixbuf;
- int emblem_size;
- GtkStyleContext *style;
+ GdkPixbuf *temp_pixbuf, *old_pixbuf;
 GdkRGBA color;
 GdkRGBA *c;
 cairo_surface_t *surface;
@@ -1726,6 +1726,10 @@ real_map_surface (CajaIconCanvasItem *icon_item)
 /* audio is the only kind of previewing right now, so this code isn't as general as it could be */
 if (icon_item->details->is_active)
 {
+ char *audio_filename;
+ GdkPixbuf *audio_pixbuf;
+ int emblem_size;
+
 emblem_size = caja_icon_get_emblem_size_for_icon_size (gdk_pixbuf_get_width (temp_pixbuf));
 /* Load the audio symbol. */
 audio_filename = caja_pixmap_file ("audio.svg");
@@ -1764,6 +1768,8 @@ real_map_surface (CajaIconCanvasItem *icon_item)
 if (icon_item->details->is_highlighted_for_selection
 || icon_item->details->is_highlighted_for_drop)
 {
+ GtkStyleContext *style;
+
 style = gtk_widget_get_style_context (GTK_WIDGET (canvas));
 
 if (gtk_widget_has_focus (GTK_WIDGET (canvas))) {
@@ -1828,8 +1834,6 @@ draw_embedded_text (CajaIconCanvasItem *item,
 int x, int y)
 {
 PangoLayout *layout;
- PangoContext *context;
- PangoFontDescription *desc;
 GtkWidget *widget;
 GtkStyleContext *style_context;
 
@@ -1848,6 +1852,9 @@ draw_embedded_text (CajaIconCanvasItem *item,
 }
 else
 {
+ PangoContext *context;
+ PangoFontDescription *desc;
+
 context = gtk_widget_get_pango_context (widget);
 layout = pango_layout_new (context);
 pango_layout_set_text (layout, item->details->embedded_text, -1);
@@ -1961,9 +1968,7 @@ create_label_layout (CajaIconCanvasItem *item,
 PangoFontDescription *desc;
 CajaIconContainer *container;
 EelCanvasItem *canvas_item;
- GString *str;
 char *zeroified_text;
- const char *p;
 
 canvas_item = EEL_CANVAS_ITEM (item);
 
@@ -1975,6 +1980,9 @@ create_label_layout (CajaIconCanvasItem *item,
 
 if (text != NULL)
 {
+ GString *str;
+ const char *p;
+
 str = g_string_new (NULL);
 
 for (p = text; *p != '\0'; p++)
@@ -2063,7 +2071,6 @@ static int
 caja_icon_canvas_item_event (EelCanvasItem *item, GdkEvent *event)
 {
 CajaIconCanvasItem *icon_item;
- GdkCursor *cursor;
 GdkWindow *cursor_window;
 
 icon_item = CAJA_ICON_CANVAS_ITEM (item);
@@ -2083,6 +2090,8 @@ caja_icon_canvas_item_event (EelCanvasItem *item, GdkEvent *event)
 /* show a hand cursor */
 if (in_single_click_mode ())
 {
+ GdkCursor *cursor;
+
 cursor = gdk_cursor_new_for_display (gdk_display_get_default(),
 GDK_HAND2);
 gdk_window_set_cursor (cursor_window, cursor);
@@ -2392,7 +2401,6 @@ caja_icon_canvas_item_ensure_bounds_up_to_date (CajaIconCanvasItem *icon_item)
 EelIRect text_rect, text_rect_for_layout, text_rect_for_entire_text;
 EelIRect total_rect, total_rect_for_layout, total_rect_for_entire_text;
 EelCanvasItem *item;
- double pixels_per_unit;
 gint width, height;
 EmblemLayout emblem_layout;
 GdkPixbuf *emblem_pixbuf;
@@ -2403,6 +2411,8 @@ caja_icon_canvas_item_ensure_bounds_up_to_date (CajaIconCanvasItem *icon_item)
 
 if (!details->bounds_cached)
 {
+ double pixels_per_unit;
+
 measure_label_text (icon_item);
 
 pixels_per_unit = EEL_CANVAS_ITEM (item)->canvas->pixels_per_unit;
@@ -3064,8 +3074,8 @@ caja_icon_canvas_item_accessible_get_index_in_parent (AtkObject *accessible)
 CajaIconCanvasItem *item;
 CajaIconContainer *container;
 GList *l;
- CajaIcon *icon;
 int i;
+ CajaIcon *icon = NULL;
 
 item = CAJA_ICON_CANVAS_ITEM (atk_gobject_accessible_get_object (ATK_GOBJECT_ACCESSIBLE (accessible)));
 if (!item)
@@ -3098,8 +3108,6 @@ static const gchar* caja_icon_canvas_item_accessible_get_image_description(AtkIm
 {
 CajaIconCanvasItemAccessiblePrivate* priv;
 CajaIconCanvasItem* item;
- CajaIcon* icon;
- CajaIconContainer* container;
 char* description;
 
 priv = GET_PRIV (image);
@@ -3110,6 +3118,9 @@ static const gchar* caja_icon_canvas_item_accessible_get_image_description(AtkIm
 }
 else
 {
+ CajaIcon* icon;
+ CajaIconContainer* container;
+
 item = CAJA_ICON_CANVAS_ITEM (atk_gobject_accessible_get_object (ATK_GOBJECT_ACCESSIBLE (image)));
 
 if (item == NULL)
diff --git a/libcaja-private/caja-icon-container.c b/libcaja-private/caja-icon-container.c
index ac78e451..dbb7050a 100644
--- a/libcaja-private/caja-icon-container.c
+++ b/libcaja-private/caja-icon-container.c
@@ -319,15 +319,8 @@ icon_set_position (CajaIcon *icon,
 double x, double y)
 {
 CajaIconContainer *container;
- double pixels_per_unit;
- int container_left, container_top, container_right, container_bottom;
 int x1, x2, y1, y2;
- int container_x, container_y, container_width, container_height;
 EelDRect icon_bounds;
- int item_width, item_height;
- int height_above, width_left;
- int min_x, max_x, min_y, max_y;
- int scale;
 
 if (icon->x == x && icon->y == y)
 {
@@ -343,6 +336,14 @@ icon_set_position (CajaIcon *icon,
 
 if (caja_icon_container_get_is_fixed_size (container))
 {
+ double pixels_per_unit;
+ int container_left, container_top, container_right, container_bottom;
+ int container_x, container_y, container_width, container_height;
+ int item_width, item_height;
+ int height_above, width_left;
+ int min_x, max_x, min_y, max_y;
+ int scale;
+
 /* FIXME: This should be:
 
 container_x = GTK_WIDGET (container)->allocation.x;
@@ -704,8 +705,8 @@ icon_get_row_and_column_bounds (CajaIconContainer *container,
 gboolean safety_pad)
 {
 GList *p;
- CajaIcon *one_icon;
 EelIRect one_bounds;
+ CajaIcon *one_icon = NULL;
 
 item_get_canvas_bounds (EEL_CANVAS_ITEM (icon->item), bounds, safety_pad);
 
@@ -1262,11 +1263,11 @@ lay_down_one_line (CajaIconContainer *container,
 gboolean whole_text)
 {
 GList *p;
- CajaIcon *icon;
 double x, y_offset;
 IconPositions *position;
 int i;
 gboolean is_rtl;
+ CajaIcon *icon = NULL;
 
 is_rtl = caja_icon_container_is_layout_rtl (container);
 
@@ -1310,11 +1311,11 @@ lay_down_one_column (CajaIconContainer *container,
 GArray *positions)
 {
 GList *p;
- CajaIcon *icon;
 double y;
- IconPositions *position;
 int i;
 gboolean is_rtl;
+ IconPositions *position = NULL;
+ CajaIcon *icon = NULL;
 
 is_rtl = caja_icon_container_is_layout_rtl (container);
 
@@ -1346,21 +1347,19 @@ lay_down_icons_horizontal (CajaIconContainer *container,
 GList *p, *line_start;
 CajaIcon *icon;
 double canvas_width, y;
- GArray *positions;
- IconPositions *position;
 EelDRect bounds;
 EelDRect icon_bounds;
 EelDRect text_bounds;
 double max_height_above, max_height_below;
- double height_above, height_below;
 double line_width;
 gboolean gridded_layout;
 double grid_width;
 double max_text_width, max_icon_width;
 int icon_width;
 int i;
- int num_columns;
 GtkAllocation allocation;
+ GArray *positions;
+ IconPositions *position = NULL;
 
 g_assert (CAJA_IS_ICON_CONTAINER (container));
 
@@ -1394,6 +1393,8 @@ lay_down_icons_horizontal (CajaIconContainer *container,
 }
 else
 {
+ int num_columns;
+
 num_columns = floor(canvas_width / STANDARD_ICON_GRID_WIDTH);
 num_columns = fmax(num_columns, 1);
 /* Minimum of one column */
@@ -1412,6 +1413,8 @@ lay_down_icons_horizontal (CajaIconContainer *container,
 max_height_below = 0;
 for (p = icons; p != NULL; p = p->next)
 {
+ double height_above, height_below;
+
 icon = p->data;
 
 /* Assume it's only one level hierarchy to avoid costly affine calculations */
@@ -1539,11 +1542,11 @@ get_max_icon_dimensions (GList *icon_start,
 double *max_text_height,
 double *max_bounds_height)
 {
- CajaIcon *icon;
 EelDRect icon_bounds;
 EelDRect text_bounds;
 GList *p;
 double y1, y2;
+ CajaIcon *icon = NULL;
 
 *max_icon_width = *max_text_width = 0.0;
 *max_icon_height = *max_text_height = 0.0;
@@ -1576,7 +1579,6 @@ lay_down_icons_vertical (CajaIconContainer *container,
 double start_y)
 {
 GList *p, *line_start;
- CajaIcon *icon;
 double x, canvas_height;
 GArray *positions;
 IconPositions *position;
@@ -1596,9 +1598,10 @@ lay_down_icons_vertical (CajaIconContainer *container,
 
 double max_text_width, max_icon_width;
 double max_text_height, max_icon_height;
- int height;
 int i;
 
+ CajaIcon *icon = NULL;
+
 g_assert (CAJA_IS_ICON_CONTAINER (container));
 g_assert (container->details->label_position == CAJA_ICON_LABEL_POSITION_BESIDE);
 
@@ -1637,6 +1640,8 @@ lay_down_icons_vertical (CajaIconContainer *container,
 
 for (p = icons; p != NULL; p = p->next)
 {
+ int height;
+
 icon = p->data;
 
 /* If this icon doesn't fit, it's time to lay out the column that's queued up. */
@@ -2088,8 +2093,7 @@ static void
 caja_icon_container_set_rtl_positions (CajaIconContainer *container)
 {
 GList *l;
- CajaIcon *icon;
- double x;
+ CajaIcon *icon = NULL;
 
 if (!container->details->icons)
 {
@@ -2098,6 +2102,8 @@ caja_icon_container_set_rtl_positions (CajaIconContainer *container)
 
 for (l = container->details->icons; l != NULL; l = l->next)
 {
+ double x;
+
 icon = l->data;
 x = get_mirror_x_position (container, icon, icon->saved_ltr_x);
 icon_set_position (icon, x, icon->y);
@@ -2110,7 +2116,7 @@ lay_down_icons_vertical_desktop (CajaIconContainer *container, GList *icons)
 GList *p, *placed_icons, *unplaced_icons;
 int total, new_length, placed;
 CajaIcon *icon;
- int height, max_width, column_width, icon_width, icon_height;
+ int height;
 int x, y, x1, x2, y1, y2;
 EelDRect icon_rect;
 GtkAllocation allocation;
@@ -2193,6 +2199,7 @@ lay_down_icons_vertical_desktop (CajaIconContainer *container, GList *icons)
 
 while (icons != NULL)
 {
+ int max_width, column_width, icon_height;
 int center_x;
 int baseline;
 int icon_height_for_bound_check;
@@ -2207,6 +2214,8 @@ lay_down_icons_vertical_desktop (CajaIconContainer *container, GList *icons)
 /* Calculate max width for column */
 for (p = icons; p != NULL; p = p->next)
 {
+ int icon_width;
+
 icon = p->data;
 
 icon_get_bounding_box (icon, &x1, &y1, &x2, &y2,
@@ -2404,12 +2413,12 @@ static void
 reload_icon_positions (CajaIconContainer *container)
 {
 GList *p, *no_position_icons;
- CajaIcon *icon;
 gboolean have_stored_position;
 CajaIconPosition position;
 EelDRect bounds;
 double bottom;
 EelCanvasItem *item;
+ CajaIcon *icon = NULL;
 
 g_assert (!container->details->auto_layout);
 
@@ -2474,7 +2483,7 @@ static void
 invalidate_label_sizes (CajaIconContainer *container)
 {
 GList *p;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 for (p = container->details->icons; p != NULL; p = p->next)
 {
@@ -2489,7 +2498,7 @@ static void
 invalidate_labels (CajaIconContainer *container)
 {
 GList *p;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 for (p = container->details->icons; p != NULL; p = p->next)
 {
@@ -2507,7 +2516,7 @@ select_range (CajaIconContainer *container,
 {
 gboolean selection_changed;
 GList *p;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 CajaIcon *unmatched_icon;
 gboolean select;
 
@@ -2560,7 +2569,7 @@ select_one_unselect_others (CajaIconContainer *container,
 {
 gboolean selection_changed;
 GList *p;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 selection_changed = FALSE;
 
@@ -2667,9 +2676,9 @@ rubberband_select (CajaIconContainer *container,
 {
 GList *p;
 gboolean selection_changed, is_in, canvas_rect_calculated;
- CajaIcon *icon;
 EelIRect canvas_rect;
 EelCanvas *canvas;
+ CajaIcon *icon = NULL;
 
 selection_changed = FALSE;
 canvas_rect_calculated = FALSE;
@@ -6609,7 +6618,7 @@ static void
 update_selected (CajaIconContainer *container)
 {
 GList *node;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 for (node = container->details->icons; node != NULL; node = node->next)
 {
@@ -6657,7 +6666,6 @@ get_text_ellipsis_limit_for_zoom (char **strs,
 const char *zoom_level,
 int *limit)
 {
- char **p;
 char *str;
 gboolean success;
 
@@ -6677,6 +6685,8 @@ get_text_ellipsis_limit_for_zoom (char **strs,
 
 if (strs != NULL)
 {
+ char **p;
+
 for (p = strs; *p != NULL; p++)
 {
 if (sscanf (*p, str, limit))
@@ -6998,8 +7008,8 @@ void
 caja_icon_container_clear (CajaIconContainer *container)
 {
 CajaIconContainerDetails *details;
- CajaIcon *icon;
 GList *p;
+ CajaIcon *icon = NULL;
 
 g_return_if_fail (CAJA_IS_ICON_CONTAINER (container));
 
@@ -7146,10 +7156,10 @@ caja_icon_container_scroll_to_icon (CajaIconContainer *container,
 CajaIconData *data)
 {
 GList *l;
- CajaIcon *icon;
 GtkAdjustment *hadj, *vadj;
 EelIRect bounds;
 GtkAllocation allocation;
+ CajaIcon *icon = NULL;
 
 hadj = gtk_scrollable_get_hadjustment (GTK_SCROLLABLE (container));
 vadj = gtk_scrollable_get_vadjustment (GTK_SCROLLABLE (container));
@@ -7467,9 +7477,9 @@ caja_icon_container_update_visible_icons (CajaIconContainer *container)
 double min_x, max_x;
 double x0, y0, x1, y1;
 GList *node;
- CajaIcon *icon;
 gboolean visible;
 GtkAllocation allocation;
+ CajaIcon *icon = NULL;
 
 hadj = gtk_scrollable_get_hadjustment (GTK_SCROLLABLE (container));
 vadj = gtk_scrollable_get_vadjustment (GTK_SCROLLABLE (container));
@@ -8035,7 +8045,7 @@ void
 caja_icon_container_request_update_all (CajaIconContainer *container)
 {
 GList *node;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 g_return_if_fail (CAJA_IS_ICON_CONTAINER (container));
 
@@ -8205,7 +8215,7 @@ caja_icon_container_select_all (CajaIconContainer *container)
 {
 gboolean selection_changed;
 GList *p;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 g_return_if_fail (CAJA_IS_ICON_CONTAINER (container));
 
@@ -8240,7 +8250,7 @@ caja_icon_container_set_selection (CajaIconContainer *container,
 gboolean selection_changed;
 GHashTable *hash;
 GList *p;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 g_return_if_fail (CAJA_IS_ICON_CONTAINER (container));
 
@@ -8282,7 +8292,7 @@ caja_icon_container_select_list_unselect_others (CajaIconContainer *container,
 gboolean selection_changed;
 GHashTable *hash;
 GList *p;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 g_return_if_fail (CAJA_IS_ICON_CONTAINER (container));
 
@@ -8375,8 +8385,8 @@ static CajaIcon *
 get_nth_selected_icon (CajaIconContainer *container, int index)
 {
 GList *p;
- CajaIcon *icon;
 int selection_count;
+ CajaIcon *icon = NULL;
 
 g_assert (index > 0);
 
@@ -8412,7 +8422,7 @@ static gboolean
 all_selected (CajaIconContainer *container)
 {
 GList *p;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 for (p = container->details->icons; p != NULL; p = p->next)
 {
@@ -8508,7 +8518,7 @@ gboolean
 caja_icon_container_is_stretched (CajaIconContainer *container)
 {
 GList *p;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 for (p = container->details->icons; p != NULL; p = p->next)
 {
@@ -8531,7 +8541,7 @@ void
 caja_icon_container_unstretch (CajaIconContainer *container)
 {
 GList *p;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 for (p = container->details->icons; p != NULL; p = p->next)
 {
@@ -8936,7 +8946,6 @@ caja_icon_container_start_renaming_selected_item (CajaIconContainer *container,
 CajaIcon *icon;
 EelDRect icon_rect;
 EelDRect text_rect;
- PangoContext *context;
 PangoFontDescription *desc;
 const char *editable_text;
 int x, y, width;
@@ -9015,6 +9024,8 @@ caja_icon_container_start_renaming_selected_item (CajaIconContainer *container,
 }
 else
 {
+ PangoContext *context;
+
 context = gtk_widget_get_pango_context (GTK_WIDGET (container));
 desc = pango_font_description_copy (pango_context_get_font_description (context));
 pango_font_description_set_size (desc,
@@ -9093,8 +9104,6 @@ static void
 end_renaming_mode (CajaIconContainer *container, gboolean commit)
 {
 CajaIcon *icon;
- const char *changed_text;
- AtkObject *accessible_icon;
 
 set_pending_icon_to_rename (container, NULL);
 
@@ -9119,10 +9128,14 @@ end_renaming_mode (CajaIconContainer *container, gboolean commit)
 
 if (commit)
 {
+ const char *changed_text;
+
 /* Verify that text has been modified before signalling change. */
 changed_text = eel_editable_label_get_text (EEL_EDITABLE_LABEL (container->details->rename_widget));
 if (strcmp (container->details->original_text, changed_text) != 0)
 {
+ AtkObject *accessible_icon;
+
 g_signal_emit (container,
 signals[ICON_TEXT_CHANGED], 0,
 icon->data,
@@ -9165,9 +9178,9 @@ gboolean
 caja_icon_container_has_stored_icon_positions (CajaIconContainer *container)
 {
 GList *p;
- CajaIcon *icon;
 gboolean have_stored_position;
 CajaIconPosition position;
+ CajaIcon *icon = NULL;
 
 for (p = container->details->icons; p != NULL; p = p->next)
 {
@@ -9403,8 +9416,8 @@ caja_icon_container_set_highlighted_for_clipboard (CajaIconContainer *container,
 GList *clipboard_icon_data)
 {
 GList *l;
- CajaIcon *icon;
 gboolean highlighted_for_clipboard;
+ CajaIcon *icon = NULL;
 
 g_return_if_fail (CAJA_IS_ICON_CONTAINER (container));
 
@@ -9554,7 +9567,7 @@ caja_icon_container_accessible_update_selection (AtkObject *accessible)
 CajaIconContainer *container;
 CajaIconContainerAccessiblePrivate *priv;
 GList *l;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 container = CAJA_ICON_CONTAINER (gtk_accessible_get_widget (GTK_ACCESSIBLE (accessible)));
 
@@ -9592,9 +9605,6 @@ caja_icon_container_accessible_icon_added_cb (CajaIconContainer *container,
 gpointer data)
 {
 CajaIcon *icon;
- AtkObject *atk_parent;
- AtkObject *atk_child;
- int index;
 
 // We don't want to emit children_changed signals during any type of load.
 if (container->details->is_loading || container->details->is_populating_container)
@@ -9603,6 +9613,10 @@ caja_icon_container_accessible_icon_added_cb (CajaIconContainer *container,
 icon = g_hash_table_lookup (container->details->icon_set, icon_data);
 if (icon)
 {
+ AtkObject *atk_parent;
+ AtkObject *atk_child;
+ int index;
+
 atk_parent = ATK_OBJECT (data);
 atk_child = atk_gobject_accessible_for_object
 (G_OBJECT (icon->item));
@@ -9619,13 +9633,14 @@ caja_icon_container_accessible_icon_removed_cb (CajaIconContainer *container,
 gpointer data)
 {
 CajaIcon *icon;
- AtkObject *atk_parent;
- AtkObject *atk_child;
- int index;
 
 icon = g_hash_table_lookup (container->details->icon_set, icon_data);
 if (icon)
 {
+ AtkObject *atk_parent;
+ AtkObject *atk_child;
+ int index;
+
 atk_parent = ATK_OBJECT (data);
 atk_child = atk_gobject_accessible_for_object
 (G_OBJECT (icon->item));
@@ -9650,7 +9665,6 @@ caja_icon_container_accessible_add_selection (AtkSelection *accessible,
 {
 GtkWidget *widget;
 CajaIconContainer *container;
- GList *selection;
 CajaIcon *icon;
 
 widget = gtk_accessible_get_widget (GTK_ACCESSIBLE (accessible));
@@ -9664,6 +9678,8 @@ caja_icon_container_accessible_add_selection (AtkSelection *accessible,
 icon = g_list_nth_data (container->details->icons, i);
 if (icon)
 {
+ GList *selection;
+
 selection = caja_icon_container_get_selection (container);
 selection = g_list_prepend (selection,
 icon->data);
@@ -9699,7 +9715,6 @@ static AtkObject *
 caja_icon_container_accessible_ref_selection (AtkSelection *accessible,
 int i)
 {
- AtkObject *atk_object;
 CajaIconContainerAccessiblePrivate *priv;
 CajaIcon *icon;
 
@@ -9709,6 +9724,8 @@ caja_icon_container_accessible_ref_selection (AtkSelection *accessible,
 icon = g_list_nth_data (priv->selection, i);
 if (icon)
 {
+ AtkObject *atk_object;
+
 atk_object = atk_gobject_accessible_for_object (G_OBJECT (icon->item));
 if (atk_object)
 {
@@ -9763,7 +9780,6 @@ caja_icon_container_accessible_remove_selection (AtkSelection *accessible,
 {
 CajaIconContainer *container;
 CajaIconContainerAccessiblePrivate *priv;
- GList *selection;
 CajaIcon *icon;
 GtkWidget *widget;
 
@@ -9781,6 +9797,8 @@ caja_icon_container_accessible_remove_selection (AtkSelection *accessible,
 icon = g_list_nth_data (priv->selection, i);
 if (icon)
 {
+ GList *selection;
+
 selection = caja_icon_container_get_selection (container);
 selection = g_list_remove (selection, icon->data);
 caja_icon_container_set_selection (container, selection);
@@ -9912,7 +9930,6 @@ static void
 caja_icon_container_accessible_initialize (AtkObject *accessible,
 gpointer data)
 {
- CajaIconContainer *container;
 CajaIconContainerAccessiblePrivate *priv;
 
 if (ATK_OBJECT_CLASS (accessible_parent_class)->initialize)
@@ -9927,6 +9944,8 @@ caja_icon_container_accessible_initialize (AtkObject *accessible,
 
 if (GTK_IS_ACCESSIBLE (accessible))
 {
+ CajaIconContainer *container;
+
 caja_icon_container_accessible_update_selection
 (ATK_OBJECT (accessible));
 
@@ -10115,9 +10134,9 @@ caja_icon_container_begin_loading (CajaIconContainer *container)
 static void
 store_layout_timestamps_now (CajaIconContainer *container)
 {
- CajaIcon *icon;
 GList *p;
 gboolean dummy;
+ CajaIcon *icon = NULL;
 
 container->details->layout_timestamp = time (NULL);
 g_signal_emit (container,
diff --git a/libcaja-private/caja-icon-dnd.c b/libcaja-private/caja-icon-dnd.c
index 40f57cc1..a41271e0 100644
--- a/libcaja-private/caja-icon-dnd.c
+++ b/libcaja-private/caja-icon-dnd.c
@@ -282,7 +282,7 @@ caja_icon_container_each_selected_icon (CajaIconContainer *container,
 gboolean (*each_function) (CajaIcon *, gpointer), gpointer data)
 {
 GList *p;
- CajaIcon *icon;
+ CajaIcon *icon = NULL;
 
 for (p = container->details->icons; p != NULL; p = p->next)
 {
@@ -424,7 +424,6 @@ get_direct_save_filename (GdkDragContext *context)
 static void
 set_direct_save_uri (GtkWidget *widget, GdkDragContext *context, CajaDragInfo *drag_info, int x, int y)
 {
- GFile *base, *child;
 char *filename, *drop_target;
 gchar *uri;
 
@@ -445,6 +444,8 @@ set_direct_save_uri (GtkWidget *widget, GdkDragContext *context, CajaDragInfo *d
 
 if (filename != NULL && drop_target != NULL)
 {
+ GFile *base, *child;
+
 /* Resolve relative path */
 base = g_file_new_for_uri (drop_target);
 child = g_file_get_child (base, filename);
@@ -949,12 +950,12 @@ handle_local_move (CajaIconContainer *container,
 double world_x, double world_y)
 {
 GList *moved_icons, *p;
- CajaDragSelectionItem *item;
- CajaIcon *icon;
 CajaFile *file;
 char screen_string[32];
 GdkScreen *screen;
 time_t now;
+ CajaDragSelectionItem *item = NULL;
+ CajaIcon *icon = NULL;
 
 if (container->details->auto_layout)
 {
@@ -1022,7 +1023,6 @@ handle_nonlocal_move (CajaIconContainer *container,
 GList *source_uris, *p;
 GArray *source_item_locations;
 gboolean free_target_uri, is_rtl;
- int index, item_x;
 GtkAllocation allocation;
 
 if (container->details->dnd_info->drag_info.selection_list == NULL)
@@ -1043,6 +1043,8 @@ handle_nonlocal_move (CajaIconContainer *container,
 source_item_locations = g_array_new (FALSE, TRUE, sizeof (GdkPoint));
 if (!icon_hit)
 {
+ int index;
+
 /* Drop onto a container. Pass along the item points to allow placing
 * the items in their same relative positions in the new container.
 */
@@ -1052,6 +1054,8 @@ handle_nonlocal_move (CajaIconContainer *container,
 for (index = 0, p = container->details->dnd_info->drag_info.selection_list;
 p != NULL; index++, p = p->next)
 {
+ int item_x;
+
 item_x = ((CajaDragSelectionItem *)p->data)->icon_x;
 if (is_rtl)
 item_x = -item_x - ((CajaDragSelectionItem *)p->data)->icon_width;
@@ -1101,9 +1105,6 @@ caja_icon_container_find_drop_target (CajaIconContainer *container,
 {
 CajaIcon *drop_target_icon;
 double world_x, world_y;
- CajaFile *file;
- char *icon_uri;
- char *container_uri;
 
 if (icon_hit)
 {
@@ -1127,9 +1128,13 @@ caja_icon_container_find_drop_target (CajaIconContainer *container,
 drop_target_icon = caja_icon_container_item_at (container, world_x, world_y);
 if (drop_target_icon != NULL)
 {
+ char *icon_uri;
+
 icon_uri = caja_icon_container_get_icon_uri (container, drop_target_icon);
 if (icon_uri != NULL)
 {
+ CajaFile *file;
+
 file = caja_file_get_by_uri (icon_uri);
 
 if (!caja_drag_can_accept_info (file,
@@ -1149,6 +1154,8 @@ caja_icon_container_find_drop_target (CajaIconContainer *container,
 
 if (drop_target_icon == NULL)
 {
+ char *container_uri;
+
 if (icon_hit)
 {
 *icon_hit = FALSE;
@@ -1225,7 +1232,6 @@ caja_icon_container_receive_dropped_icons (CajaIconContainer *container,
 double world_x, world_y;
 gboolean icon_hit;
 GdkDragAction action, real_action;
- CajaDragSelectionItem *selected_item;
 
 drop_target = NULL;
 
@@ -1259,6 +1265,8 @@ caja_icon_container_receive_dropped_icons (CajaIconContainer *container,
 
 if (real_action == (GdkDragAction) CAJA_DND_ACTION_SET_AS_BACKGROUND)
 {
+ CajaDragSelectionItem *selected_item;
+
 selected_item = container->details->dnd_info->drag_info.selection_list->data;
 eel_background_set_dropped_image (eel_get_widget_background (GTK_WIDGET (container)),
 real_action, selected_item->uri);
@@ -1406,9 +1414,7 @@ caja_icon_dnd_update_drop_target (CajaIconContainer *container,
 int x, int y)
 {
 CajaIcon *icon;
- CajaFile *file;
 double world_x, world_y;
- char *uri;
 
 g_assert (CAJA_IS_ICON_CONTAINER (container));
 
@@ -1426,6 +1432,9 @@ caja_icon_dnd_update_drop_target (CajaIconContainer *container,
 /* Find if target icon accepts our drop. */
 if (icon != NULL && (container->details->dnd_info->drag_info.data_type != CAJA_ICON_DND_KEYWORD))
 {
+ CajaFile *file;
+ char *uri;
+
 uri = caja_icon_container_get_icon_uri (container, icon);
 file = caja_file_get_by_uri (uri);
 g_free (uri);
@@ -1751,10 +1760,6 @@ drag_data_received_callback (GtkWidget *widget,
 gpointer user_data)
 {
 CajaDragInfo *drag_info;
- EelBackground *background;
- char *tmp;
- const char *tmp_raw;
- int length;
 gboolean success;
 
 drag_info = &(CAJA_ICON_CONTAINER (widget)->details->dnd_info->drag_info);
@@ -1801,6 +1806,10 @@ drag_data_received_callback (GtkWidget *widget,
 */
 if (drag_info->drop_occured)
 {
+ EelBackground *background;
+ char *tmp;
+ const char *tmp_raw;
+ int length;
 
 success = FALSE;
 switch (info)
diff --git a/libcaja-private/caja-icon-info.c b/libcaja-private/caja-icon-info.c
index b3f33735..4ad58e45 100644
--- a/libcaja-private/caja-icon-info.c
+++ b/libcaja-private/caja-icon-info.c
@@ -148,7 +148,7 @@ caja_icon_info_new_for_icon_info (GtkIconInfo *icon_info,
 GdkPoint *points;
 gint n_points;
 const char *filename;
- char *basename, *p;
+ char *basename;
 
 icon = g_object_new (CAJA_TYPE_ICON_INFO, NULL);
 
@@ -168,6 +168,8 @@ caja_icon_info_new_for_icon_info (GtkIconInfo *icon_info,
 filename = gtk_icon_info_get_filename (icon_info);
 if (filename != NULL)
 {
+ char *p;
+
 basename = g_path_get_basename (filename);
 p = strrchr (basename, '.');
 if (p)
diff --git a/libcaja-private/caja-keep-last-vertical-box.c b/libcaja-private/caja-keep-last-vertical-box.c
index b158c81a..65b6541d 100644
--- a/libcaja-private/caja-keep-last-vertical-box.c
+++ b/libcaja-private/caja-keep-last-vertical-box.c
@@ -89,7 +89,6 @@ static void
 caja_keep_last_vertical_box_size_allocate (GtkWidget *widget,
 GtkAllocation *allocation)
 {
- GtkWidget *last_child, *child;
 GList *children, *l;
 GtkAllocation last_child_allocation, child_allocation, tiny_allocation;
 
@@ -103,6 +102,8 @@ caja_keep_last_vertical_box_size_allocate (GtkWidget *widget,
 
 if (l != NULL)
 {
+ GtkWidget *last_child;
+
 last_child = l->data;
 l = l->prev;
 
@@ -114,6 +115,7 @@ caja_keep_last_vertical_box_size_allocate (GtkWidget *widget,
 if (last_child_allocation.y + last_child_allocation.height >
 allocation->y + allocation->height)
 {
+ GtkWidget *child = NULL;
 
 while (l != NULL)
 {
diff --git a/libcaja-private/caja-link.c b/libcaja-private/caja-link.c
index 8c2902b5..232ee213 100644
--- a/libcaja-private/caja-link.c
+++ b/libcaja-private/caja-link.c
@@ -200,7 +200,7 @@ caja_link_local_create (const char *directory_uri,
 gboolean unique_filename)
 {
 char *real_directory_uri;
- char *uri, *contents;
+ char *contents;
 GFile *file;
 GList dummy_list;
 CajaFileChangesQueuePosition item;
@@ -227,6 +227,8 @@ caja_link_local_create (const char *directory_uri,
 
 if (unique_filename)
 {
+ char *uri;
+
 uri = caja_ensure_unique_file_name (real_directory_uri,
 base_name, ".desktop");
 if (uri == NULL)
@@ -481,7 +483,7 @@ caja_link_get_link_name_from_desktop (GKeyFile *key_file)
 static char *
 caja_link_get_link_icon_from_desktop (GKeyFile *key_file)
 {
- char *icon_uri, *icon, *p, *type;
+ char *icon_uri, *icon, *type;
 
 icon_uri = g_key_file_get_string (key_file, MAIN_GROUP, "X-Caja-Icon", NULL);
 if (icon_uri != NULL)
@@ -494,6 +496,8 @@ caja_link_get_link_icon_from_desktop (GKeyFile *key_file)
 {
 if (!g_path_is_absolute (icon))
 {
+ char *p;
+
 /* Strip out any extension on non-filename icons. Old desktop files may have this */
 p = strchr (icon, '.');
 /* Only strip known icon extensions */
diff --git a/libcaja-private/caja-metadata.c b/libcaja-private/caja-metadata.c
index 2b12962f..b854215d 100644
--- a/libcaja-private/caja-metadata.c
+++ b/libcaja-private/caja-metadata.c
@@ -66,10 +66,11 @@ guint
 caja_metadata_get_id (const char *metadata)
 {
 static GHashTable *hash;
- int i;
 
 if (hash == NULL)
 {
+ int i;
+
 hash = g_hash_table_new (g_str_hash, g_str_equal);
 for (i = 0; used_metadata_names[i] != NULL; i++)
 g_hash_table_insert (hash,
diff --git a/libcaja-private/caja-mime-actions.c b/libcaja-private/caja-mime-actions.c
index e9edd2ba..83ca70b9 100644
--- a/libcaja-private/caja-mime-actions.c
+++ b/libcaja-private/caja-mime-actions.c
@@ -134,7 +134,7 @@ static GList *
 get_file_list_for_launch_locations (GList *locations)
 {
 GList *files, *l;
- LaunchLocation *location;
+ LaunchLocation *location = NULL;
 
 files = NULL;
 for (l = locations; l != NULL; l = l->next)
@@ -183,8 +183,8 @@ static LaunchLocation *
 find_launch_location_for_file (GList *list,
 CajaFile *file)
 {
- LaunchLocation *location;
 GList *l;
+ LaunchLocation *location = NULL;
 
 for (l = list; l != NULL; l = l->next)
 {
@@ -240,12 +240,13 @@ static GList*
 filter_caja_handler (GList *apps)
 {
 GList *l, *next;
- GAppInfo *application;
- const char *id;
+ GAppInfo *application = NULL;
 
 l = apps;
 while (l != NULL)
 {
+ const char *id;
+
 application = (GAppInfo *) l->data;
 next = l->next;
 
@@ -268,7 +269,7 @@ static GList*
 filter_non_uri_apps (GList *apps)
 {
 GList *l, *next;
- GAppInfo *app;
+ GAppInfo *app = NULL;
 
 for (l = apps; l != NULL; l = next)
 {
@@ -337,7 +338,6 @@ caja_mime_get_default_application_for_file (CajaFile *file)
 {
 GAppInfo *app;
 char *mime_type;
- char *uri_scheme;
 
 if (!caja_mime_actions_check_if_required_attributes_ready (file))
 {
@@ -350,6 +350,8 @@ caja_mime_get_default_application_for_file (CajaFile *file)
 
 if (app == NULL)
 {
+ char *uri_scheme;
+
 uri_scheme = caja_file_get_uri_scheme (file);
 if (uri_scheme != NULL)
 {
@@ -447,7 +449,6 @@ caja_mime_get_applications_for_file (CajaFile *file)
 char *mime_type;
 char *uri_scheme;
 GList *result;
- GAppInfo *uri_handler;
 
 if (!caja_mime_actions_check_if_required_attributes_ready (file))
 {
@@ -459,6 +460,8 @@ caja_mime_get_applications_for_file (CajaFile *file)
 uri_scheme = caja_file_get_uri_scheme (file);
 if (uri_scheme != NULL)
 {
+ GAppInfo *uri_handler;
+
 uri_handler = g_app_info_get_default_for_uri_scheme (uri_scheme);
 if (uri_handler)
 {
@@ -486,7 +489,6 @@ caja_mime_has_any_applications_for_file (CajaFile *file)
 char *mime_type;
 gboolean result;
 char *uri_scheme;
- GAppInfo *uri_handler;
 
 mime_type = caja_file_get_mime_type (file);
 
@@ -495,6 +497,8 @@ caja_mime_has_any_applications_for_file (CajaFile *file)
 uri_scheme = caja_file_get_uri_scheme (file);
 if (uri_scheme != NULL)
 {
+ GAppInfo *uri_handler;
+
 uri_handler = g_app_info_get_default_for_uri_scheme (uri_scheme);
 if (uri_handler)
 {
@@ -529,8 +533,8 @@ GAppInfo *
 caja_mime_get_default_application_for_files (GList *files)
 {
 GList *l, *sorted_files;
- CajaFile *file;
 GAppInfo *app, *one_app;
+ CajaFile *file = NULL;
 
 g_assert (files != NULL);
 
@@ -588,8 +592,8 @@ intersect_application_lists (GList *a,
 {
 GList *l, *m;
 GList *ret;
- GAppInfo *a_app, *b_app;
- int cmp;
+ GAppInfo *a_app = NULL;
+ GAppInfo *b_app = NULL;
 
 ret = NULL;
 
@@ -598,6 +602,8 @@ intersect_application_lists (GList *a,
 
 while (l != NULL && m != NULL)
 {
+ int cmp;
+
 a_app = (GAppInfo *) l->data;
 b_app = (GAppInfo *) m->data;
 
@@ -634,8 +640,8 @@ GList *
 caja_mime_get_applications_for_files (GList *files)
 {
 GList *l, *sorted_files;
- CajaFile *file;
 GList *one_ret, *ret;
+ CajaFile *file = NULL;
 
 g_assert (files != NULL);
 
@@ -979,10 +985,10 @@ make_activation_parameters (GList *uris,
 GList **unhandled_uris)
 {
 GList *ret, *l, *app_uris;
- CajaFile *file;
- GAppInfo *app, *old_app;
 GHashTable *app_table;
- char *uri;
+ GAppInfo *old_app;
+ GAppInfo *app = NULL;
+ CajaFile *file = NULL;
 
 ret = NULL;
 *unhandled_uris = NULL;
@@ -995,6 +1001,8 @@ make_activation_parameters (GList *uris,
 
 for (l = uris; l != NULL; l = l->next)
 {
+ char *uri;
+
 uri = l->data;
 file = caja_file_get_by_uri (uri);
 
@@ -1246,11 +1254,12 @@ get_application_no_mime_type_handler_message (CajaFile *file, char *uri)
 char *uri_for_display;
 char *nice_uri;
 char *error_message;
- GFile *location;
 
 /* For local files, we want to use filename if possible */
 if (caja_file_is_local (file))
 {
+ GFile *location;
+
 location = caja_file_get_location (file);
 nice_uri = g_file_get_parse_name (location);
 g_object_unref (location);
@@ -1478,13 +1487,13 @@ application_unhandled_file_install (GtkDialog *dialog,
 gint response_id,
 ActivateParametersInstall *parameters_install)
 {
- char *mime_type;
-
 gtk_widget_destroy (GTK_WIDGET (dialog));
 parameters_install->dialog = NULL;
 
 if (response_id == GTK_RESPONSE_YES)
 {
+ char *mime_type;
+
 mime_type = caja_file_get_mime_type (parameters_install->file);
 search_for_application_mime_type (parameters_install, mime_type);
 g_free (mime_type);
@@ -1686,8 +1695,6 @@ activate_desktop_file (ActivateParameters *parameters,
 CajaFile *file)
 {
 ActivateParametersDesktop *parameters_desktop;
- char *primary, *secondary, *display_name;
- GtkWidget *dialog;
 GdkScreen *screen;
 char *uri;
 
@@ -1695,6 +1702,9 @@ activate_desktop_file (ActivateParameters *parameters,
 
 if (!caja_file_is_trusted_link (file))
 {
+ char *primary, *secondary, *display_name;
+ GtkWidget *dialog;
+
 /* copy the parts of parameters we are interested in as the orignal will be freed */
 parameters_desktop = g_new0 (ActivateParametersDesktop, 1);
 if (parameters->parent_window)
@@ -1767,7 +1777,6 @@ activate_files (ActivateParameters *parameters)
 GList *open_in_app_uris;
 GList *open_in_app_parameters;
 GList *unhandled_open_in_app_uris;
- ApplicationLaunchParameters *one_parameters;
 GList *open_in_view_files;
 GList *l;
 int count;
@@ -1777,6 +1786,7 @@ activate_files (ActivateParameters *parameters)
 ActivationAction action;
 GdkScreen *screen;
 LaunchLocation *location;
+ ApplicationLaunchParameters *one_parameters = NULL;
 
 screen = gtk_widget_get_screen (GTK_WIDGET (parameters->parent_window));
 
@@ -2014,7 +2024,6 @@ activation_mount_not_mounted_callback (GObject *source_object,
 ActivateParameters *parameters = user_data;
 GError *error;
 CajaFile *file;
- LaunchLocation *loc;
 
 file = parameters->not_mounted->data;
 
@@ -2033,6 +2042,8 @@ activation_mount_not_mounted_callback (GObject *source_object,
 if (error->domain != G_IO_ERROR ||
 error->code != G_IO_ERROR_ALREADY_MOUNTED)
 {
+ LaunchLocation *loc;
+
 loc = find_launch_location_for_file (parameters->locations,
 file);
 if (loc)
@@ -2056,14 +2067,15 @@ activation_mount_not_mounted_callback (GObject *source_object,
 static void
 activation_mount_not_mounted (ActivateParameters *parameters)
 {
- CajaFile *file;
- GFile *location;
- LaunchLocation *loc;
- GMountOperation *mount_op;
+ LaunchLocation *loc = NULL;
 GList *l, *next, *files;
 
 if (parameters->not_mounted != NULL)
 {
+ CajaFile *file;
+ GFile *location;
+ GMountOperation *mount_op;
+
 file = parameters->not_mounted->data;
 mount_op = gtk_mount_operation_new (parameters->parent_window);
 g_mount_operation_set_password_save (mount_op, G_PASSWORD_SAVE_FOR_SESSION);
@@ -2110,8 +2122,8 @@ activate_callback (GList *files, gpointer callback_data)
 {
 ActivateParameters *parameters = callback_data;
 GList *l, *next;
- CajaFile *file;
 LaunchLocation *location;
+ CajaFile *file = NULL;
 
 parameters->files_handle = NULL;
 
@@ -2161,8 +2173,8 @@ activate_activation_uris_ready_callback (GList *files_ignore,
 {
 ActivateParameters *parameters = callback_data;
 GList *l, *next, *files;
- CajaFile *file;
 LaunchLocation *location;
+ CajaFile *file = NULL;
 
 parameters->files_handle = NULL;
 
@@ -2239,8 +2251,8 @@ static void
 activation_get_activation_uris (ActivateParameters *parameters)
 {
 GList *l, *files;
- CajaFile *file;
 LaunchLocation *location;
+ CajaFile *file = NULL;
 
 /* link target info might be stale, re-read it */
 for (l = parameters->locations; l != NULL; l = l->next)
@@ -2287,7 +2299,6 @@ activation_mountable_mounted (CajaFile *file,
 gpointer callback_data)
 {
 ActivateParameters *parameters = callback_data;
- CajaFile *target_file;
 LaunchLocation *location;
 
 /* Remove from list of files that have to be mounted */
@@ -2297,6 +2308,8 @@ activation_mountable_mounted (CajaFile *file,
 
 if (error == NULL)
 {
+ CajaFile *target_file;
+
 /* Replace file with the result of the mount */
 target_file = caja_file_get (result_location);
 
@@ -2351,11 +2364,11 @@ activation_mountable_mounted (CajaFile *file,
 static void
 activation_mount_mountables (ActivateParameters *parameters)
 {
- CajaFile *file;
- GMountOperation *mount_op;
-
 if (parameters->mountables != NULL)
 {
+ CajaFile *file;
+ GMountOperation *mount_op;
+
 file = parameters->mountables->data;
 mount_op = gtk_mount_operation_new (parameters->parent_window);
 g_mount_operation_set_password_save (mount_op, G_PASSWORD_SAVE_FOR_SESSION);
@@ -2438,11 +2451,11 @@ activation_mountable_started (CajaFile *file,
 static void
 activation_start_mountables (ActivateParameters *parameters)
 {
- CajaFile *file;
- GMountOperation *start_op;
-
 if (parameters->start_mountables != NULL)
 {
+ CajaFile *file;
+ GMountOperation *start_op;
+
 file = parameters->start_mountables->data;
 start_op = gtk_mount_operation_new (parameters->parent_window);
 g_signal_connect (start_op, "notify::is-showing",
@@ -2479,11 +2492,10 @@ caja_mime_activate_files (GtkWindow *parent_window,
 gboolean user_confirmation)
 {
 ActivateParameters *parameters;
- char *file_name;
 int file_count;
 GList *l, *next;
- CajaFile *file;
 LaunchLocation *location;
+ CajaFile *file = NULL;
 
 if (files == NULL)
 {
@@ -2512,6 +2524,8 @@ caja_mime_activate_files (GtkWindow *parent_window,
 file_count = g_list_length (files);
 if (file_count == 1)
 {
+ char *file_name;
+
 file_name = caja_file_get_display_name (files->data);
 parameters->timed_wait_prompt = g_strdup_printf (_("Opening \"%s\"."), file_name);
 g_free (file_name);
diff --git a/libcaja-private/caja-mime-application-chooser.c b/libcaja-private/caja-mime-application-chooser.c
index 3e9c1812..e842cf27 100644
--- a/libcaja-private/caja-mime-application-chooser.c
+++ b/libcaja-private/caja-mime-application-chooser.c
@@ -134,7 +134,6 @@ default_toggled_cb (GtkCellRendererToggle *renderer,
 gboolean is_default;
 gboolean success;
 GAppInfo *info;
- char *message;
 
 gtk_tree_model_get (GTK_TREE_MODEL (chooser->details->model),
 &iter,
@@ -160,6 +159,8 @@ default_toggled_cb (GtkCellRendererToggle *renderer,
 
 if (!success)
 {
+ char *message;
+
 message = g_strdup_printf (_("Could not set application as the default: %s"), error->message);
 eel_show_error_dialog (_("Could not set as default application"),
 message,
diff --git a/libcaja-private/caja-monitor.c b/libcaja-private/caja-monitor.c
index 2116b9d8..bd83458f 100644
--- a/libcaja-private/caja-monitor.c
+++ b/libcaja-private/caja-monitor.c
@@ -45,10 +45,11 @@ caja_monitor_active (void)
 static gboolean tried_monitor = FALSE;
 static gboolean monitor_success;
 GFileMonitor *dir_monitor;
- GFile *file;
 
 if (tried_monitor == FALSE)
 {
+ GFile *file;
+
 file = g_file_new_for_path (g_get_home_dir ());
 dir_monitor = g_file_monitor_directory (file, G_FILE_MONITOR_NONE, NULL, NULL);
 g_object_unref (file);
diff --git a/libcaja-private/caja-open-with-dialog.c b/libcaja-private/caja-open-with-dialog.c
index 1c4e285b..8d21c7cd 100644
--- a/libcaja-private/caja-open-with-dialog.c
+++ b/libcaja-private/caja-open-with-dialog.c
@@ -221,12 +221,9 @@ static GAppInfo *
 add_or_find_application (CajaOpenWithDialog *dialog)
 {
 GAppInfo *app;
- char *app_name;
- const char *commandline;
 GError *error;
 gboolean success, should_set_default;
 char *message;
- GList *applications;
 
 error = NULL;
 app = NULL;
@@ -236,6 +233,9 @@ add_or_find_application (CajaOpenWithDialog *dialog)
 }
 else
 {
+ char *app_name;
+ const char *commandline;
+
 commandline = gtk_entry_get_text (GTK_ENTRY (dialog->details->entry));
 app_name = get_app_name (commandline, &error);
 if (app_name != NULL)
@@ -284,6 +284,8 @@ add_or_find_application (CajaOpenWithDialog *dialog)
 }
 else
 {
+ GList *applications;
+
 applications = g_app_info_get_all_for_type (dialog->details->content_type);
 if (dialog->details->content_type && applications != NULL)
 {
@@ -327,13 +329,13 @@ response_cb (CajaOpenWithDialog *dialog,
 int response_id,
 gpointer data)
 {
- GAppInfo *application;
-
 switch (response_id)
 {
 case RESPONSE_OPEN:
 if (check_application (dialog))
 {
+ GAppInfo *application;
+
 application = add_or_find_application (dialog);
 
 if (application)
@@ -500,7 +502,6 @@ static cairo_surface_t *
 get_surface_for_icon (GIcon *icon)
 {
 cairo_surface_t *surface;
- char *filename;
 gint icon_scale;
 
 surface = NULL;
@@ -508,6 +509,8 @@ get_surface_for_icon (GIcon *icon)
 
 if (G_IS_FILE_ICON (icon))
 {
+ char *filename;
+
 filename = g_file_get_path (g_file_icon_get_file (G_FILE_ICON (icon)));
 if (filename)
 {
@@ -524,13 +527,14 @@ get_surface_for_icon (GIcon *icon)
 else if (G_IS_THEMED_ICON (icon))
 {
 const char * const *names;
- char *icon_no_extension;
- char *p;
 
 names = g_themed_icon_get_names (G_THEMED_ICON (icon));
 
 if (names != NULL && names[0] != NULL)
 {
+ char *icon_no_extension;
+ char *p;
+
 icon_no_extension = g_strdup (names[0]);
 p = strrchr (icon_no_extension, '.');
 if (p &&
@@ -556,11 +560,11 @@ get_surface_for_icon (GIcon *icon)
 static gboolean
 caja_open_with_dialog_add_icon_idle (CajaOpenWithDialog *dialog)
 {
- GtkTreeIter iter;
- GtkTreePath *path;
 cairo_surface_t *surface;
 GIcon *icon;
 gboolean long_operation;
+ GtkTreeIter iter;
+ GtkTreePath *path = NULL;
 
 long_operation = FALSE;
 do
@@ -616,14 +620,14 @@ caja_open_with_search_equal_func (GtkTreeModel *model,
 GtkTreeIter *iter,
 gpointer user_data)
 {
- char *normalized_key;
- char *name, *normalized_name;
- char *path, *normalized_path;
- char *basename, *normalized_basename;
+ char *name;
+ char *path;
 gboolean ret;
 
 if (key != NULL)
 {
+ char *normalized_key;
+
 normalized_key = g_utf8_casefold (key, -1);
 g_assert (normalized_key != NULL);
 
@@ -636,6 +640,8 @@ caja_open_with_search_equal_func (GtkTreeModel *model,
 
 if (name != NULL)
 {
+ char *normalized_name;
+
 normalized_name = g_utf8_casefold (name, -1);
 g_assert (normalized_name != NULL);
 
@@ -649,6 +655,9 @@ caja_open_with_search_equal_func (GtkTreeModel *model,
 
 if (ret && path != NULL)
 {
+ char *normalized_path;
+ char *basename, *normalized_basename;
+
 normalized_path = g_utf8_casefold (path, -1);
 g_assert (normalized_path != NULL);
 
@@ -1028,7 +1037,6 @@ set_uri_and_type (CajaOpenWithDialog *dialog,
 char *label;
 char *emname;
 char *name, *extension;
- char *description;
 char *checkbox_text;
 
 name = NULL;
@@ -1087,6 +1095,8 @@ set_uri_and_type (CajaOpenWithDialog *dialog,
 }
 else
 {
+ char *description;
+
 dialog->details->content_type = g_strdup (mime_type);
 description = g_content_type_get_description (mime_type);
 
diff --git a/libcaja-private/caja-program-choosing.c b/libcaja-private/caja-program-choosing.c
index 05e1c182..5e3d22b3 100644
--- a/libcaja-private/caja-program-choosing.c
+++ b/libcaja-private/caja-program-choosing.c
@@ -64,17 +64,17 @@ application_cannot_open_location (GAppInfo *application,
 GtkWindow *parent_window)
 {
 #ifdef NEW_MIME_COMPLETE
- GtkDialog *message_dialog;
- LaunchParameters *launch_parameters;
 char *prompt;
 char *message;
 char *file_name;
- int response;
 
 file_name = caja_file_get_display_name (file);
 
 if (caja_mime_has_any_applications_for_file (file))
 {
+ GtkDialog *message_dialog;
+ int response;
+
 if (application != NULL)
 {
 prompt = _("Open Failed, would you like to choose another application?");
@@ -102,6 +102,8 @@ application_cannot_open_location (GAppInfo *application,
 
 if (response == GTK_RESPONSE_YES)
 {
+ LaunchParameters *launch_parameters;
+
 launch_parameters = launch_parameters_new (file, parent_window);
 caja_choose_application_for_file
 (file,
@@ -192,16 +194,15 @@ caja_launch_application_by_uri (GAppInfo *application,
 GList *uris,
 GtkWindow *parent_window)
 {
- char *uri, *uri_scheme;
- GList *locations, *l;
- GFile *location;
- CajaFile *file;
- gboolean result;
+ CajaFile *file;
+ gboolean result;
 GError *error;
 GdkDisplay *display;
 GdkAppLaunchContext *launch_context;
 CajaIconInfo *icon;
 int count;
+ GList *locations, *l;
+ GFile *location = NULL;
 
 g_assert (uris != NULL);
 
@@ -210,6 +211,8 @@ caja_launch_application_by_uri (GAppInfo *application,
 locations = NULL;
 for (l = uris; l != NULL; l = l->next)
 {
+ char *uri;
+
 uri = l->data;
 
 location = g_file_new_for_uri (uri);
@@ -263,6 +266,8 @@ caja_launch_application_by_uri (GAppInfo *application,
 if (error->domain == G_IO_ERROR &&
 error->code == G_IO_ERROR_NOT_SUPPORTED)
 {
+ char *uri_scheme;
+
 uri_scheme = g_uri_parse_scheme (uris->data);
 application_cannot_open_location (application,
 file,
@@ -312,7 +317,6 @@ caja_launch_application_from_command (GdkScreen *screen,
 ...)
 {
 char *full_command, *tmp;
- char *quoted_parameter;
 char *parameter;
 va_list ap;
 
@@ -322,6 +326,8 @@ caja_launch_application_from_command (GdkScreen *screen,
 
 while ((parameter = va_arg (ap, char *)) != NULL)
 {
+ char *quoted_parameter;
+
 quoted_parameter = g_shell_quote (parameter);
 tmp = g_strconcat (full_command, " ", quoted_parameter, NULL);
 g_free (quoted_parameter);
@@ -339,8 +345,6 @@ caja_launch_application_from_command (GdkScreen *screen,
 }
 else
 {
- GdkAppLaunchContext *launch_context;
- GdkDisplay *display;
 GAppInfo *app_info = NULL;
 app_info = g_app_info_create_from_commandline (full_command,
 NULL,
@@ -348,6 +352,9 @@ caja_launch_application_from_command (GdkScreen *screen,
 NULL);
 if (app_info != NULL)
 {
+ GdkAppLaunchContext *launch_context;
+ GdkDisplay *display;
+
 display = gdk_screen_get_display (screen);
 launch_context = gdk_display_get_app_launch_context (display);
 gdk_app_launch_context_set_screen (launch_context, screen);
@@ -378,15 +385,17 @@ caja_launch_application_from_command_array (GdkScreen *screen,
 const char * const * parameters)
 {
 char *full_command, *tmp;
- char *quoted_parameter;
- const char * const *p;
 
 full_command = g_strdup (command_string);
 
 if (parameters != NULL)
 {
+ const char * const *p;
+
 for (p = parameters; *p != NULL; p++)
 {
+ char *quoted_parameter;
+
 quoted_parameter = g_shell_quote (*p);
 tmp = g_strconcat (full_command, " ", quoted_parameter, NULL);
 g_free (quoted_parameter);
@@ -402,8 +411,6 @@ caja_launch_application_from_command_array (GdkScreen *screen,
 }
 else
 {
- GdkAppLaunchContext *launch_context;
- GdkDisplay *display;
 GAppInfo *app_info = NULL;
 app_info = g_app_info_create_from_commandline (full_command,
 NULL,
@@ -411,6 +418,9 @@ caja_launch_application_from_command_array (GdkScreen *screen,
 NULL);
 if (app_info != NULL)
 {
+ GdkAppLaunchContext *launch_context;
+ GdkDisplay *display;
+
 display = gdk_screen_get_display (screen);
 launch_context = gdk_display_get_app_launch_context (display);
 gdk_app_launch_context_set_screen (launch_context, screen);
@@ -430,13 +440,14 @@ caja_launch_desktop_file (GdkScreen *screen,
 GtkWindow *parent_window)
 {
 GError *error;
- char *message, *desktop_file_path;
+ char *desktop_file_path;
 const GList *p;
 GList *files;
 int total, count;
- GFile *file, *desktop_file;
 GDesktopAppInfo *app_info;
 GdkAppLaunchContext *context;
+ GFile *desktop_file;
+ GFile *file = NULL;
 
 /* Don't allow command execution from remote locations
 * to partially mitigate the security
@@ -529,6 +540,8 @@ caja_launch_desktop_file (GdkScreen *screen,
 &error);
 if (error != NULL)
 {
+ char *message;
+
 message = g_strconcat (_("Details: "), error->message, NULL);
 eel_show_error_dialog
 (_("There was an error launching the application."),
diff --git a/libcaja-private/caja-progress-info.c b/libcaja-private/caja-progress-info.c
index 590fae13..dce63460 100644
--- a/libcaja-private/caja-progress-info.c
+++ b/libcaja-private/caja-progress-info.c
@@ -307,7 +307,7 @@ progress_widget_data_free (ProgressWidgetData *data)
 static void
 update_data (ProgressWidgetData *data)
 {
- char *status, *details, *curstat, *t;
+ char *status, *details, *curstat;
 char *markup;
 
 status = caja_progress_info_get_status (data->info);
@@ -330,6 +330,8 @@ update_data (ProgressWidgetData *data)
 }
 
 if (curstat != NULL) {
+ char *t;
+
 t = status;
 status = g_strconcat (status, " \xE2\x80\x94 ", curstat, NULL);
 g_free (t);
@@ -376,9 +378,9 @@ get_running_operations ()
 static void
 foreach_get_queued_widget (GtkWidget * widget, GtkWidget ** out)
 {
- ProgressWidgetData *data;
-
 if (*out == NULL) {
+ ProgressWidgetData *data;
+
 data = (ProgressWidgetData*) g_object_get_data (
 G_OBJECT(widget), "data");
 
@@ -463,10 +465,10 @@ widget_reposition_as_queued (GtkWidget * widget)
 static void
 widget_reposition_as_paused (GtkWidget * widget)
 {
- ProgressWidgetData *data;
+ int i;
 GList *children, *child;
+ ProgressWidgetData *data = NULL;
 gboolean abort = FALSE;
- int i;
 GtkWidget * container = get_widgets_container();
 
 children = gtk_container_get_children (GTK_CONTAINER(container));
@@ -495,8 +497,8 @@ widget_reposition_as_paused (GtkWidget * widget)
 static void
 widget_reposition_as_running (GtkWidget * widget)
 {
- ProgressWidgetData *data;
 GList *children, *child;
+ ProgressWidgetData *data = NULL;
 gboolean abort = FALSE;
 int i, mypos = -1;
 GtkWidget * container = get_widgets_container();
@@ -561,13 +563,14 @@ widget_state_transit_to (ProgressWidgetData *data,
 static void
 update_queue ()
 {
- GtkWidget *next;
- ProgressWidgetData *data;
-
 if (get_running_operations () == 0) {
+ GtkWidget *next;
+
 next = get_first_queued_widget ();
 
 if (next != NULL) {
+ ProgressWidgetData *data;
+
 data = (ProgressWidgetData*) g_object_get_data (
 G_OBJECT(next), "data");
 widget_state_transit_to (data, STATE_RUNNING);
@@ -596,13 +599,14 @@ update_status_icon_and_window (void)
 {
 char *tooltip;
 gboolean toshow;
- GIcon *icon;
 GNotification *notification;
 gboolean show_notifications = g_settings_get_boolean (caja_preferences, CAJA_PREFERENCES_SHOW_NOTIFICATIONS);
 static gboolean window_shown = FALSE;
 
 if (show_notifications)
 {
+ GIcon *icon;
+
 notification = g_notification_new ("caja");
 icon = g_themed_icon_new ("system-file-manager");
 g_notification_set_icon (notification, icon);
diff --git a/libcaja-private/caja-query.c b/libcaja-private/caja-query.c
index 63c71372..e7baf72f 100644
--- a/libcaja-private/caja-query.c
+++ b/libcaja-private/caja-query.c
@@ -198,7 +198,6 @@ encode_home_uri (const char *uri)
 static char *
 decode_home_uri (const char *uri)
 {
- char *home_uri;
 char *decoded_uri;
 
 if (g_str_has_prefix (uri, "file:"))
@@ -207,6 +206,8 @@ decode_home_uri (const char *uri)
 }
 else
 {
+ char *home_uri;
+
 home_uri = caja_get_home_directory_uri ();
 
 decoded_uri = g_strconcat (home_uri, "/", uri, NULL);
@@ -380,9 +381,6 @@ caja_query_to_xml (CajaQuery *query)
 {
 GString *xml;
 char *text;
- char *uri;
- char *mimetype;
- char *tag;
 GList *l;
 
 xml = g_string_new ("");
@@ -396,6 +394,8 @@ caja_query_to_xml (CajaQuery *query)
 
 if (query->details->location_uri)
 {
+ char *uri;
+
 uri = encode_home_uri (query->details->location_uri);
 g_string_append_printf (xml, " <location>%s</location>\n", uri);
 g_free (uri);
@@ -406,6 +406,8 @@ caja_query_to_xml (CajaQuery *query)
 g_string_append (xml, " <mimetypes>\n");
 for (l = query->details->mime_types; l != NULL; l = l->next)
 {
+ char *mimetype;
+
 mimetype = g_markup_escape_text (l->data, -1);
 g_string_append_printf (xml, " <mimetype>%s</mimetype>\n", mimetype);
 g_free (mimetype);
@@ -418,7 +420,9 @@ caja_query_to_xml (CajaQuery *query)
 g_string_append (xml, " <tags>\n");
 for (l = query->details->tags; l != NULL; l = l->next)
 {
- tag = g_markup_escape_text (l->data, -1);
+ char *tag;
+
+ tag = g_markup_escape_text (l->data, -1);
 g_string_append_printf (xml, " <tag>%s</tag>\n", tag);
 g_free (tag);
 }
diff --git a/libcaja-private/caja-search-directory-file.c b/libcaja-private/caja-search-directory-file.c
index 3ff83a7f..f455dc01 100644
--- a/libcaja-private/caja-search-directory-file.c
+++ b/libcaja-private/caja-search-directory-file.c
@@ -100,10 +100,10 @@ search_directory_file_get_item_count (CajaFile *file,
 guint *count,
 gboolean *count_unreadable)
 {
- GList *file_list;
-
 if (count)
 {
+ GList *file_list;
+
 file_list = caja_directory_get_file_list (file->details->directory);
 
 *count = g_list_length (file_list);
@@ -122,10 +122,10 @@ search_directory_file_get_deep_counts (CajaFile *file,
 goffset *total_size,
 goffset *total_size_on_disk)
 {
- CajaFile *dir_file;
 GList *file_list, *l;
 guint dirs, files;
 GFileType type;
+ CajaFile *dir_file = NULL;
 
 file_list = caja_directory_get_file_list (file->details->directory);
 
@@ -182,8 +182,6 @@ void
 caja_search_directory_file_update_display_name (CajaSearchDirectoryFile *search_file)
 {
 CajaFile *file;
- CajaSearchDirectory *search_dir;
- CajaQuery *query;
 char *display_name;
 gboolean changed;
 
@@ -192,6 +190,9 @@ caja_search_directory_file_update_display_name (CajaSearchDirectoryFile *search_
 file = CAJA_FILE (search_file);
 if (file->details->directory)
 {
+ CajaSearchDirectory *search_dir;
+ CajaQuery *query;
+
 search_dir = CAJA_SEARCH_DIRECTORY (file->details->directory);
 query = caja_search_directory_get_query (search_dir);
 
diff --git a/libcaja-private/caja-search-directory.c b/libcaja-private/caja-search-directory.c
index effab562..8209e546 100644
--- a/libcaja-private/caja-search-directory.c
+++ b/libcaja-private/caja-search-directory.c
@@ -112,8 +112,8 @@ static void
 reset_file_list (CajaSearchDirectory *search)
 {
 GList *list, *monitor_list;
- CajaFile *file;
 SearchMonitor *monitor;
+ CajaFile *file = NULL;
 
 /* Remove file connections */
 for (list = search->details->files; list != NULL; list = list->next)
@@ -189,7 +189,7 @@ search_monitor_add (CajaDirectory *directory,
 GList *list;
 SearchMonitor *monitor;
 CajaSearchDirectory *search;
- CajaFile *file;
+ CajaFile *file = NULL;
 
 search = CAJA_SEARCH_DIRECTORY (directory);
 
@@ -220,7 +220,7 @@ static void
 search_monitor_remove_file_monitors (SearchMonitor *monitor, CajaSearchDirectory *search)
 {
 GList *list;
- CajaFile *file;
+ CajaFile *file = NULL;
 
 for (list = search->details->files; list != NULL; list = list->next)
 {
@@ -243,8 +243,8 @@ search_monitor_remove (CajaDirectory *directory,
 gconstpointer client)
 {
 CajaSearchDirectory *search;
- SearchMonitor *monitor;
 GList *list;
+ SearchMonitor *monitor = NULL;
 
 search = CAJA_SEARCH_DIRECTORY (directory);
 
@@ -322,7 +322,7 @@ static void
 search_callback_add_file_callbacks (SearchCallback *callback)
 {
 GList *file_list_copy, *list;
- CajaFile *file;
+ CajaFile *file = NULL;
 
 file_list_copy = g_list_copy (callback->file_list);
 
@@ -341,8 +341,8 @@ search_callback_add_file_callbacks (SearchCallback *callback)
 static SearchCallback *
 search_callback_find (CajaSearchDirectory *search, CajaDirectoryCallback callback, gpointer callback_data)
 {
- SearchCallback *search_callback;
 GList *list;
+ SearchCallback *search_callback = NULL;
 
 for (list = search->details->callback_list; list != NULL; list = list->next)
 {
@@ -361,8 +361,8 @@ search_callback_find (CajaSearchDirectory *search, CajaDirectoryCallback callbac
 static SearchCallback *
 search_callback_find_pending (CajaSearchDirectory *search, CajaDirectoryCallback callback, gpointer callback_data)
 {
- SearchCallback *search_callback;
 GList *list;
+ SearchCallback *search_callback = NULL;
 
 for (list = search->details->pending_callback_list; list != NULL; list = list->next)
 {
@@ -501,7 +501,6 @@ search_engine_hits_added (CajaSearchEngine *engine, GList *hits,
 GList *hit_list;
 GList *file_list;
 CajaFile *file;
- char *uri;
 SearchMonitor *monitor;
 GList *monitor_list;
 
@@ -509,6 +508,8 @@ search_engine_hits_added (CajaSearchEngine *engine, GList *hits,
 
 for (hit_list = hits; hit_list != NULL; hit_list = hit_list->next)
 {
+ char *uri;
+
 uri = hit_list->data;
 
 if (g_str_has_suffix (uri, CAJA_SAVED_SEARCH_EXTENSION))
@@ -549,13 +550,14 @@ search_engine_hits_subtracted (CajaSearchEngine *engine, GList *hits,
 GList *monitor_list;
 SearchMonitor *monitor;
 GList *file_list;
- char *uri;
 CajaFile *file;
 
 file_list = NULL;
 
 for (hit_list = hits; hit_list != NULL; hit_list = hit_list->next)
 {
+ char *uri;
+
 uri = hit_list->data;
 file = caja_file_get_by_uri (uri);
 
@@ -852,7 +854,6 @@ CajaSearchDirectory *
 caja_search_directory_new_from_saved_search (const char *uri)
 {
 CajaSearchDirectory *search;
- CajaQuery *query;
 char *file;
 
 search = CAJA_SEARCH_DIRECTORY (g_object_new (CAJA_TYPE_SEARCH_DIRECTORY, NULL));
@@ -862,6 +863,8 @@ caja_search_directory_new_from_saved_search (const char *uri)
 file = g_filename_from_uri (uri, NULL, NULL);
 if (file != NULL)
 {
+ CajaQuery *query;
+
 query = caja_query_load (file);
 if (query != NULL)
 {
diff --git a/libcaja-private/caja-search-engine-beagle.c b/libcaja-private/caja-search-engine-beagle.c
index b891fdbb..a07fa75d 100644
--- a/libcaja-private/caja-search-engine-beagle.c
+++ b/libcaja-private/caja-search-engine-beagle.c
@@ -214,7 +214,6 @@ beagle_hits_added (BeagleQuery *query,
 {
 GSList *hits, *list;
 GList *hit_uris;
- const char *uri;
 
 hit_uris = NULL;
 
@@ -222,6 +221,8 @@ beagle_hits_added (BeagleQuery *query,
 
 for (list = hits; list != NULL; list = list->next)
 {
+ const char *uri;
+
 BeagleHit *hit = BEAGLE_HIT (list->data);
 
 uri = beagle_hit_get_uri (hit);
@@ -290,7 +291,7 @@ caja_search_engine_beagle_start (CajaSearchEngine *engine)
 CajaSearchEngineBeagle *beagle;
 GError *error;
 GList *mimetypes, *l;
- char *text, *mimetype;
+ char *text;
 
 error = NULL;
 beagle = CAJA_SEARCH_ENGINE_BEAGLE (engine);
@@ -325,6 +326,8 @@ caja_search_engine_beagle_start (CajaSearchEngine *engine)
 for (l = mimetypes; l != NULL; l = l->next)
 {
 char* temp;
+ char *mimetype;
+
 mimetype = l->data;
 temp = g_strconcat (" mimetype:", mimetype, NULL);
 beagle_query_add_text (beagle->details->current_query,temp);
diff --git a/libcaja-private/caja-search-engine-simple.c b/libcaja-private/caja-search-engine-simple.c
index c8d4939d..40abeb3a 100644
--- a/libcaja-private/caja-search-engine-simple.c
+++ b/libcaja-private/caja-search-engine-simple.c
@@ -197,12 +197,12 @@ search_thread_add_hits_idle (gpointer user_data)
 static void
 send_batch (SearchThreadData *data)
 {
- SearchHits *hits;
-
 data->n_processed_files = 0;
 
 if (data->uri_hits)
 {
+ SearchHits *hits;
+
 hits = g_new (SearchHits, 1);
 hits->uris = data->uri_hits;
 hits->thread_data = data;
@@ -505,7 +505,6 @@ search_thread_func (gpointer user_data)
 SearchThreadData *data;
 GFile *dir;
 GFileInfo *info;
- const char *id;
 
 data = user_data;
 
@@ -514,6 +513,8 @@ search_thread_func (gpointer user_data)
 info = g_file_query_info (dir, G_FILE_ATTRIBUTE_ID_FILE, 0, data->cancellable, NULL);
 if (info)
 {
+ const char *id;
+
 id = g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_ID_FILE);
 if (id)
 {
diff --git a/libcaja-private/caja-search-engine-tracker.c b/libcaja-private/caja-search-engine-tracker.c
index f1ec2bfe..33ac6d9a 100644
--- a/libcaja-private/caja-search-engine-tracker.c
+++ b/libcaja-private/caja-search-engine-tracker.c
@@ -246,8 +246,6 @@ static void
 search_callback (gpointer results, GError *error, gpointer user_data)
 {
 CajaSearchEngineTracker *tracker;
- char **results_p;
- GPtrArray *OUT_result;
 GList *hit_uris;
 gint i;
 char *uri;
@@ -271,6 +269,8 @@ search_callback (gpointer results, GError *error, gpointer user_data)
 
 if (tracker->details->version == TRACKER_0_8)
 {
+ GPtrArray *OUT_result;
+
 /* new tracker 0.8 API */
 OUT_result = (GPtrArray*) results;
 
@@ -289,6 +289,8 @@ search_callback (gpointer results, GError *error, gpointer user_data)
 }
 else
 {
+ char **results_p;
+
 /* old tracker 0.6 API */
 for (results_p = results; *results_p; results_p++)
 {
diff --git a/libcaja-private/caja-thumbnails.c b/libcaja-private/caja-thumbnails.c
index dedb08ea..69d4a150 100644
--- a/libcaja-private/caja-thumbnails.c
+++ b/libcaja-private/caja-thumbnails.c
@@ -190,8 +190,6 @@ thumbnail_thread_starter_cb (gpointer data)
 void
 caja_thumbnail_remove_from_queue (const char *file_uri)
 {
- GList *node;
-
 #ifdef DEBUG_THUMBNAILS
 g_message ("(Remove from queue) Locking mutex\n");
 #endif
@@ -203,6 +201,8 @@ caja_thumbnail_remove_from_queue (const char *file_uri)
 
 if (thumbnails_to_make_hash)
 {
+ GList *node;
+
 node = g_hash_table_lookup (thumbnails_to_make_hash, file_uri);
 
 if (node && node->data != currently_thumbnailing)
@@ -226,8 +226,6 @@ caja_thumbnail_remove_from_queue (const char *file_uri)
 void
 caja_thumbnail_prioritize (const char *file_uri)
 {
- GList *node;
-
 #ifdef DEBUG_THUMBNAILS
 g_message ("(Prioritize) Locking mutex\n");
 #endif
@@ -239,6 +237,8 @@ caja_thumbnail_prioritize (const char *file_uri)
 
 if (thumbnails_to_make_hash)
 {
+ GList *node;
+
 node = g_hash_table_lookup (thumbnails_to_make_hash, file_uri);
 
 if (node && node->data != currently_thumbnailing)
@@ -296,8 +296,6 @@ get_types_table (void)
 {
 static GHashTable *image_mime_types = NULL;
 GSList *format_list, *l;
- char **types;
- int i;
 
 if (image_mime_types == NULL)
 {
@@ -308,6 +306,9 @@ get_types_table (void)
 format_list = gdk_pixbuf_get_formats ();
 for (l = format_list; l; l = l->next)
 {
+ char **types;
+ int i;
+
 types = gdk_pixbuf_format_get_mime_types (l->data);
 
 for (i = 0; types[i] != NULL; i++)
@@ -387,8 +388,7 @@ caja_create_thumbnail (CajaFile *file)
 {
 time_t file_mtime = 0;
 CajaThumbnailInfo *info;
- CajaThumbnailInfo *existing_info;
- GList *existing, *node;
+ GList *existing;
 
 caja_file_set_is_thumbnailing (file, TRUE);
 
@@ -431,6 +431,8 @@ caja_create_thumbnail (CajaFile *file)
 existing = g_hash_table_lookup (thumbnails_to_make_hash, info->image_uri);
 if (existing == NULL)
 {
+ GList *node;
+
 /* Add the thumbnail to the list. */
 #ifdef DEBUG_THUMBNAILS
 g_message ("(Main Thread) Adding thumbnail: %s\n",
@@ -453,6 +455,8 @@ caja_create_thumbnail (CajaFile *file)
 }
 else
 {
+ CajaThumbnailInfo *existing_info;
+
 #ifdef DEBUG_THUMBNAILS
 g_message ("(Main Thread) Updating non-current mtime: %s\n",
 info->image_uri);
diff --git a/libcaja-private/caja-trash-monitor.c b/libcaja-private/caja-trash-monitor.c
index 678b506a..fcde8570 100644
--- a/libcaja-private/caja-trash-monitor.c
+++ b/libcaja-private/caja-trash-monitor.c
@@ -100,10 +100,7 @@ update_info_cb (GObject *source_object,
 {
 CajaTrashMonitor *trash_monitor;
 GFileInfo *info;
- GIcon *icon;
- const char * const *names;
 gboolean empty;
- int i;
 
 trash_monitor = CAJA_TRASH_MONITOR (user_data);
 
@@ -112,6 +109,8 @@ update_info_cb (GObject *source_object,
 
 if (info != NULL)
 {
+ GIcon *icon;
+
 icon = g_file_info_get_icon (info);
 
 if (icon)
@@ -121,6 +120,9 @@ update_info_cb (GObject *source_object,
 empty = TRUE;
 if (G_IS_THEMED_ICON (icon))
 {
+ const char * const *names;
+ int i;
+
 names = g_themed_icon_get_names (G_THEMED_ICON (icon));
 for (i = 0; names[i] != NULL; i++)
 {
diff --git a/libcaja-private/caja-tree-view-drag-dest.c b/libcaja-private/caja-tree-view-drag-dest.c
index c05acafb..041e26a5 100644
--- a/libcaja-private/caja-tree-view-drag-dest.c
+++ b/libcaja-private/caja-tree-view-drag-dest.c
@@ -335,7 +335,6 @@ static CajaFile *
 file_for_path (CajaTreeViewDragDest *dest, GtkTreePath *path)
 {
 CajaFile *file;
- char *uri;
 
 if (path)
 {
@@ -343,6 +342,8 @@ file_for_path (CajaTreeViewDragDest *dest, GtkTreePath *path)
 }
 else
 {
+ char *uri;
+
 uri = get_root_uri (dest);
 
 file = NULL;
@@ -506,7 +507,6 @@ drag_motion_callback (GtkWidget *widget,
 CajaTreeViewDragDest *dest;
 GtkTreePath *path;
 GtkTreePath *drop_path, *old_drop_path;
- GtkTreeModel *model;
 GtkTreeIter drop_iter;
 GtkTreeViewDropPosition pos;
 GdkWindow *bin_window;
@@ -549,6 +549,8 @@ drag_motion_callback (GtkWidget *widget,
 
 if (action)
 {
+ GtkTreeModel *model;
+
 set_drag_dest_row (dest, drop_path);
 model = gtk_tree_view_get_model (GTK_TREE_VIEW (widget));
 if (drop_path == NULL || (old_drop_path != NULL &&
@@ -907,8 +909,6 @@ drag_data_received_callback (GtkWidget *widget,
 gpointer data)
 {
 CajaTreeViewDragDest *dest;
- const char *tmp;
- int length;
 gboolean success, finished;
 
 dest = CAJA_TREE_VIEW_DRAG_DEST (data);
@@ -949,11 +949,16 @@ drag_data_received_callback (GtkWidget *widget,
 success = TRUE;
 break;
 case CAJA_ICON_DND_RAW:
+ {
+ const char *tmp;
+ int length;
+
 length = gtk_selection_data_get_length (selection_data);
 tmp = gtk_selection_data_get_data (selection_data);
 receive_dropped_raw (dest, tmp, length, context, x, y);
 success = TRUE;
 break;
+ }
 case CAJA_ICON_DND_KEYWORD:
 receive_dropped_keyword (dest, context, x, y);
 success = TRUE;
@@ -1015,9 +1020,8 @@ set_direct_save_uri (CajaTreeViewDragDest *dest,
 GdkDragContext *context,
 int x, int y)
 {
- GFile *base, *child;
 char *drop_uri;
- char *filename, *uri;
+ char *uri;
 
 g_assert (dest->details->direct_save_uri == NULL);
 
@@ -1026,9 +1030,13 @@ set_direct_save_uri (CajaTreeViewDragDest *dest,
 drop_uri = get_drop_target_uri_at_pos (dest, x, y);
 if (drop_uri != NULL)
 {
+ char *filename;
+
 filename = get_direct_save_filename (context);
 if (filename != NULL)
 {
+ GFile *base, *child;
+
 /* Resolve relative path */
 base = g_file_new_for_uri (drop_uri);
 child = g_file_get_child (base, filename);
diff --git a/libcaja-private/caja-ui-utilities.c b/libcaja-private/caja-ui-utilities.c
index 0f9da559..0a83e3c6 100644
--- a/libcaja-private/caja-ui-utilities.c
+++ b/libcaja-private/caja-ui-utilities.c
@@ -93,7 +93,6 @@ caja_ui_string_get (const char *filename)
 {
 static GHashTable *ui_cache = NULL;
 char *ui;
- char *path;
 
 if (ui_cache == NULL)
 {
@@ -104,6 +103,8 @@ caja_ui_string_get (const char *filename)
 ui = g_hash_table_lookup (ui_cache, filename);
 if (ui == NULL)
 {
+ char *path;
+
 path = caja_ui_file (filename);
 if (path == NULL || !g_file_get_contents (path, &ui, NULL, NULL))
 {
@@ -171,7 +172,6 @@ caja_action_from_menu_item (CajaMenuItem *item,
 char *name, *label, *tip, *icon_name;
 gboolean sensitive, priority;
 GtkAction *action;
- cairo_surface_t *surface;
 
 g_object_get (G_OBJECT (item),
 "name", &name, "label", &label,
@@ -187,6 +187,8 @@ caja_action_from_menu_item (CajaMenuItem *item,
 
 if (icon_name != NULL)
 {
+ cairo_surface_t *surface;
+
 surface = get_action_icon (icon_name,
 caja_get_icon_size_for_stock_size (GTK_ICON_SIZE_MENU),
 parent_widget);
@@ -220,7 +222,6 @@ caja_toolbar_action_from_menu_item (CajaMenuItem *item, GtkWidget *parent_widget
 char *name, *label, *tip, *icon_name;
 gboolean sensitive, priority;
 GtkAction *action;
- cairo_surface_t *surface;
 
 g_object_get (G_OBJECT (item),
 "name", &name, "label", &label,
@@ -236,6 +237,8 @@ caja_toolbar_action_from_menu_item (CajaMenuItem *item, GtkWidget *parent_widget
 
 if (icon_name != NULL)
 {
+ cairo_surface_t *surface;
+
 surface = get_action_icon (icon_name,
 caja_get_icon_size_for_stock_size (GTK_ICON_SIZE_LARGE_TOOLBAR),
 parent_widget);
@@ -271,11 +274,12 @@ caja_toolbar_action_from_menu_item (CajaMenuItem *item, GtkWidget *parent_widget
 static GdkPixbuf *
 caja_get_thumbnail_frame (void)
 {
- char *image_path;
 static GdkPixbuf *thumbnail_frame = NULL;
 
 if (thumbnail_frame == NULL)
 {
+ char *image_path;
+
 image_path = caja_pixmap_file ("thumbnail_frame.png");
 if (image_path != NULL)
 {
diff --git a/libcaja-private/caja-undostack-manager.c b/libcaja-private/caja-undostack-manager.c
index 8ce41a21..e875803f 100644
--- a/libcaja-private/caja-undostack-manager.c
+++ b/libcaja-private/caja-undostack-manager.c
@@ -397,10 +397,6 @@ void
 caja_undostack_manager_redo (CajaUndoStackManager * manager,
 GtkWidget * parent_view, CajaUndostackFinishCallback cb)
 {
- GList *uris;
- CajaFile *file;
- char *new_name;
- char *puri;
 CajaUndoStackManagerPrivate *priv = manager->priv;
 
 g_mutex_lock (&priv->mutex);
@@ -421,12 +417,20 @@ caja_undostack_manager_redo (CajaUndoStackManager * manager,
 priv->undo_redo_flag = TRUE;
 switch (action->type) {
 case CAJA_UNDOSTACK_COPY:
+ {
+ GList *uris;
+
 uris = construct_gfile_list (action->sources, action->src_dir);
 caja_file_operations_copy (uris, NULL,
 action->dest_dir, NULL, undo_redo_done_transfer_callback, action);
 g_list_free_full (uris, g_object_unref);
 break;
+ }
 case CAJA_UNDOSTACK_CREATEFILEFROMTEMPLATE:
+ {
+ char *new_name;
+ char *puri;
+
 puri = get_uri_parent (action->target_uri);
 new_name = get_uri_basename (action->target_uri);
 caja_file_operations_new_file_from_template (NULL,
@@ -436,20 +440,33 @@ caja_undostack_manager_redo (CajaUndoStackManager * manager,
 g_free (puri);
 g_free (new_name);
 break;
+ }
 case CAJA_UNDOSTACK_DUPLICATE:
+ {
+ GList *uris;
+
 uris = construct_gfile_list (action->sources, action->src_dir);
 caja_file_operations_duplicate (uris, NULL, NULL,
 undo_redo_done_transfer_callback, action);
 g_list_free_full (uris, g_object_unref);
 break;
+ }
 case CAJA_UNDOSTACK_RESTOREFROMTRASH:
 case CAJA_UNDOSTACK_MOVE:
+ {
+ GList *uris;
+
 uris = construct_gfile_list (action->sources, action->src_dir);
 caja_file_operations_move (uris, NULL,
 action->dest_dir, NULL, undo_redo_done_transfer_callback, action);
 g_list_free_full (uris, g_object_unref);
 break;
+ }
 case CAJA_UNDOSTACK_RENAME:
+ {
+ CajaFile *file;
+ char *new_name;
+
 new_name = get_uri_basename (action->new_uri);
 file = caja_file_get_by_uri (action->old_uri);
 caja_file_rename (file, new_name,
@@ -457,7 +474,12 @@ caja_undostack_manager_redo (CajaUndoStackManager * manager,
 g_object_unref (file);
 g_free (new_name);
 break;
+ }
 case CAJA_UNDOSTACK_CREATEEMPTYFILE:
+ {
+ char *new_name;
+ char *puri;
+
 puri = get_uri_parent (action->target_uri);
 new_name = get_uri_basename (action->target_uri);
 caja_file_operations_new_file (NULL, NULL, puri,
@@ -467,14 +489,21 @@ caja_undostack_manager_redo (CajaUndoStackManager * manager,
 g_free (puri);
 g_free (new_name);
 break;
+ }
 case CAJA_UNDOSTACK_CREATEFOLDER:
+ {
+ char *puri;
+
 puri = get_uri_parent (action->target_uri);
 caja_file_operations_new_folder (NULL, NULL, puri,
 undo_redo_done_create_callback, action);
 g_free (puri);
 break;
+ }
 case CAJA_UNDOSTACK_MOVETOTRASH:
 if (g_hash_table_size (action->trashed) > 0) {
+ GList *uris;
+
 GList *uri_to_trash = g_hash_table_get_keys (action->trashed);
 uris = uri_list_to_gfile_list (uri_to_trash);
 priv->undo_redo_flag = TRUE;
@@ -485,18 +514,29 @@ caja_undostack_manager_redo (CajaUndoStackManager * manager,
 }
 break;
 case CAJA_UNDOSTACK_CREATELINK:
+ {
+ GList *uris;
+
 uris = construct_gfile_list (action->sources, action->src_dir);
 caja_file_operations_link (uris, NULL,
 action->dest_dir, NULL, undo_redo_done_transfer_callback, action);
 g_list_free_full (uris, g_object_unref);
 break;
+ }
 case CAJA_UNDOSTACK_SETPERMISSIONS:
+ {
+ CajaFile *file;
+
 file = caja_file_get_by_uri (action->target_uri);
 caja_file_set_permissions (file,
 action->new_permissions, undo_redo_done_rename_callback, action);
 g_object_unref (file);
 break;
+ }
 case CAJA_UNDOSTACK_RECURSIVESETPERMISSIONS:
+ {
+ char *puri;
+
 puri = g_file_get_uri (action->dest_dir);
 caja_file_set_permissions_recursive (puri,
 action->file_permissions,
@@ -505,20 +545,29 @@ caja_undostack_manager_redo (CajaUndoStackManager * manager,
 action->dir_mask, undo_redo_op_callback, action);
 g_free (puri);
 break;
+ }
 case CAJA_UNDOSTACK_CHANGEGROUP:
+ {
+ CajaFile *file;
+
 file = caja_file_get_by_uri (action->target_uri);
 caja_file_set_group (file,
 action->new_group_name_or_id,
 undo_redo_done_rename_callback, action);
 g_object_unref (file);
 break;
+ }
 case CAJA_UNDOSTACK_CHANGEOWNER:
+ {
+ CajaFile *file;
+
 file = caja_file_get_by_uri (action->target_uri);
 caja_file_set_owner (file,
 action->new_user_name_or_id,
 undo_redo_done_rename_callback, action);
 g_object_unref (file);
 break;
+ }
 case CAJA_UNDOSTACK_DELETE:
 default:
 priv->undo_redo_flag = FALSE;
@@ -536,11 +585,8 @@ void
 caja_undostack_manager_undo (CajaUndoStackManager * manager,
 GtkWidget * parent_view, CajaUndostackFinishCallback cb)
 {
- GList *uris = NULL;
- GHashTable *files_to_restore;
- CajaFile *file;
- char *new_name;
 CajaUndoStackManagerPrivate *priv = manager->priv;
+ GList *uris = NULL;
 
 g_mutex_lock (&priv->mutex);
 
@@ -595,15 +641,19 @@ caja_undostack_manager_undo (CajaUndoStackManager * manager,
 g_list_free_full (uris, g_object_unref);
 break;
 case CAJA_UNDOSTACK_MOVETOTRASH:
+ {
+ GHashTable *files_to_restore;
+
 files_to_restore = retrieve_files_to_restore (action->trashed);
 if (g_hash_table_size (files_to_restore) > 0) {
 GList *l;
 GList *gfiles_in_trash = g_hash_table_get_keys (files_to_restore);
- GFile *item;
- GFile *dest;
- char *value;
+ GFile *item = NULL;
+ GFile *dest = NULL;
 
 for (l = gfiles_in_trash; l != NULL; l = l->next) {
+ char *value;
+
 item = l->data;
 value = g_hash_table_lookup (files_to_restore, item);
 dest = g_file_new_for_uri (value);
@@ -618,6 +668,7 @@ caja_undostack_manager_undo (CajaUndoStackManager * manager,
 /* Here we must do what's necessary for the callback */
 undo_redo_done_transfer_callback (NULL, action);
 break;
+ }
 case CAJA_UNDOSTACK_MOVE:
 uris = construct_gfile_list (action->destinations, action->dest_dir);
 caja_file_operations_move (uris, NULL,
@@ -625,6 +676,10 @@ caja_undostack_manager_undo (CajaUndoStackManager * manager,
 g_list_free_full (uris, g_object_unref);
 break;
 case CAJA_UNDOSTACK_RENAME:
+ {
+ CajaFile *file;
+ char *new_name;
+
 new_name = get_uri_basename (action->old_uri);
 file = caja_file_get_by_uri (action->new_uri);
 caja_file_rename (file, new_name,
@@ -632,23 +687,30 @@ caja_undostack_manager_undo (CajaUndoStackManager * manager,
 g_object_unref (file);
 g_free (new_name);
 break;
+ }
 case CAJA_UNDOSTACK_SETPERMISSIONS:
+ {
+ CajaFile *file;
+
 file = caja_file_get_by_uri (action->target_uri);
 caja_file_set_permissions (file,
 action->current_permissions,
 undo_redo_done_rename_callback, action);
 g_object_unref (file);
 break;
+ }
 case CAJA_UNDOSTACK_RECURSIVESETPERMISSIONS:
 if (g_hash_table_size (action->original_permissions) > 0) {
 GList *gfiles_list =
 g_hash_table_get_keys (action->original_permissions);
- guint32 *perm;
+
 GList *l;
- GFile *dest;
- char *item;
+ GFile *dest = NULL;
 
 for (l = gfiles_list; l != NULL; l = l->next) {
+ guint32 *perm;
+ char *item;
+
 item = l->data;
 perm = g_hash_table_lookup (action->original_permissions, item);
 dest = g_file_new_for_uri (item);
@@ -664,19 +726,27 @@ caja_undostack_manager_undo (CajaUndoStackManager * manager,
 }
 break;
 case CAJA_UNDOSTACK_CHANGEGROUP:
+ {
+ CajaFile *file;
+
 file = caja_file_get_by_uri (action->target_uri);
 caja_file_set_group (file,
 action->original_group_name_or_id,
 undo_redo_done_rename_callback, action);
 g_object_unref (file);
 break;
+ }
 case CAJA_UNDOSTACK_CHANGEOWNER:
+ {
+ CajaFile *file;
+
 file = caja_file_get_by_uri (action->target_uri);
 caja_file_set_owner (file,
 action->original_user_name_or_id,
 undo_redo_done_rename_callback, action);
 g_object_unref (file);
 break;
+ }
 case CAJA_UNDOSTACK_DELETE:
 default:
 priv->undo_redo_flag = FALSE;
@@ -1077,8 +1147,8 @@ stack_scroll_left (CajaUndoStackManagerPrivate * priv)
 static void
 stack_clear_n_oldest (GQueue * stack, guint n)
 {
- CajaUndoStackActionData *action;
 guint i;
+ CajaUndoStackActionData *action = NULL;
 
 for (i = 0; i < n; i++) {
 if ((action = (CajaUndoStackActionData *) g_queue_pop_tail (stack)) == NULL)
@@ -1881,7 +1951,7 @@ construct_gfile_list (const GList * urilist, GFile * parent)
 {
 const GList *l;
 GList *file_list = NULL;
- GFile *file;
+ GFile *file = NULL;
 
 for (l = urilist; l != NULL; l = l->next) {
 file = g_file_get_child (parent, l->data);
@@ -1910,7 +1980,7 @@ uri_list_to_gfile_list (GList * urilist)
 {
 const GList *l;
 GList *file_list = NULL;
- GFile *file;
+ GFile *file = NULL;
 
 for (l = urilist; l != NULL; l = l->next) {
 file = g_file_new_for_uri (l->data);
diff --git a/libcaja-private/caja-view-factory.c b/libcaja-private/caja-view-factory.c
index ff16981e..abe291c1 100644
--- a/libcaja-private/caja-view-factory.c
+++ b/libcaja-private/caja-view-factory.c
@@ -40,7 +40,7 @@ const CajaViewInfo *
 caja_view_factory_lookup (const char *id)
 {
 GList *l;
- CajaViewInfo *view_info;
+ CajaViewInfo *view_info = NULL;
 
 g_return_val_if_fail (id != NULL, NULL);
 
@@ -106,7 +106,7 @@ caja_view_factory_get_views_for_uri (const char *uri,
 const char *mime_type)
 {
 GList *l, *res;
- const CajaViewInfo *view_info;
+ const CajaViewInfo *view_info = NULL;
 
 res = NULL;
author	Pablo Barciela <[email protected]>	2019-05-02 01:09:20 +0200
committer	raveit65 <[email protected]>	2019-05-06 13:53:37 +0200
commit	a110c542bd7a16625f03c97e2fefa3be8da50c85 (patch)
tree	7af078f9084b33eb0d5df7f3657a1f8049ff6b83 /libcaja-private
parent	476f56a25be636970b336d525a7766b6d1eb3fff (diff)
download	caja-a110c542bd7a16625f03c97e2fefa3be8da50c85.tar.bz2 caja-a110c542bd7a16625f03c97e2fefa3be8da50c85.tar.xz