From bc39fbe1bfe481df10c10ad99f3c33c233deba5a Mon Sep 17 00:00:00 2001 From: Pietro Gagliardi Date: Sun, 27 Apr 2014 15:06:52 -0400 Subject: [PATCH] Resolved TODOs in listbox_unix.go; also rewrote some related if expressions there to be clearer. --- listbox_unix.go | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/listbox_unix.go b/listbox_unix.go index 3a4c5cc..8b3c279 100644 --- a/listbox_unix.go +++ b/listbox_unix.go @@ -5,6 +5,7 @@ package ui import ( + "fmt" "unsafe" ) @@ -146,7 +147,7 @@ func gListboxSelectedMulti(widget *C.GtkWidget) (indices []int) { sel := C.gtk_tree_view_get_selection(tv) rows := C.gtk_tree_selection_get_selected_rows(sel, &model) defer C.g_list_free_full(rows, C.GDestroyNotify(unsafe.Pointer(C.gtk_tree_path_free))) - // TODO needed? + // g_list_length() is O(N), but we need the length below, alas len := C.g_list_length(rows) if len == 0 { return nil @@ -177,9 +178,8 @@ func gListboxSelMultiTexts(widget *C.GtkWidget) (texts []string) { texts = make([]string, len) for i := C.guint(0); i < len; i++ { path := (*C.GtkTreePath)(unsafe.Pointer(rows.data)) - if !fromgbool(C.gtk_tree_model_get_iter(model, &iter, path)) { - // TODO - return + if C.gtk_tree_model_get_iter(model, &iter, path) == C.FALSE { + panic("gtk_tree_model_get_iter() failed getting Listbox selected texts; reason unknown") } C.gtkTreeModelGet(model, &iter, &gs) texts[i] = fromgstr(gs) @@ -193,9 +193,8 @@ func gListboxDelete(widget *C.GtkWidget, index int) { tv := getTreeViewFrom(widget) ls := (*C.GtkListStore)(unsafe.Pointer(C.gtk_tree_view_get_model(tv))) - if !fromgbool(C.gtk_tree_model_iter_nth_child((*C.GtkTreeModel)(unsafe.Pointer(ls)), &iter, (*C.GtkTreeIter)(nil), C.gint(index))) { // no such index - // TODO - return + if C.gtk_tree_model_iter_nth_child((*C.GtkTreeModel)(unsafe.Pointer(ls)), &iter, (*C.GtkTreeIter)(nil), C.gint(index)) == C.FALSE { + panic(fmt.Errorf("error deleting row %d from GTK+ Listbox: no such index or some other error", index)) } C.gtk_list_store_remove(ls, &iter) }