From 2f9ba2f0f9690db2bd48bb0b5d7f6393a8aabd03 Mon Sep 17 00:00:00 2001 From: Peter Stephenson Date: Wed, 8 May 2002 14:25:57 +0000 Subject: 17095: Bug in setting associative array; remove unused counter --- Src/Modules/zselect.c | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) (limited to 'Src/Modules') diff --git a/Src/Modules/zselect.c b/Src/Modules/zselect.c index 827a97797..2eee59d03 100644 --- a/Src/Modules/zselect.c +++ b/Src/Modules/zselect.c @@ -37,8 +37,7 @@ * Return 1 for error (after printing a message), 0 for OK. */ static int -handle_digits(char *nam, char *argptr, fd_set *fdset, int *fdcount, - int *fdmax) +handle_digits(char *nam, char *argptr, fd_set *fdset, int *fdmax) { int fd; char *endptr; @@ -54,7 +53,6 @@ handle_digits(char *nam, char *argptr, fd_set *fdset, int *fdcount, } FD_SET(fd, fdset); - (*fdcount)++; if (fd+1 > *fdmax) *fdmax = fd+1; return 0; @@ -67,7 +65,7 @@ static int bin_zselect(char *nam, char **args, char *ops, int func) { #ifdef HAVE_SELECT - int i, fd, fdsetind = 0, fdcount = 0, fdmax = 0; + int i, fd, fdsetind = 0, fdmax = 0, fdcount; fd_set fdset[3]; const char fdchar[3] = "rwe"; struct timeval tv, *tvptr = NULL; @@ -166,12 +164,11 @@ bin_zselect(char *nam, char **args, char *ops, int func) /* Digits following option without arguments are fd's. */ default: if (handle_digits(nam, argptr, fdset+fdsetind, - &fdcount, &fdmax)) + &fdmax)) return 1; } } - } else if (handle_digits(nam, argptr, fdset+fdsetind, &fdcount, - &fdmax)) + } else if (handle_digits(nam, argptr, fdset+fdsetind, &fdmax)) return 1; } @@ -220,7 +217,7 @@ bin_zselect(char *nam, char **args, char *ops, int func) strcpy(buf, data); for (ptr = buf; *ptr; ptr++) ; - *ptr++ = fdchar[1]; + *ptr++ = fdchar[i]; *ptr = '\0'; zsfree(data); *dataptr = ztrdup(buf); -- cgit 1.4.1