From a110c542bd7a16625f03c97e2fefa3be8da50c85 Mon Sep 17 00:00:00 2001 From: Pablo Barciela Date: Thu, 2 May 2019 01:09:20 +0200 Subject: libcaja-private: reduce the scope of some variables --- libcaja-private/caja-autorun.c | 3 +- libcaja-private/caja-bookmark.c | 9 +- libcaja-private/caja-clipboard-monitor.c | 13 ++- libcaja-private/caja-clipboard.c | 5 +- libcaja-private/caja-customization-data.c | 14 ++- libcaja-private/caja-debug-log.c | 7 +- libcaja-private/caja-desktop-icon-file.c | 6 +- libcaja-private/caja-desktop-link-monitor.c | 13 ++- libcaja-private/caja-desktop-metadata.c | 6 +- libcaja-private/caja-directory-async.c | 122 +++++++++++--------- libcaja-private/caja-directory.c | 41 ++++--- libcaja-private/caja-dnd.c | 14 ++- libcaja-private/caja-entry.c | 3 +- libcaja-private/caja-file-changes-queue.c | 4 +- libcaja-private/caja-file-conflict-dialog.c | 3 +- libcaja-private/caja-file-dnd.c | 7 +- libcaja-private/caja-file-operations.c | 123 ++++++++++++-------- libcaja-private/caja-file-utilities.c | 52 ++++++--- libcaja-private/caja-file.c | 146 +++++++++++++++--------- libcaja-private/caja-icon-canvas-item.c | 45 +++++--- libcaja-private/caja-icon-container.c | 133 ++++++++++++--------- libcaja-private/caja-icon-dnd.c | 39 ++++--- libcaja-private/caja-icon-info.c | 4 +- libcaja-private/caja-keep-last-vertical-box.c | 4 +- libcaja-private/caja-link.c | 8 +- libcaja-private/caja-metadata.c | 3 +- libcaja-private/caja-mime-actions.c | 90 +++++++++------ libcaja-private/caja-mime-application-chooser.c | 3 +- libcaja-private/caja-monitor.c | 3 +- libcaja-private/caja-open-with-dialog.c | 40 ++++--- libcaja-private/caja-program-choosing.c | 47 +++++--- libcaja-private/caja-progress-info.c | 24 ++-- libcaja-private/caja-query.c | 14 ++- libcaja-private/caja-search-directory-file.c | 11 +- libcaja-private/caja-search-directory.c | 23 ++-- libcaja-private/caja-search-engine-beagle.c | 7 +- libcaja-private/caja-search-engine-simple.c | 7 +- libcaja-private/caja-search-engine-tracker.c | 6 +- libcaja-private/caja-thumbnails.c | 20 ++-- libcaja-private/caja-trash-monitor.c | 8 +- libcaja-private/caja-tree-view-drag-dest.c | 20 +++- libcaja-private/caja-ui-utilities.c | 12 +- libcaja-private/caja-undostack-manager.c | 104 ++++++++++++++--- libcaja-private/caja-view-factory.c | 4 +- 44 files changed, 794 insertions(+), 476 deletions(-) (limited to 'libcaja-private') 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, " %s\n", uri); g_free (uri); @@ -406,6 +406,8 @@ caja_query_to_xml (CajaQuery *query) g_string_append (xml, " \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, " %s\n", mimetype); g_free (mimetype); @@ -418,7 +420,9 @@ caja_query_to_xml (CajaQuery *query) g_string_append (xml, " \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, " %s\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; -- cgit v1.2.1