diff options
author | John Sullivan <sullivan@src.gnome.org> | 2000-09-29 02:18:25 +0000 |
---|---|---|
committer | John Sullivan <sullivan@src.gnome.org> | 2000-09-29 02:18:25 +0000 |
commit | 855ed4b96dce1d8596531ac2a9a5ed44b2cb9fc8 (patch) | |
tree | f2cf3912affd10763847dee27240c9d21bb0d5a3 /src/nautilus-view-frame.c | |
parent | 335d5f982249ed6cc6a1dca00d0ae991ceaf06e7 (diff) | |
download | nautilus-855ed4b96dce1d8596531ac2a9a5ed44b2cb9fc8.tar.gz |
Fixed part of bug 3322 (context menus don't work correctly
in icon view when using FTP). Now there is no degenerate menu
displayed, but the real menu still doesn't appear due to the
remaining selection bug.
Also did some more work to make some sidebar view failures
less deadly to Nautilus.
* src/file-manager/fm-directory-view.c:
(create_selection_context_menu): Complain and return NULL if
no files are selected.
(fm_directory_view_pop_up_selection_context_menu):
Don't put up a menu at all if create_selection_context_menu
returned NULL.
* src/nautilus-window-private.h:
Added field details->dead_view_name.
* src/nautilus-view-frame.c: (nautilus_view_frame_get_label):
return-if-fail if view frame has already been destroyed but
not finalized (so details is NULL). It used to segfault. I
also changed the one caller I knew about that sometimes called
this function in this situation.
* src/nautilus-window-manage-views.c:
(nautilus_window_set_state_info): in VIEW_ERROR case, store the
name of the (first) failing view frame here, where it can still
be reliably read.
(nautilus_window_update_state): Removed redundant and misleading
window->error_views != NULL check; changed if/else logic to
not check whether new_content_view is also content_view, since
when this check failed the code would assume that new_content_view
was a sidebar panel (I don't know of anyone ever running into
this case though); free and NULL out dead_view_name after
processing first error_view.
(report_sidebar_panel_failure_to_user): Use previously-stored
dead_view_name rather than trusting the view frame to still
have useful information in this case, since it doesn't in
the many cases where it self-destructed when running into
a problem.
(nautilus_window_end_location_change_callback): Tweaked an
error message.
* src/nautilus-window.c: (nautilus_window_destroy):
free details->dead_view_name.
Diffstat (limited to 'src/nautilus-view-frame.c')
-rw-r--r-- | src/nautilus-view-frame.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/nautilus-view-frame.c b/src/nautilus-view-frame.c index e0df41bd1..ecbaefe48 100644 --- a/src/nautilus-view-frame.c +++ b/src/nautilus-view-frame.c @@ -985,6 +985,7 @@ char * nautilus_view_frame_get_label (NautilusViewFrame *view) { g_return_val_if_fail (NAUTILUS_IS_VIEW_FRAME (view), NULL); + g_return_val_if_fail (view->details != NULL, NULL); return g_strdup (view->details->label); } |