tech-userlevel archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: Importing aiomixer to src

On Thu, May 06, 2021 at 07:55:34PM +0200, Tobias Nygren wrote:
> On Thu, 6 May 2021 13:44:47 +0000
> nia <> 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'

> Kind regards,
> -Tobias

Home | Main Index | Thread Index | Old Index