From 1620f20c4d1e13b05f108407508653841d4887eb Mon Sep 17 00:00:00 2001 From: Vincent Sanders Date: Thu, 30 Oct 2014 22:43:16 +0000 Subject: make global history treeview callback return error returns an error instead of directly waning the user. --- desktop/global_history.c | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) (limited to 'desktop/global_history.c') diff --git a/desktop/global_history.c b/desktop/global_history.c index 7d679e5a2..aaf4239ad 100644 --- a/desktop/global_history.c +++ b/desktop/global_history.c @@ -661,10 +661,12 @@ static nserror global_history_tree_node_folder_cb( return NSERROR_OK; } -static nserror global_history_tree_node_entry_cb( - struct treeview_node_msg msg, void *data) + +static nserror +global_history_tree_node_entry_cb(struct treeview_node_msg msg, void *data) { struct global_history_entry *e = data; + nserror ret = NSERROR_OK; switch (msg.msg) { case TREE_MSG_NODE_DELETE: @@ -678,10 +680,8 @@ static nserror global_history_tree_node_entry_cb( case TREE_MSG_NODE_LAUNCH: { - nserror error; struct browser_window *existing = NULL; - enum browser_window_create_flags flags = - BW_CREATE_HISTORY; + enum browser_window_create_flags flags = BW_CREATE_HISTORY; /* TODO: Set existing to window that new tab appears in */ @@ -693,16 +693,14 @@ static nserror global_history_tree_node_entry_cb( /* TODO: flags ^= BW_CREATE_TAB; */ } - error = browser_window_create(flags, e->url, NULL, + ret = browser_window_create(flags, e->url, NULL, existing, NULL); - if (error != NSERROR_OK) { - warn_user(messages_get_errorcode(error), 0); - } } break; } - return NSERROR_OK; + return ret; } + struct treeview_callback_table gh_tree_cb_t = { .folder = global_history_tree_node_folder_cb, .entry = global_history_tree_node_entry_cb -- cgit v1.2.3