diff --git a/unix/newcontrol.c b/unix/newcontrol.c
index a1a29e93..82342a65 100644
--- a/unix/newcontrol.c
+++ b/unix/newcontrol.c
@@ -170,8 +170,7 @@ static void onDestroy(GtkWidget *widget, gpointer data)
 	singleWidget *s = (singleWidget *) data;
 
 	if (!s->canDestroy)
-		// TODO switch to complain()
-		g_error("trying to destroy control with singleWidget at %p before uiControlDestroy()", s);
+		complain("trying to destroy control with singleWidget at %p before uiControlDestroy()", s);
 	uiFree(s);
 }
 
diff --git a/unix/parent.c b/unix/parent.c
index 451fdd17..a279c814 100644
--- a/unix/parent.c
+++ b/unix/parent.c
@@ -44,8 +44,7 @@ static void uipParent_dispose(GObject *obj)
 
 	// don't free mainControl here; that should have been done by uiParentDestroy()
 	if (!p->canDestroy)
-		// TODO switch to complain()
-		g_error("attempt to dispose uiParent with uipParent at %p before uiParentDestroy()", p);
+		complain("attempt to dispose uiParent with uipParent at %p before uiParentDestroy()", p);
 	if (p->children != NULL) {
 		g_ptr_array_unref(p->children);
 		p->children = NULL;
@@ -58,8 +57,7 @@ static void uipParent_finalize(GObject *obj)
 	uipParent *p = uipParent(obj);
 
 	if (!p->canDestroy)
-		// TODO switch to complain()
-		g_error("attempt to finalize uiParent with uipParent at %p before uiParentDestroy()", p);
+		complain("attempt to finalize uiParent with uipParent at %p before uiParentDestroy()", p);
 	G_OBJECT_CLASS(uipParent_parent_class)->finalize(obj);
 	if (options.debugLogAllocations)
 		fprintf(stderr, "%p free\n", obj);
diff --git a/unix/window.c b/unix/window.c
index f89541c9..a64cd189 100644
--- a/unix/window.c
+++ b/unix/window.c
@@ -33,8 +33,7 @@ static void onDestroy(GtkWidget *widget, gpointer data)
 	struct window *w = (struct window *) data;
 
 	if (!w->canDestroy)
-		// TODO switch to complain()
-		g_error("attempt to dispose uiWindow at %p before uiWindowDestroy()", w);
+		complain("attempt to dispose uiWindow at %p before uiWindowDestroy()", w);
 	uiFree(w);
 }