serial patch - second attempt

Jacek Trzcinski jacek@certum.pl
Wed Jul 24 01:00:00 GMT 2002


Hi !
I think that there is a problem with this name. In case of serial
initialization (in method fhandler_serial::open()) it could be utilized
because its goal is connected with name - it initializes RTS and DTR
serial lines. Problem is in method fhandler_serial::ioctl().
Conditional paths utilized by me are concerned with the fact that under
NT , native serial device driver supports reading of RTS and DTR lines
by DeviceIoControl() function. Under 9x native serial driver does not
support it so utilized variables dsr and rts, setting in different
places of fhandler_serial.cc. Thus in my opinion, saying
"must_init_serial_line" in ioctl() method is not accurate because there
is none initialization there.

Jacek

Corinna Vinschen wrote:
> 
> On Mon, Jul 15, 2002 at 09:52:45AM +0200, Jacek Trzcinski wrote:
> > 13 July 2002 Jacek Trzcinski <jacek@certum.pl>
> >
> >     * fhandler.h (class fhandler_serial): Add new members of
> >     the class - rts,dtr and method ioctl(). Variables rts and dtr
> >     important for Win 9x only.
> >     * fhandler_serial.cc (fhandler_serial::open): Add initial setting
> >     of dtr and rts. Important for Win 9x only.
> >     (fhandler_serial::ioctl): New function. Implements commands TIOCMGET,
> >     TIOCMSET and TIOCINQ.
> >     (fhandler_serial::tcflush): Fixed found error.
> >     (fhandler_serial::tcsetattr): Add settings of rts and dtr. Important
> >     for Win 9x only.
> >     * termios.h: Add new defines as a support for ioctl() function
> >     on serial device.
> 
> It's applied.  Today I've changed the
> 
>         if (!wincap.is_winnt ())
> 
> to using a new capability
> 
>         if (wincap.must_init_serial_line ())
> 
> which is set to true for each 9x/Me, to false for NT/2K/XP.
> 
> If you think that the name of the new capability is inappropriate,
> please suggest another name.
> 
> 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