Debugger: Refactor widget packing

This fixes a cppcheck 2.2+ confusion about uninitialized widgets array
and also makes the code a bit more readable.
Closes geany#1014.

debugger/src/tpage.c

@@ -139,22 +139,20 @@ void tpage_pack_widgets(gboolean tabbed)
 {
 	/* root box */
 	GtkWidget *root = NULL, *oldroot = NULL;
-	GtkWidget **widget = NULL;
 	GList *children = gtk_container_get_children(GTK_CONTAINER(tab_target));
 	if (children)
 	{
-		int i = 0;
-
+		GtkWidget **widget_ref = NULL;
+		
 		oldroot = (GtkWidget*)children->data;
 
 		/* unparent widgets */
-		do
+		for (int i = 0; widgets[i]; i++)
 		{
-			widget = widgets[i];
-			g_object_ref(widget);
-			gtk_container_remove(GTK_CONTAINER(gtk_widget_get_parent(*widget)), *widget);
-			i++;
-		} while (widget);
+			widget_ref = widgets[i];
+			g_object_ref(*widget_ref);
+			gtk_container_remove(GTK_CONTAINER(gtk_widget_get_parent(*widget_ref)), *widget_ref);
+		}
 
 		g_list_free(children);
 	}
@@ -266,13 +264,10 @@ void tpage_pack_widgets(gboolean tabbed)
 
 	if (oldroot)
 	{
-		int i = 0;
-		do
+		for (int i = 0; widgets[i]; i++)
 		{
-			widget = widgets[i];
 			g_object_unref(*widgets[i]);
-			i++;
-		} while (widget);
+		}
 		gtk_container_remove(GTK_CONTAINER(tab_target), oldroot);
 	}