search

magiblot / tvision

A modern port of Turbo Vision 2.0, the classical framework for text-based user interfaces. Now cross-platform and with Unicode support.
Other
2k stars 151 forks source link

tvhc: use stderr #41

Closed electroly closed 3 years ago

electroly commented 3 years ago

This is a pretty minor issue, but it would be nice if tvhc used cerr to print warnings/errors and cout for the copyright notice. Currently it uses cout for everything, so there's no easy way to silence the copyright notice without also silencing errors.

magiblot commented 3 years ago

Okay. This also matches the behaviour or gcc and clang: standard output for things like --help and standard error for compiler diagnostics.

Can you check whether https://github.com/magiblot/tvision/commit/40e616d968c872745f22c217da87a779dc8738f2 works well enough for you? The only messages still printed with cout are the copyright notice and the usage information. Note that the "Overwrite?" messages were already using cerr and I haven't changed that (although I can do so if you think it's a good idea).

Cheers.

electroly commented 3 years ago

That's perfect. Thank you!