getchar-returns-int-20021026
authorSam Hartman <hartmans@mit.edu>
Sat, 26 Oct 2002 16:39:08 +0000 (16:39 +0000)
committerDerrick Brashear <shadow@dementia.org>
Sat, 26 Oct 2002 16:39:08 +0000 (16:39 +0000)
don't assume it returns char

src/butc/read_tape.c
src/butc/recoverDb.c
src/kauth/klogin.c
src/login/login.c
src/lwp/waitkey.c
src/volser/vos.c
src/volser/vsprocs.c

index 47d8c5b..1624676 100644 (file)
@@ -163,7 +163,7 @@ int openOutFile(headerPtr)
   struct volumeHeader *headerPtr;
 {
   afs_int32 len;
-  char ch;
+  int ch;
   int rc;
   int oflag;
   int  skip, first;
index 4ddd2fa..d40870a 100644 (file)
@@ -113,7 +113,7 @@ afs_int32
 Ask(st)
      char *st;
 {
-    char response;
+    int  response;
 
     while (1)
     {
@@ -123,7 +123,7 @@ Ask(st)
        fflush(stdout);
        response = getchar();
        if      ( response == 'y' ) return(1);
-       else if ( response == 'n' ) return(0);
+       else if ( response == 'n' || response == EOF) return(0);
        printf("please answer y/n\n");
     }
 }
index 68bdfc6..bb97439 100644 (file)
@@ -461,7 +461,7 @@ int main(
 static void getloginname(register struct utmp *up)
 {
        register char *namep;
-       char c;
+       int c;
 
        while (up->ut_name[0] == '\0') {
                namep = up->ut_name;
@@ -469,10 +469,10 @@ static void getloginname(register struct utmp *up)
                while ((c = getchar()) != '\n') {
                        if (c == ' ')
                                c = '_';
-                       if (c == (char)EOF)
+                       if (c == EOF)
                                exit(0);
                        if (namep < up->ut_name+NMAX)
-                               *namep++ = c;
+                               *namep++ = (char) c;
                }
        }
        strncpy(lusername, up->ut_name, NMAX);
index 01bb231..d58a1bd 100644 (file)
@@ -1439,7 +1439,7 @@ int prompt;
                                exit(0);
                        }
                        if (p < nbuf + UT_NAMESIZE)
-                               *p++ = ch;
+                               *p++ = (char) ch;
                }
                if (p > nbuf) {
                        if (nbuf[0] == '-')
index 2586f9c..fddb348 100644 (file)
@@ -260,7 +260,7 @@ int LWP_GetResponseKey(int seconds, char *key)
 #ifdef AFS_NT40_ENV
   *key = getche(); /* get char and echo it to screen */
 #else
-  *key = getchar();
+  *key = (char ) getchar();
 #endif
 
   return rc;
index 325fab8..00b62ab 100644 (file)
@@ -2108,7 +2108,7 @@ register struct cmd_syndesc *as;
        else {                    /* volume exists - do we do a full incremental or abort */
            int Oserver, Opart, Otype, vol_elsewhere = 0;
            struct nvldbentry Oentry;
-           char   c, dc;
+           int   c, dc;
 
            if(avolid == 0) {
                avolid = entry.volumeId[voltype];
index c768f93..918b535 100644 (file)
@@ -179,7 +179,7 @@ void MapPartIdIntoName(afs_int32 partId, char *partName)
 
 int yesprompt(char *str)
 {
-    char response, c;
+    int response, c;
     int code;
 
     fprintf(STDERR, "Do you want to %s? [yn](n): ", str);