rust-terminfo
rust-terminfo copied to clipboard
meh
Terminal information for Rust.
First of all, thanks for writing this crate, it has a nice clean API. My reading of the WTFPL is that it is meant to allow any sort of usage...
A simpler (in my opinion) design here is to have the database just have its name and description as public fields; the builder pattern seems a bit excessive.
There were a couple problems with the way the library previously did error handling: * Error chains didn’t inform much, as they didn’t give any context to the error *...
This function is kind of a mess, it’s absolutely huge and uses macros all over the place. This PR contains a couple refactors of it.
Currently [`Expansion>::parameters`](https://docs.rs/terminfo/latest/terminfo/capability/struct.Expansion.html#method.parameters-409) takes 9 consecutive boolean parameters, which isn’t very readable and is very prone to mistakes. Consider also using [anstyle](https://docs.rs/anstyle) since it may hit 1.0 soon.
```rust ---- parser::source::test::parsing stdout ---- thread 'parser::source::test::parsing' panicked at 'incomplete', src/parser/source.rs:137:6 note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace. ``` This has happened on Fedora. Let me know...
While working on a way to programatically generate the capability list, I ran into a duplicate termcap (legacy) code: ML. It can either be: | variable | capname | TCap...
That fixes the remaining warnings reported by clippy in the current stable and nightly Rust.
