Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tiny panics without visible message when nicks is an empty list #184

Closed
e00E opened this issue Apr 12, 2020 · 2 comments
Closed

Tiny panics without visible message when nicks is an empty list #184

e00E opened this issue Apr 12, 2020 · 2 comments
Labels
Milestone

Comments

@e00E
Copy link
Contributor

e00E commented Apr 12, 2020

Default configuration but nicks is [] for freenode.

Then run tiny. A message flashes on the terminal but disappears immediately before I can read it. By redirecting stderr to a file I can recover this:

thread 'main' panicked at 'called Result::unwrap() on an Err value: Os { code: 25, kind: Other, message: "Inappropriate ioctl for device" }', libtiny_tui/src/tui.rs:69:18

@trevarj
Copy link
Contributor

trevarj commented Apr 12, 2020

If you need a way to get around this, the following works:

      nicks:
        -

@osa1
Copy link
Owner

osa1 commented Apr 12, 2020

Thanks for reporting. So first of all we should improve error reporting. Printing to stderr is not helpful in a TUI app. In the worst case I think we could redirect stderr to a file, but I think ideally all errors would be shown in tiny. Not sure if that's easily possible.

Secondly I think we should improve checking user errors in config files.

@osa1 osa1 added the bug label Apr 13, 2020
@osa1 osa1 added this to the 0.5.2 milestone May 11, 2020
@osa1 osa1 closed this as completed in 733e665 May 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants