Fix some warnings and header-file improvement
[xboard.git] / xgamelist.c
index 4eb1d38..4f86718 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * xgamelist.c -- Game list window, part of X front end for XBoard
  *
- * Copyright 1995, 2009, 2010 Free Software Foundation, Inc.
+ * Copyright 1995, 2009, 2010, 2011 Free Software Foundation, Inc.
  * ------------------------------------------------------------------------
  *
  * GNU XBoard is free software: you can redistribute it and/or modify
@@ -93,12 +93,6 @@ extern char *getenv();
 
 void SetFocus P((Widget w, XtPointer data, XEvent *event, Boolean *b));
 
-extern Widget formWidget, shellWidget, boardWidget, menuBarWidget, gameListShell;
-extern Display *xDisplay;
-extern int squareSize;
-extern Pixmap xMarkPixmap;
-extern char *layoutName;
-
 static Widget filterText;
 static char filterString[MSG_SIZ];
 static int listLength;
@@ -126,11 +120,6 @@ typedef struct {
 } GameListClosure;
 static GameListClosure *glc = NULL;
 
-static Arg layoutArgs[] = {
-    { XtNborderWidth, 0 },
-    { XtNdefaultDistance, 0 }
-};
-
 Widget
 GameListCreate(name, callback, client_data)
      char *name;
@@ -184,7 +173,7 @@ GameListCreate(name, callback, client_data)
     XtSetArg(args[j], XtNdefaultColumns, 1);  j++;
     XtSetArg(args[j], XtNforceColumns, True);  j++;
     XtSetArg(args[j], XtNverticalList, True);  j++;
-    listwidg = 
+    listwidg =
       XtCreateManagedWidget("list", listWidgetClass, viewport, args, j);
     XawListHighlight(listwidg, 0);
     XtAugmentTranslations(listwidg,
@@ -357,14 +346,13 @@ GameListPrepare()
 
 static void
 GameListReplace()
-{   // [HGM] filter: put in separate routine, to make callable from call-back
-    Arg args[16];
-    int j;
-    Widget listwidg;
+{
+  // filter: put in separate routine, to make callable from call-back
+  Widget listwidg;
 
-       listwidg = XtNameToWidget(glc->shell, "*form.viewport.list");
-       XawListChange(listwidg, glc->strings, 0, 0, True);
-       XawListHighlight(listwidg, 0);
+  listwidg = XtNameToWidget(glc->shell, "*form.viewport.list");
+  XawListChange(listwidg, glc->strings, 0, 0, True);
+  XawListHighlight(listwidg, 0);
 }
 
 void
@@ -415,7 +403,7 @@ GameListCallback(w, client_data, call_data)
         j = 0;
         XtSetArg(args[j], XtNstring, &name);  j++;
        XtGetValues(filterText, args, j);
-        strcpy(filterString, name);
+        safeStrCpy(filterString, name, sizeof(filterString)/sizeof(filterString[0]));
        XawListHighlight(listwidg, 0);
         if(GameListPrepare()) GameListReplace(); // crashes on empty list...
         return;
@@ -434,9 +422,7 @@ GameListPopUp(fp, filename)
      char *filename;
 {
     Arg args[16];
-    int j, nstrings;
-    Widget listwidg;
-    ListGame *lg;
+    int j;
     char **st;
 
     if (glc == NULL) {
@@ -461,7 +447,7 @@ GameListPopUp(fp, filename)
 
 
     if (glc->shell == NULL) {
-       glc->shell = GameListCreate(filename, GameListCallback, glc); 
+       glc->shell = GameListCreate(filename, GameListCallback, glc);
     } else {
         GameListReplace(); // [HGM] filter: code put in separate routine
        j = 0;
@@ -474,7 +460,7 @@ GameListPopUp(fp, filename)
     glc->up = True;
     j = 0;
     XtSetArg(args[j], XtNleftBitmap, xMarkPixmap); j++;
-    XtSetValues(XtNameToWidget(menuBarWidget, "menuMode.Show Game List"),
+    XtSetValues(XtNameToWidget(menuBarWidget, "menuView.Show Game List"),
                args, j);
 }
 
@@ -517,8 +503,9 @@ ShowGameListProc(w, event, prms, nprms)
     glc->up = True;
     j = 0;
     XtSetArg(args[j], XtNleftBitmap, xMarkPixmap); j++;
-    XtSetValues(XtNameToWidget(menuBarWidget, "menuMode.Show Game List"),
+    XtSetValues(XtNameToWidget(menuBarWidget, "menuView.Show Game List"),
                args, j);
+    GameListHighlight(lastLoadGameNumber);
 }
 
 void
@@ -538,7 +525,7 @@ LoadSelectedProc(w, event, prms, nprms)
     index = rs->list_index;
     if (index < 0) return;
     if(direction != 0) {
-       index += direction; 
+       index += direction;
        if(direction == -2) index = 0;
        if(direction == 2) index = listLength-1;
        if(index < 0 || index >= listLength) return;
@@ -566,7 +553,7 @@ SetFilterProc(w, event, prms, nprms)
         int j = 0;
         XtSetArg(args[j], XtNstring, &name);  j++;
        XtGetValues(filterText, args, j);
-        strcpy(filterString, name);
+        safeStrCpy(filterString, name, sizeof(filterString)/sizeof(filterString[0]));
         if(GameListPrepare()) GameListReplace(); // crashes on empty list...
        list = XtNameToWidget(glc->shell, "*form.viewport.list");
        XawListHighlight(list, 0);
@@ -598,7 +585,7 @@ GameListPopDown()
     glc->up = False;
     j = 0;
     XtSetArg(args[j], XtNleftBitmap, None); j++;
-    XtSetValues(XtNameToWidget(menuBarWidget, "menuMode.Show Game List"),
+    XtSetValues(XtNameToWidget(menuBarWidget, "menuView.Show Game List"),
                args, j);
 }
 
@@ -621,6 +608,35 @@ GameListIsUp()
     return glc && glc->up;
 }
 
+int SaveGameListAsText(FILE *f)\r
+{\r
+    ListGame * lg = (ListGame *) gameList.head;\r
+    int nItem;\r
+\r
+    if( !glc || ((ListGame *) gameList.tailPred)->number <= 0 ) {\r
+        DisplayError("Game list not loaded or empty", 0);\r
+        return False;\r
+    }\r
+\r
+    /* Copy the list into the global memory block */\r
+    if( f != NULL ) {\r
+        lg = (ListGame *) gameList.head;\r
+\r
+        for (nItem = 0; nItem < ((ListGame *) gameList.tailPred)->number; nItem++){\r
+            char * st = GameListLineFull(lg->number, &lg->gameInfo);\r
+           char *line = GameListLine(lg->number, &lg->gameInfo);
+           if(filterString[0] == NULLCHAR || SearchPattern( line, filterString ) )
+                   fprintf( f, "%s\n", st );
+           free(st); free(line);\r
+            lg = (ListGame *) lg->node.succ;\r
+        }\r
+\r
+        fclose(f);
+       return True;\r
+    }
+    return False;\r
+}\r
 //--------------------------------- Game-List options dialog ------------------------------------------
 
 Widget gameListOptShell, listwidg;
@@ -642,8 +658,8 @@ void GLT_AddToList(char *name)
 
 Boolean GLT_GetFromList(int index, char *name)
 {
-    strcpy(name, strings[index]);
-    return TRUE;
+  safeStrCpy(name, strings[index], MSG_SIZ);
+  return TRUE;
 }
 
 void GLT_DeSelectList()
@@ -655,14 +671,12 @@ void GLT_DeSelectList()
 void
 GameListOptionsPopDown()
 {
-    Arg args[16];
-    int j;
+  if (gameListOptShell == NULL) return;
 
-    if (gameListOptShell == NULL) return;
-    XtPopdown(gameListOptShell);
-    XtDestroyWidget(gameListOptShell);
-    gameListOptShell = 0;
-    XtSetKeyboardFocus(shellWidget, formWidget);
+  XtPopdown(gameListOptShell);
+  XtDestroyWidget(gameListOptShell);
+  gameListOptShell = 0;
+  XtSetKeyboardFocus(shellWidget, formWidget);
 }
 
 void
@@ -711,9 +725,9 @@ GameListOptionsCallback(w, client_data, call_data)
        strings[--index] = p;
     } else
     if (strcmp(name, _("factory")) == 0) {
-       strcpy(lpUserGLT, GLT_DEFAULT_TAGS);
-       GLT_TagsToList(lpUserGLT);
-       index = 0;
+      safeStrCpy(lpUserGLT, GLT_DEFAULT_TAGS, LPUSERGLT_SIZE);
+      GLT_TagsToList(lpUserGLT);
+      index = 0;
     }
     XawListHighlight(listwidg, index);
 }
@@ -810,7 +824,7 @@ GameListOptionsCreate()
       XtCreateManagedWidget(_("OK"), commandWidgetClass, form, args, j);
     XtAddCallback(b_close, XtNcallback, GameListOptionsCallback, client_data);
 
-    strcpy(lpUserGLT, appData.gameListTags);
+    safeStrCpy(lpUserGLT, appData.gameListTags, LPUSERGLT_SIZE);
     GLT_TagsToList(lpUserGLT);
 
     XtRealizeWidget(shell);
@@ -822,15 +836,10 @@ GameListOptionsCreate()
 void
 GameListOptionsPopUp(Widget w, XEvent *event, String *prms, Cardinal *nprms)
 {
-    Arg args[16];
-    int j, nstrings;
-    Widget listwidg;
-
-    if (gameListOptShell == NULL) {
-       gameListOptShell = GameListOptionsCreate(); 
-    }
+  if (gameListOptShell == NULL)
+    gameListOptShell = GameListOptionsCreate();
 
-    XtPopup(gameListOptShell, XtGrabNone);
+  XtPopup(gameListOptShell, XtGrabNone);
 }