Merge lp:~pqp/pantheon-files/fix-1079365 into lp:~elementary-apps/pantheon-files/trunk

Proposed by Patrick Pace
Status: Merged
Merged at revision: 1031
Proposed branch: lp:~pqp/pantheon-files/fix-1079365
Merge into: lp:~elementary-apps/pantheon-files/trunk
Diff against target: 39 lines (+16/-7)
1 file modified
src/fm-directory-view.c (+16/-7)
To merge this branch: bzr merge lp:~pqp/pantheon-files/fix-1079365
Reviewer Review Type Date Requested Status
Cody Garver (community) Approve
Review via email: mp+135999@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Cody Garver (codygarver) wrote :

Greatness.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/fm-directory-view.c'
2--- src/fm-directory-view.c 2012-11-06 22:19:43 +0000
3+++ src/fm-directory-view.c 2012-11-24 04:39:19 +0000
4@@ -2644,10 +2644,15 @@
5
6 locations = g_list_reverse (locations);
7
8- marlin_file_operations_trash_or_delete (locations,
9- GTK_WINDOW (view->details->window),
10- (MarlinDeleteCallback) trash_or_delete_done_cb,
11- view);
12+ if (locations != NULL) {
13+ marlin_file_operations_trash_or_delete (locations,
14+ GTK_WINDOW (view->details->window),
15+ (MarlinDeleteCallback) trash_or_delete_done_cb,
16+ view);
17+ }
18+ else
19+ return;
20+
21 g_list_free_full (locations, g_object_unref);
22 }
23
24@@ -2662,9 +2667,13 @@
25 */
26 if (!view->details->selection_was_removed) {
27 selection = fm_directory_view_get_selection_for_file_transfer (view);
28- trash_or_delete_files (view, selection, TRUE);
29- gof_file_list_free (selection);
30- view->details->selection_was_removed = TRUE;
31+ if (selection != NULL) {
32+ trash_or_delete_files (view, selection, TRUE);
33+ gof_file_list_free (selection);
34+ view->details->selection_was_removed = TRUE;
35+ }
36+ else
37+ return;
38 }
39 }
40

Subscribers

People subscribed via source and target branches

to all changes: