tty.cc and fhandler/tty.cc

Corinna Vinschen corinna-cygwin@cygwin.com
Tue Nov 22 09:36:28 GMT 2022


On Nov 22 17:31, Takashi Yano wrote:
> Current cygwin git master has two tty.cc, one is winsup/cygwin/tty.cc
> and the other is winsup/cygwin/fhandler/tty.cc (it was originally
> fhandler_tty.cc). This is somewhat confusing.
> 
> fhandler/tty.cc is defining fhandler_pty_{common,master,slave} class,
> so, what about renaming fhandler/tty.cc to fhandler/pty.cc?

Maybe we could split it up.  The extern functions could go into
syscalls.cc and the tty classes could go into fhandler/termios.cc,
perhaps.  There are already two tty_min methods in that file.

What do you think?


Corinna


More information about the Cygwin-developers mailing list