diff options
author | Christopher Faylor <me@cgf.cx> | 2013-12-01 06:52:54 +0400 |
---|---|---|
committer | Christopher Faylor <me@cgf.cx> | 2013-12-01 06:52:54 +0400 |
commit | 499494d2cca1f8cf69f54d03437692582bf8415c (patch) | |
tree | f94e1f345b9e7cecb6dc27d0ca68956bcb87e380 /winsup/cygwin/dtable.cc | |
parent | 483c843a6a67780f4072726c40c51f11b08cb55f (diff) |
* dtable.h (dtable::first_fd_for_open): Change declaration to size_t.
(dtable::extend): Change parameter to size_t.
(dtable::find_unused_handle): Ditto.
* dtable.cc: Remove now-unused header.
(dtable::extend): Remove pointless test. Change parameter to size_t.
(dtable::find_unused_handle): Rework to avoid MAX calculation in extend() call.
Change parameter to size_t.
Diffstat (limited to 'winsup/cygwin/dtable.cc')
-rw-r--r-- | winsup/cygwin/dtable.cc | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/winsup/cygwin/dtable.cc b/winsup/cygwin/dtable.cc index 7cf52c4bc..bbec732a5 100644 --- a/winsup/cygwin/dtable.cc +++ b/winsup/cygwin/dtable.cc @@ -16,7 +16,6 @@ details. */ #include <stdio.h> #include <unistd.h> #include <wchar.h> -#include <sys/param.h> #define USE_SYS_TYPES_FD_SET #include <winsock.h> @@ -73,14 +72,11 @@ set_std_handle (int fd) } int -dtable::extend (int howmuch) +dtable::extend (size_t howmuch) { int new_size = size + howmuch; fhandler_base **newfds; - if (howmuch <= 0) - return 0; - if (new_size > OPEN_MAX_MAX) { set_errno (EMFILE); @@ -225,8 +221,10 @@ dtable::delete_archetype (fhandler_base *fh) } int -dtable::find_unused_handle (int start) +dtable::find_unused_handle (size_t start) { + size_t extendby = (start > size) ? start - size : NOFILE_INCR; + /* This do loop should only ever execute twice. */ do { for (size_t i = start; i < size; i++) @@ -234,7 +232,7 @@ dtable::find_unused_handle (int start) if (fds[i] == NULL) return i; } - while (extend (MAX (NOFILE_INCR, start - size))); + while (extend (extendby)); return -1; } |