On Thu, May 06, 2021 at 07:55:34PM +0200, Tobias Nygren wrote:
> On Thu, 6 May 2021 13:44:47 +0000
> nia <n...@netbsd.org> wrote:
> 
> > Review and comments welcome, of course.
> 
> Hi,
> 
> This looks useful, I support importing it into src.
> Found some minor bugs though:
> 
> If I exit by pressing ^C instead of 'q' the terminal cursor is not
> restored but remains hidden (at least with xterm).

Thanks. I suppose curses is not setting up a handler for
SIGTERM by default?

> 
> Scrolling down and then scrolling up in the output list results in the
> selected output's title being hidden "above the scroll".

Scrolling needs more testing, none of my hardware devices
create enough controls for it. I will fire up QEMU with an eap
card attached.

> 
> usage() needs __dead annotation to compile clean w/ clang:
> 
> main.c:56:1: error: function 'usage' could be declared with attribute 
> 'noreturn'

Thx.
> 
> Kind regards,
> -Tobias

Reply via email to