This is the mail archive of the cygwin mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: TIOCPKT mode of PTY is broken if ONLCR bit is cleared.


Hi Corinna,

On Mon, 23 Mar 2015 11:08:23 +0100
Corinna Vinschen <corinna-cygwin@cygwin.com> wrote:

> With this out of the way, I'm just looking through your patch and it
> looks good to me.  I have only a few minor style nits:
> 
> +                     rc ++;
> 
> No space between the variable and the increment operator, please.
> 
> +                 if ((get_ttyp ()->ti.c_oflag & ONOCR) && get_ttyp ()->column == 0)
> 
> Please try to keep lines <= 80 chars:
> 
> +                 if ((get_ttyp ()->ti.c_oflag & ONOCR)
> +                     && get_ttyp ()->column == 0)

Thank you for checking my patch. 

> However, due to the changes in fhandler_tty.cc, this patch doesn't
> apply cleanly anymore.  Any chance you could resend it (ideally to the
> cygwin-patches ML) diff'ed against git HEAD?

I will post a revised patch to cygwin-patches ML soon.

Thank you in advance.

-- 
Takashi Yano <takashi.yano@nifty.ne.jp>

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]