search

jeaye / ncurses-rs

A low-level ncurses wrapper for Rust
Other
685 stars 99 forks source link

Replace the deprecated printw with addstr. #182

Closed maxking closed 5 years ago

gyscos commented 5 years ago

Note that there is already mvaddstr, which may be what you want. It doesn't make much sense to have 2 identical functions with different names.

If we deprecate printw, then it would make sense instead to also deprecate mvprintw.

Though in the future, we might have a better solution: there is an accepted RFC for variadic FFI compatibility. It's not complete yet, but when it reaches stable we might be able to better expose the underlying ncurses API.

jeaye commented 5 years ago

ncurses is a terribly unsafe library and ncurses-rs is the lightest weight wrapper for it possible. It's meant to expose the ncurses fns as-is. All of the issues and discussions regarding removing fns, deprecating them, etc. are all missing the point of ncurses-rs. If you want a safe and idiomatic Rust library for TUIs, look elsewhere. If you want a 1:1 C:Rust port, ncurses-rs will do the trick.