Fix crash on double-click in Game List Tags
authorH.G.Muller <hgm@hgm-xboard.(none)>
Tue, 29 Jul 2014 13:03:04 +0000 (15:03 +0200)
committerH.G.Muller <hgm@hgm-xboard.(none)>
Tue, 29 Jul 2014 13:03:04 +0000 (15:03 +0200)
A double-click in the Game List Tags listbox has no function, but it
crashed XBoard, because the textValue field of teh ListBox Option definition,
supposed to contain a pointer to the user-supplied handler, contained a
pointer to an empty string.

dialogs.c

index c8505ee..23f8633 100644 (file)
--- a/dialogs.c
+++ b/dialogs.c
@@ -2107,7 +2107,7 @@ static void GLT_Button P((int n));
 static int GLT_OK P((int n));
 
 static Option listOptions[] = {
 static int GLT_OK P((int n));
 
 static Option listOptions[] = {
-{300, LR|TB, 200, NULL, (void*) strings, "", NULL, ListBox, "" }, // For GTK we need to specify a height, as default would just show 3 lines
+{300, LR|TB, 200, NULL, (void*) strings, NULL, NULL, ListBox, "" }, // For GTK we need to specify a height, as default would just show 3 lines
 { 0,    0,     0, NULL, (void*) &GLT_Button, NULL, NULL, Button, N_("factory") },
 { 0, SAME_ROW, 0, NULL, (void*) &GLT_Button, NULL, NULL, Button, N_("up") },
 { 0, SAME_ROW, 0, NULL, (void*) &GLT_Button, NULL, NULL, Button, N_("down") },
 { 0,    0,     0, NULL, (void*) &GLT_Button, NULL, NULL, Button, N_("factory") },
 { 0, SAME_ROW, 0, NULL, (void*) &GLT_Button, NULL, NULL, Button, N_("up") },
 { 0, SAME_ROW, 0, NULL, (void*) &GLT_Button, NULL, NULL, Button, N_("down") },