diff options
author | Steven Rostedt <srostedt@redhat.com> | 2010-01-07 17:08:39 -0500 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2010-01-07 17:08:39 -0500 |
commit | 4c15ef754ee624337f7c0c5acc93a685aa19a40c (patch) | |
tree | 958a0e32b19e321c1a73d48c78469eb4dd781f8f | |
parent | 21cc816a3c027f9ffe3466588dcc783702fdb642 (diff) |
trace-view: Do not select a row if there are none
Selecting a row use to give errors when there were no rows.
Just prevent a row from being selected when there are none.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
-rw-r--r-- | trace-view.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/trace-view.c b/trace-view.c index a00e877..fef2a37 100644 --- a/trace-view.c +++ b/trace-view.c | |||
@@ -302,7 +302,7 @@ void trace_view_update_filters(GtkWidget *treeview, | |||
302 | g_object_unref(model); | 302 | g_object_unref(model); |
303 | 303 | ||
304 | /* Keep selection near previous selection */ | 304 | /* Keep selection near previous selection */ |
305 | if (row >= 0 && trace_view_store_visible_rows(TRACE_VIEW_STORE(model))) | 305 | if (row >= 0) |
306 | trace_view_select(treeview, time); | 306 | trace_view_select(treeview, time); |
307 | } | 307 | } |
308 | 308 | ||
@@ -321,6 +321,10 @@ void trace_view_select(GtkWidget *treeview, guint64 time) | |||
321 | /* This can be called when we NULL out the model */ | 321 | /* This can be called when we NULL out the model */ |
322 | if (!model) | 322 | if (!model) |
323 | return; | 323 | return; |
324 | |||
325 | if (!trace_view_store_visible_rows(TRACE_VIEW_STORE(model))) | ||
326 | return; | ||
327 | |||
324 | page = trace_view_store_get_page(TRACE_VIEW_STORE(model)); | 328 | page = trace_view_store_get_page(TRACE_VIEW_STORE(model)); |
325 | select_page = trace_view_store_get_timestamp_page(TRACE_VIEW_STORE(model), | 329 | select_page = trace_view_store_get_timestamp_page(TRACE_VIEW_STORE(model), |
326 | time); | 330 | time); |