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

remove documentation about speciality of setupterm's descriptor, use os.devnull #95

Open
jquast opened this issue Mar 3, 2015 · 2 comments

Comments

@jquast
Copy link
Collaborator

jquast commented Mar 3, 2015

It's actually perfectly fine to call curses.setupterm(kind, open(os.devnull).fileno()).

More importantly, this removes a chunk of documentation and comments, such as:

            Terminal initialization sequences will be sent to ``stream`` if it
            has a file descriptor and to ``sys.__stdout__`` otherwise. (``setupterm()`` demands to send them
            somewhere, and stdout is probably where the output is ultimately
            headed. If not, stderr is probably bound to the same terminal.)

and

            # (...) Explicit args make setupterm()
            # work even when -s is passed to nosetests. Lean toward sending
            # init sequences to the stream if it has a file descriptor, and
            # send them to stdout as a fallback, since they have to go
            # somewhere.
@erikrose
Copy link
Owner

I'd like to give this more consideration if I ever return my attention to blessings.

@erikrose erikrose reopened this Feb 20, 2019
@jquast jquast closed this as completed Jan 12, 2020
@jquast jquast reopened this Feb 1, 2020
@jquast
Copy link
Collaborator Author

jquast commented Feb 1, 2020

oops, sorry to close, didn't see your message last time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants