Bug#676461: libncursesw5: buggy test causes select() to be used instead of poll(), making ncurses really slow

2012-06-09 Thread Sven Joachim
On 2012-06-09 01:51 +0200, Thomas Dickey wrote: On Fri, Jun 08, 2012 at 09:58:00PM +0200, Sven Joachim wrote: Do you also redirect stdin? Otherwise it's hard to get a non-zero no - just stdout/stderr On the other hand, rpm and dpkg build tools can redirect things as well. Indeed, and

Bug#676461: libncursesw5: buggy test causes select() to be used instead of poll(), making ncurses really slow

2012-06-09 Thread Sven Joachim
On 2012-06-09 09:47 +0200, Sven Joachim wrote: On 2012-06-09 01:51 +0200, Thomas Dickey wrote: The exit code is inverted: it is true if nonzero, which could be -1 or some number greater than zero (the latter would indicate some input happening, though which of the cases is hard to tell).

Bug#676461: libncursesw5: buggy test causes select() to be used instead of poll(), making ncurses really slow

2012-06-09 Thread Thomas Dickey
On Sat, Jun 09, 2012 at 10:36:00AM +0200, Sven Joachim wrote: On 2012-06-09 09:47 +0200, Sven Joachim wrote: On 2012-06-09 01:51 +0200, Thomas Dickey wrote: The exit code is inverted: it is true if nonzero, which could be -1 or some number greater than zero (the latter would indicate

Bug#676461: libncursesw5: buggy test causes select() to be used instead of poll(), making ncurses really slow

2012-06-09 Thread Sven Joachim
On 2012-06-09 11:29 +0200, Thomas Dickey wrote: On Sat, Jun 09, 2012 at 10:36:00AM +0200, Sven Joachim wrote: Also, shouldn't the test check whether opening /dev/tty was actually successful? I assumed in that instance that poll would simply fail :-) Well, it returns 0 which you treat as

Bug#676461: libncursesw5: buggy test causes select() to be used instead of poll(), making ncurses really slow

2012-06-08 Thread shawn
resend, seems like i got blocked from the other address, even @ BTS On Thu, 2012-06-07 at 06:51 -0400, Thomas Dickey wrote: On Thu, Jun 07, 2012 at 06:42:24AM +, shawn wrote: Package: libncursesw5 Version: 5.9-8 Severity: important Tags: upstream int main() { struct

Bug#676461: libncursesw5: buggy test causes select() to be used instead of poll(), making ncurses really slow

2012-06-08 Thread Sven Joachim
On 2012-06-07 22:04 +0200, Thomas Dickey wrote: On Thu, Jun 07, 2012 at 01:43:11PM +0200, Sven Joachim wrote: On 2012-06-07 12:51 +0200, Thomas Dickey wrote: On Thu, Jun 07, 2012 at 06:42:24AM +, shawn wrote: Package: libncursesw5 Version: 5.9-8 Severity: important Tags:

Bug#676461: libncursesw5: buggy test causes select() to be used instead of poll(), making ncurses really slow

2012-06-08 Thread Thomas Dickey
On Fri, Jun 08, 2012 at 09:58:00PM +0200, Sven Joachim wrote: Do you also redirect stdin? Otherwise it's hard to get a non-zero no - just stdout/stderr On the other hand, rpm and dpkg build tools can redirect things as well. result from poll(): you have to manage pressing the return key

Bug#676461: libncursesw5: buggy test causes select() to be used instead of poll(), making ncurses really slow

2012-06-07 Thread shawn
Package: libncursesw5 Version: 5.9-8 Severity: important Tags: upstream at line 15732 in ./configure echo $as_me:15718: checking if poll really works 5 echo $ECHO_N checking if poll really works... $ECHO_C 6 if test ${cf_cv_working_poll+set} = set; then echo $ECHO_N (cached) $ECHO_C 6 else if

Bug#676461: libncursesw5: buggy test causes select() to be used instead of poll(), making ncurses really slow

2012-06-07 Thread Thomas Dickey
On Thu, Jun 07, 2012 at 06:42:24AM +, shawn wrote: Package: libncursesw5 Version: 5.9-8 Severity: important Tags: upstream int main() { struct pollfd myfds; int ret; myfds.fd = 0; myfds.events = POLLIN; ret = poll(myfds, 1, 100);

Bug#676461: libncursesw5: buggy test causes select() to be used instead of poll(), making ncurses really slow

2012-06-07 Thread Sven Joachim
On 2012-06-07 12:51 +0200, Thomas Dickey wrote: On Thu, Jun 07, 2012 at 06:42:24AM +, shawn wrote: Package: libncursesw5 Version: 5.9-8 Severity: important Tags: upstream int main() { struct pollfd myfds; int ret; myfds.fd = 0; myfds.events =

Bug#676461: libncursesw5: buggy test causes select() to be used instead of poll(), making ncurses really slow

2012-06-07 Thread Thomas Dickey
On Thu, Jun 07, 2012 at 01:43:11PM +0200, Sven Joachim wrote: On 2012-06-07 12:51 +0200, Thomas Dickey wrote: On Thu, Jun 07, 2012 at 06:42:24AM +, shawn wrote: Package: libncursesw5 Version: 5.9-8 Severity: important Tags: upstream int main() { struct pollfd myfds;