fhandler_base::ioctl (FIONBIO)
Corinna Vinschen
cygwin-patches@cygwin.com
Sat Oct 25 12:33:00 GMT 2003
On Fri, Oct 24, 2003 at 09:50:38AM -0500, Brian Ford wrote:
> On Fri, 24 Oct 2003, Corinna Vinschen wrote:
>
> > On Thu, Oct 23, 2003 at 06:06:09PM -0500, Brian Ford wrote:
> > > Any reason not to support this? It seams to me that this patch just
> > > parallels what is already in fhandler_base::fcntl (F_SETFL) for
> > > O_NONBLOCK.
> >
> > Yes, I think you're right. However, I'd like to ask you to rearrange
> > your patch a bit. Most (all?) other ioctl methods are using a switch
> > statement rather than a if/else clause. To allow later easier extension,
> > I think using a switch here would be better as well, even though there's
> > only one case so far.
> >
> Ok. Revised as suggested and attached. Same ChangeLog entry.
> [...]
> > > * fhandler.cc (fhandler_base::ioctl): Handle FIONBIO.
Thanks. I've checked it in.
Corinna
--
Corinna Vinschen Please, send mails regarding Cygwin to
Cygwin Developer mailto:cygwin@cygwin.com
Red Hat, Inc.
More information about the Cygwin-patches
mailing list