Re: lib/52063: halfdelay() does not honour timeout value

[Onno van der Linden <o.vd.linden%quicknet.nl@localhost>]

The following reply was made to PR lib/52063; it has been noted by GNATS.

From: Onno van der Linden <o.vd.linden%quicknet.nl@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: lib/52063: halfdelay() does not honour timeout value
Date: Tue, 7 Aug 2018 21:51:21 +0200

 After Brett Lymn told me he wasn't comfortable with the patch I tried
 to come up with something better. Noticed a difference between
 wgetch() and wget_wch() in handling win->delay. Made that handling
 in wgetch() equal to wget_wch():
 
 $ diff -u `pwd`/getch.c.orig `pwd`/getch.c
 --- /usr/src/lib/libcurses/getch.c.orig 2017-01-31 10:17:53.000000000 +0100
 +++ /usr/src/lib/libcurses/getch.c      2018-08-07 21:33:41.361102323 +0200
 @@ -867,8 +867,6 @@
                 switch (win->delay)
                 {
                 case -1:
 -                       if (__delay() == ERR)
 -                               return ERR;
                         break;
                 case 0:
                         if (__nodelay() == ERR)
 
 
 And now the testscript works fine.
 
 Onno