afsmonitor & gtx: Don't cast returns from malloc()
[openafs.git] / src / gtx / frame.c
index 694b8a1..c219c7e 100644 (file)
@@ -1,21 +1,18 @@
 /*
  * Copyright 2000, International Business Machines Corporation and others.
  * All Rights Reserved.
- * 
+ *
  * This software has been released under the terms of the IBM Public
  * License.  For details, see the LICENSE file in the top-level source
  * directory or online at http://www.openafs.org/dl/license10.html
  */
 
-#define        IGNORE_STDS_H
-#include <afs/param.h>
 #include <afsconfig.h>
+#include <afs/param.h>
+#include <afs/stds.h>
 
-RCSID("$Header$");
+#include <roken.h>
 
-#ifdef AFS_HPUX_ENV
-#include <sys/types.h>
-#endif
 #include <lwp.h>
 
 #include "gtxobjects.h"
@@ -25,41 +22,48 @@ RCSID("$Header$");
 #include "gtxkeymap.h"
 #include "gtxframe.h"
 
-extern char *gtx_CopyString();
-static struct keymap_map *recursiveMap=0;
-static char menubuffer[1024];          /*Buffer for menu selections*/
-int gtxframe_exitValue = 0;            /*Program exit value*/
+static struct keymap_map *recursiveMap = 0;
+static char menubuffer[1024];  /*Buffer for menu selections */
+int gtxframe_exitValue = 0;    /*Program exit value */
 
-gtxframe_CtrlUCmd(awindow, arock)
-char *arock;
-struct gwin *awindow; {
+int
+gtxframe_CtrlUCmd(void *aparam, void *arock)
+{
+    struct gwin *awindow = (struct gwin *) aparam;
     struct gtx_frame *tframe;
 
     tframe = awindow->w_frame;
-    if (!tframe->defaultLine) return -1;
+    if (!tframe->defaultLine)
+       return -1;
     *(tframe->defaultLine) = 0;
     return 0;
 }
 
-gtxframe_CtrlHCmd(awindow, arock)
-char *arock;
-struct gwin *awindow; {
-    register struct gtx_frame *tframe;
-    register char *tp;
-    register int pos;
+int
+gtxframe_CtrlHCmd(void *aparam, void *arock)
+{
+    struct gwin *awindow = (struct gwin *) aparam;
+
+    struct gtx_frame *tframe;
+    char *tp;
+    int pos;
 
     tframe = awindow->w_frame;
-    if (!(tp=tframe->defaultLine)) return -1;
+    if (!(tp = tframe->defaultLine))
+       return -1;
     pos = strlen(tp);
-    if (pos == 0) return 0;    /* rubout at the end of the line */
-    tp[pos-1] = 0;
+    if (pos == 0)
+       return 0;               /* rubout at the end of the line */
+    tp[pos - 1] = 0;
     return 0;
 }
 
-gtxframe_RecursiveEndCmd(awindow, arock)
-char *arock;
-register struct gwin *awindow; {
-    register struct gtx_frame *tframe;
+int
+gtxframe_RecursiveEndCmd(void *aparam, void *arock)
+{
+    struct gwin *awindow = (struct gwin *) aparam;
+
+    struct gtx_frame *tframe;
 
     tframe = awindow->w_frame;
     tframe->flags |= GTXFRAME_RECURSIVEEND;
@@ -67,10 +71,12 @@ register struct gwin *awindow; {
     return 0;
 }
 
-gtxframe_RecursiveErrCmd(awindow, arock)
-char *arock;
-register struct gwin *awindow; {
-    register struct gtx_frame *tframe;
+int
+gtxframe_RecursiveErrCmd(void *aparam, void *arock)
+{
+    struct gwin *awindow = (struct gwin *) aparam;
+
+    struct gtx_frame *tframe;
 
     tframe = awindow->w_frame;
     tframe->flags |= GTXFRAME_RECURSIVEEND;
@@ -78,75 +84,58 @@ register struct gwin *awindow; {
     return 0;
 }
 
-gtxframe_SelfInsertCmd(awindow, arock)
-int arock;
-struct gwin *awindow; {
-    register struct gtx_frame *tframe;
-    register int pos;
-    register char *tp;
+int
+gtxframe_SelfInsertCmd(void *aparam, void *rockparam)
+{
+    struct gwin *awindow = (struct gwin *) aparam;
+
+    int arock = (intptr_t)rockparam;
+
+    struct gtx_frame *tframe;
+    int pos;
+    char *tp;
 
     tframe = awindow->w_frame;
-    if (!(tp = tframe->defaultLine)) return -1;
+    if (!(tp = tframe->defaultLine))
+       return -1;
     pos = strlen(tp);
-    tp[pos] = arock;   /* arock has char to insert */
-    tp[pos+1] = 0;     /* null-terminate it, too */
+    tp[pos] = arock;           /* arock has char to insert */
+    tp[pos + 1] = 0;           /* null-terminate it, too */
     return 0;
 }
 
 /* save map, setup recursive map and install it */
-static SaveMap(aframe)
-register struct gtx_frame *aframe; {
+static int
+SaveMap(struct gtx_frame *aframe)
+{
     char tstring[2];
-    register int i;
+    int i;
 
     if (!recursiveMap) {
        /* setup recursive edit map if not previously done */
        recursiveMap = keymap_Create();
-       keymap_BindToString(recursiveMap,
-                           "\010",
-                           gtxframe_CtrlHCmd,
-                           (char *) 0,
-                           (char *) 0);
-       keymap_BindToString(recursiveMap,
-                           "\177",gtxframe_CtrlHCmd,
-                           (char *) 0,
-                           (char *) 0);
-       keymap_BindToString(recursiveMap,
-                           "\025",
-                           gtxframe_CtrlUCmd,
-                           (char *) 0,
-                           (char *) 0);
-       keymap_BindToString(recursiveMap,
-                           "\033",
-                           gtxframe_RecursiveEndCmd,
-                           (char *) 0,
-                           (char *) 0);
-       keymap_BindToString(recursiveMap,
-                           "\015",
-                           gtxframe_RecursiveEndCmd,
-                           (char *) 0,
-                           (char *) 0);
-       keymap_BindToString(recursiveMap,
-                           "\012",
-                           gtxframe_RecursiveEndCmd,
-                           (char *) 0,
-                           (char *) 0);
-       keymap_BindToString(recursiveMap,
-                           "\003",
-                           gtxframe_RecursiveErrCmd,
-                           (char *) 0,
-                           (char *) 0);
-       keymap_BindToString(recursiveMap,
-                           "\007",
-                           gtxframe_RecursiveErrCmd,
-                           (char *) 0,
-                           (char *) 0);
-
-       for(i=040; i<0177; i++) {
+       keymap_BindToString(recursiveMap, "\010", gtxframe_CtrlHCmd, NULL,
+                           NULL);
+       keymap_BindToString(recursiveMap, "\177", gtxframe_CtrlHCmd, NULL,
+                           NULL);
+       keymap_BindToString(recursiveMap, "\025", gtxframe_CtrlUCmd, NULL,
+                           NULL);
+       keymap_BindToString(recursiveMap, "\033", gtxframe_RecursiveEndCmd,
+                           NULL, NULL);
+       keymap_BindToString(recursiveMap, "\015", gtxframe_RecursiveEndCmd,
+                           NULL, NULL);
+       keymap_BindToString(recursiveMap, "\012", gtxframe_RecursiveEndCmd,
+                           NULL, NULL);
+       keymap_BindToString(recursiveMap, "\003", gtxframe_RecursiveErrCmd,
+                           NULL, NULL);
+       keymap_BindToString(recursiveMap, "\007", gtxframe_RecursiveErrCmd,
+                           NULL, NULL);
+
+       for (i = 040; i < 0177; i++) {
            tstring[0] = i;
            tstring[1] = 0;
            keymap_BindToString(recursiveMap, tstring, gtxframe_SelfInsertCmd,
-                               (char *) 0, i);
+                               NULL, (void *)(intptr_t)i);
        }
     }
     aframe->savemap = aframe->keymap;
@@ -156,112 +145,111 @@ register struct gtx_frame *aframe; {
 }
 
 /* Restore map to previous value */
-static RestoreMap(aframe)
-register struct gtx_frame *aframe; {
+static int
+RestoreMap(struct gtx_frame *aframe)
+{
     aframe->keymap = aframe->savemap;
-    aframe->savemap = (struct keymap_map *) 0;
+    aframe->savemap = (struct keymap_map *)0;
     keymap_InitState(aframe->keystate, aframe->keymap);
     return 0;
 }
 
-gtxframe_SetFrame(awin, aframe)
-struct gtx_frame *aframe;
-struct gwin *awin; {
+int
+gtxframe_SetFrame(struct gwin *awin, struct gtx_frame *aframe)
+{
     if (awin->w_frame) {
        /* Unthread this frame */
-       awin->w_frame->window = (struct gwin *) 0;
+       awin->w_frame->window = NULL;
     }
     awin->w_frame = aframe;
     aframe->window = awin;     /* Set frame's window ptr */
     return 0;
 }
 
-struct gtx_frame *gtxframe_GetFrame(awin)
-struct gwin *awin; {
+struct gtx_frame *
+gtxframe_GetFrame(struct gwin *awin)
+{
     return awin->w_frame;
 }
 
 /* Add a menu string to display list */
-gtxframe_AddMenu(aframe, alabel, astring)
-char *alabel;
-char *astring;
-struct gtx_frame *aframe; {
-    register struct gtxframe_menu *tmenu;
+int
+gtxframe_AddMenu(struct gtx_frame *aframe, char *alabel, char *astring)
+{
+    struct gtxframe_menu *tmenu;
 
     if (aframe->menus)
-      for(tmenu = aframe->menus; tmenu; tmenu = tmenu->next) {
-       if (strcmp(alabel, tmenu->name)==0) break;
-      }
-    else
-      tmenu = (struct gtxframe_menu *)0;
+       for (tmenu = aframe->menus; tmenu; tmenu = tmenu->next) {
+           if (strcmp(alabel, tmenu->name) == 0)
+               break;
+    } else
+       tmenu = (struct gtxframe_menu *)0;
     if (!tmenu) {
        /* Handle everything but the command string, which is handled by the
-          common-case code below */
-       tmenu = (struct gtxframe_menu *) malloc(sizeof(*tmenu));
+        * common-case code below */
+       tmenu = calloc(1, sizeof(*tmenu));
        if (tmenu == (struct gtxframe_menu *)0)
-         return(-1);
-       bzero(tmenu, sizeof(*tmenu));
+           return (-1);
        tmenu->next = aframe->menus;
        aframe->menus = tmenu;
        tmenu->name = gtx_CopyString(alabel);
     }
 
     /*
-      * Common case: redo the string labels.  Note: at this point, tmenu
-      * points to a valid menu.
-      */
+     * Common case: redo the string labels.  Note: at this point, tmenu
+     * points to a valid menu.
+     */
     if (tmenu->cmdString)
-      free(tmenu->cmdString);
+       free(tmenu->cmdString);
     tmenu->cmdString = gtx_CopyString(astring);
     return 0;
 }
 
 /* Delete a given menu from a frame*/
-gtxframe_DeleteMenu(aframe, alabel)
-struct gtx_frame *aframe;
-char *alabel; {
-    register struct gtxframe_menu *tm, **lm;
+int
+gtxframe_DeleteMenu(struct gtx_frame *aframe, char *alabel)
+{
+    struct gtxframe_menu *tm, **lm;
 
-    for(lm = &aframe->menus, tm = *lm; tm; lm = &tm->next, tm = *lm) {
+    for (lm = &aframe->menus, tm = *lm; tm; lm = &tm->next, tm = *lm) {
        if (strcmp(alabel, tm->name) == 0) {
            /* found it, remove and return success */
-           *lm = tm->next;             /* unthread from list */
+           *lm = tm->next;     /* unthread from list */
            free(tm->name);
            free(tm->cmdString);
            free(tm);
-           return(0);
+           return (0);
        }
     }
-    return(-1);        /* failed to find entry to delete */
+    return (-1);               /* failed to find entry to delete */
 }
 
 /* Function to remove all known menus */
-gtxframe_ClearMenus(aframe)
-struct gtx_frame *aframe; {
+int
+gtxframe_ClearMenus(struct gtx_frame *aframe)
+{
 
-    register struct gtxframe_menu *tm, *nm;
+    struct gtxframe_menu *tm, *nm;
 
     if (aframe->menus != (struct gtxframe_menu *)0) {
-      for(tm=aframe->menus; tm; tm=nm) {
-       nm = tm->next;
-       free(tm->name);
-       free(tm->cmdString);
-       free(tm);
-      }
+       for (tm = aframe->menus; tm; tm = nm) {
+           nm = tm->next;
+           free(tm->name);
+           free(tm->cmdString);
+           free(tm);
+       }
     }
 
-    aframe->menus = (struct gtxframe_menu *) 0;
+    aframe->menus = (struct gtxframe_menu *)0;
     return 0;
 }
 
-gtxframe_AskForString(aframe, aprompt, adefault, aresult, aresultSize)
-register struct gtx_frame *aframe;
-char *aprompt;
-char *adefault;
-char *aresult;
-int aresultSize; {
-    register int code;
-    register char *tp;
+int
+gtxframe_AskForString(struct gtx_frame *aframe, char *aprompt,
+                     char *adefault, char *aresult, int aresultSize)
+{
+    int code;
+    char *tp;
 
     /* Ensure recursive-edit map is initialized */
     SaveMap(aframe);
@@ -272,9 +260,9 @@ int aresultSize; {
     if (aframe->defaultLine)
        free(aframe->defaultLine);
     aframe->promptLine = gtx_CopyString(aprompt);
-    tp = aframe->defaultLine = (char *) malloc(1024);
-    if (tp == (char *)0)
-      return(-1);
+    tp = aframe->defaultLine = malloc(1024);
+    if (tp == NULL)
+       return (-1);
     if (adefault)
        strcpy(tp, adefault);
     else
@@ -290,7 +278,7 @@ int aresultSize; {
        goto done;
     }
     code = strlen(tp);
-    if (code+1 > aresultSize) {
+    if (code + 1 > aresultSize) {
        code = -2;
        goto done;
     }
@@ -304,74 +292,76 @@ int aresultSize; {
        free(aframe->promptLine);
     if (aframe->defaultLine)
        free(aframe->defaultLine);
-    aframe->defaultLine = aframe->promptLine = (char *) 0;
+    aframe->defaultLine = aframe->promptLine = NULL;
     if (code)
        gtxframe_DisplayString(aframe, "[Aborted]");
-    return(code);
+    return (code);
 }
 
-gtxframe_DisplayString(aframe, amsgLine)
-char *amsgLine;
-register struct gtx_frame *aframe; {
+int
+gtxframe_DisplayString(struct gtx_frame *aframe, char *amsgLine)
+{
     if (aframe->messageLine)
-      free(aframe->messageLine);
+       free(aframe->messageLine);
     aframe->messageLine = gtx_CopyString(amsgLine);
+    return 0;
 }
 
 /* Called by input processor to try to clear the dude */
-gtxframe_ClearMessageLine(aframe)
-struct gtx_frame *aframe; {
+int
+gtxframe_ClearMessageLine(struct gtx_frame *aframe)
+{
     /* If we haven't shown message long enough yet, just return */
     if (aframe->flags & GTXFRAME_NEWDISPLAY)
-      return(0);
+       return (0);
     if (aframe->messageLine)
        free(aframe->messageLine);
-    aframe->messageLine = (char *) 0;
-    return(0);
+    aframe->messageLine = NULL;
+    return (0);
 }
 
-static ShowMessageLine(aframe)
-struct gtx_frame *aframe; {
+static int
+ShowMessageLine(struct gtx_frame *aframe)
+{
     struct gwin_strparams strparms;
     struct gwin_sizeparams sizeparms;
-    register char *tp;
+    char *tp;
 
-    if (!aframe->window) return -1;
+    if (!aframe->window)
+       return -1;
 
     /* First, find window size */
     WOP_GETDIMENSIONS(aframe->window, &sizeparms);
 
     if (aframe->promptLine) {
-       bzero(&strparms, sizeof(strparms));
+       memset(&strparms, 0, sizeof(strparms));
        strparms.x = 0;
-       strparms.y = sizeparms.maxy-1;
+       strparms.y = sizeparms.maxy - 1;
        strparms.highlight = 1;
-       tp = strparms.s = (char *) malloc(1024);
+       tp = strparms.s = malloc(1024);
        strcpy(tp, aframe->promptLine);
        strcat(tp, aframe->defaultLine);
        WOP_DRAWSTRING(aframe->window, &strparms);
        aframe->flags |= GTXFRAME_NEWDISPLAY;
-    }
-    else if (aframe->messageLine) {
+    } else if (aframe->messageLine) {
        /* Otherwise we're visible, print the message at the bottom */
-       bzero(&strparms, sizeof(strparms));
+       memset(&strparms, 0, sizeof(strparms));
        strparms.highlight = 1;
        strparms.x = 0;
-       strparms.y = sizeparms.maxy-1;
+       strparms.y = sizeparms.maxy - 1;
        strparms.s = aframe->messageLine;
        WOP_DRAWSTRING(aframe->window, &strparms);
        aframe->flags |= GTXFRAME_NEWDISPLAY;
     }
-    return(0);
+    return (0);
 }
 
 /* Exit function, returning whatever has been put in its argument */
-int gtxframe_ExitCmd(a_exitValuep)
-    char *a_exitValuep;
-
-{ /*gtxframe_ExitCmd*/
+int
+gtxframe_ExitCmd(void *a_exitValuep, void *arock)
+{                              /*gtxframe_ExitCmd */
 
-    int exitval;       /*Value we've been asked to exit with*/
+    int exitval;               /*Value we've been asked to exit with */
 
     /* This next call should be type independent! */
     gator_cursesgwin_cleanup(&gator_basegwin);
@@ -379,166 +369,165 @@ int gtxframe_ExitCmd(a_exitValuep)
     exitval = *((int *)(a_exitValuep));
     exit(exitval);
 
-} /*gtxframe_ExitCmd*/
+}                              /*gtxframe_ExitCmd */
 
-struct gtx_frame *gtxframe_Create() {
+struct gtx_frame *
+gtxframe_Create(void)
+{
     struct gtx_frame *tframe;
     struct keymap_map *newkeymap;
     struct keymap_state *newkeystate;
 
     /*
-      * Allocate all the pieces first: frame, keymap, and key state.
-      */
-    tframe = (struct gtx_frame *) malloc (sizeof(struct gtx_frame));
+     * Allocate all the pieces first: frame, keymap, and key state.
+     */
+    tframe = calloc(1, sizeof(struct gtx_frame));
     if (tframe == (struct gtx_frame *)0) {
-      return((struct gtx_frame *)0);
+       return ((struct gtx_frame *)0);
     }
 
     newkeymap = keymap_Create();
     if (newkeymap == (struct keymap_map *)0) {
-      /*
-       * Get rid of the frame before exiting.
-       */
-      free(tframe);
-      return((struct gtx_frame *)0);
+       /*
+        * Get rid of the frame before exiting.
+        */
+       free(tframe);
+       return ((struct gtx_frame *)0);
     }
 
-    newkeystate = (struct keymap_state *)
-      malloc(sizeof(struct keymap_state));
+    newkeystate = malloc(sizeof(struct keymap_state));
     if (newkeystate == (struct keymap_state *)0) {
-      /*
-       * Get rid of the frame AND the keymap before exiting.
-       */
-      free(tframe);
-      free(newkeymap);
-      return((struct gtx_frame *)0);
+       /*
+        * Get rid of the frame AND the keymap before exiting.
+        */
+       free(tframe);
+       free(newkeymap);
+       return ((struct gtx_frame *)0);
     }
 
     /*
-      * Now that all the pieces exist, fill them in and stick them in
-      * the right places.
-      */
-    bzero(tframe, sizeof(struct gtx_frame));
-    tframe->keymap   = newkeymap;
+     * Now that all the pieces exist, fill them in and stick them in
+     * the right places.
+     */
+    tframe->keymap = newkeymap;
     tframe->keystate = newkeystate;
     keymap_InitState(tframe->keystate, tframe->keymap);
-    keymap_BindToString(tframe->keymap,
-                       "\003",
-                       gtxframe_ExitCmd,
-                       "ExitCmd",
-                       (char *) (&gtxframe_exitValue));
+    keymap_BindToString(tframe->keymap, "\003", gtxframe_ExitCmd, "ExitCmd",
+                       (char *)(&gtxframe_exitValue));
 
     /*
-      * At this point, we return successfully.
-      */
-    return(tframe);
+     * At this point, we return successfully.
+     */
+    return (tframe);
 }
 
-gtxframe_Delete(aframe)
-register struct gtx_frame *aframe; {
+int
+gtxframe_Delete(struct gtx_frame *aframe)
+{
     keymap_Delete(aframe->keymap);
     free(aframe->keystate);
     if (aframe->messageLine)
-      free(aframe->messageLine);
+       free(aframe->messageLine);
     free(aframe);
     return 0;
 }
 
-gtxframe_Display(aframe, awm)
-struct gwin *awm;
-register struct gtx_frame *aframe; {
-    register struct gtxframe_dlist *tlist;
-    register struct gtxframe_menu *tm;
+int
+gtxframe_Display(struct gtx_frame *aframe, struct gwin *awm)
+{
+    struct gtxframe_dlist *tlist;
+    struct gtxframe_menu *tm;
     struct gwin_strparams strparms;
 
     /* Run through the menus, displaying them on the top line */
     *menubuffer = 0;
-    for(tm = aframe->menus; tm; tm = tm->next) {
+    for (tm = aframe->menus; tm; tm = tm->next) {
        strcat(menubuffer, tm->name);
        strcat(menubuffer, ":");
        strcat(menubuffer, tm->cmdString);
        strcat(menubuffer, " ");
     }
     if (menubuffer[0] != 0) {
-      bzero(&strparms, sizeof(strparms));
-      strparms.x = 0;
-      strparms.y = 0;
-      strparms.s = menubuffer;
-      strparms.highlight = 1;
-      WOP_DRAWSTRING(awm, &strparms);
+       memset(&strparms, 0, sizeof(strparms));
+       strparms.x = 0;
+       strparms.y = 0;
+       strparms.s = menubuffer;
+       strparms.highlight = 1;
+       WOP_DRAWSTRING(awm, &strparms);
     }
 
     /* Run through the display list, displaying all objects */
-    for(tlist=aframe->display; tlist; tlist=tlist->next) {
+    for (tlist = aframe->display; tlist; tlist = tlist->next) {
        OOP_DISPLAY(((struct onode *)(tlist->data)));
     }
 
     /* Finally, show the message line */
     ShowMessageLine(awm->w_frame);
-    return(0);
+    return (0);
 }
 
 /* Add an object to a window's display list */
-gtxframe_AddToList(aframe, aobj)
-struct onode *aobj;
-register struct gtx_frame *aframe; {
-    register struct gtxframe_dlist *tlist;
-
-    for(tlist = aframe->display; tlist; tlist=tlist->next) {
-       if (tlist->data == (char *) aobj) {
-         /*
-           * Don't add the same thing twice.
-           */
-         return(-1);
+int
+gtxframe_AddToList(struct gtx_frame *aframe, struct onode *aobj)
+{
+    struct gtxframe_dlist *tlist;
+
+    for (tlist = aframe->display; tlist; tlist = tlist->next) {
+       if (tlist->data == (char *)aobj) {
+           /*
+            * Don't add the same thing twice.
+            */
+           return (-1);
        }
     }
 
     /*
-      * OK, it's not alreadyt there.  Create a new list object, fill it
-      * in, and splice it on.
-      */
-    tlist = (struct gtxframe_dlist *) malloc(sizeof(struct gtxframe_dlist));
+     * OK, it's not alreadyt there.  Create a new list object, fill it
+     * in, and splice it on.
+     */
+    tlist = malloc(sizeof(struct gtxframe_dlist));
     if (tlist == (struct gtxframe_dlist *)0)
-      return(-1);
-    tlist->data = (char *) aobj;
+       return (-1);
+    tlist->data = (char *)aobj;
     tlist->next = aframe->display;
     aframe->display = tlist;
-    return(0);
+    return (0);
 }
 
 /* Remove an object from a display list, if it is already there */
-gtxframe_RemoveFromList(aframe, aobj)
-struct onode *aobj;
-register struct gtx_frame *aframe; {
-    register struct gtxframe_dlist *tlist, **plist;
-    
+int
+gtxframe_RemoveFromList(struct gtx_frame *aframe, struct onode *aobj)
+{
+    struct gtxframe_dlist *tlist, **plist;
+
     plist = &aframe->display;
-    for(tlist = *plist; tlist; plist = &tlist->next, tlist = *plist) {
-       if (tlist->data == (char *) aobj) {
+    for (tlist = *plist; tlist; plist = &tlist->next, tlist = *plist) {
+       if (tlist->data == (char *)aobj) {
            *plist = tlist->next;
            free(tlist);
            return 0;
        }
     }
-    return(-1);        /* Item not found */
+    return (-1);               /* Item not found */
 }
 
 /* Clear out everything on the display list for the given frame*/
-gtxframe_ClearList(aframe)
-register struct gtx_frame *aframe; {
-    register struct gtxframe_dlist *tlist, *nlist;
+int
+gtxframe_ClearList(struct gtx_frame *aframe)
+{
+    struct gtxframe_dlist *tlist, *nlist;
 
     if (aframe->display != (struct gtxframe_dlist *)0) {
-      /*
-       * Throw away each display list structure (we have at least
-       * one).
-       */
-      for(tlist = aframe->display; tlist; tlist = nlist) {
-       nlist = tlist->next;
-       free(tlist);
-      }
+       /*
+        * Throw away each display list structure (we have at least
+        * one).
+        */
+       for (tlist = aframe->display; tlist; tlist = nlist) {
+           nlist = tlist->next;
+           free(tlist);
+       }
     }
 
-    aframe->display = (struct gtxframe_dlist *) 0;
+    aframe->display = (struct gtxframe_dlist *)0;
     return 0;
 }