Skip to content

/ bpython

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

Jump to bottom

Curses error on resizing terminal #65

Closed
ikanobori opened this issue Jul 4, 2014 · 10 comments
Closed

Curses error on resizing terminal #65

ikanobori opened this issue Jul 4, 2014 · 10 comments
Labels
bug

Comments

@ikanobori
Copy link
Contributor

@ikanobori ikanobori commented Jul 4, 2014

When I try to resize bpython terminal window, after a while of resizing (not instantly) it crashes with following stacktrace

#!

Traceback (most recent call last):
  File "/usr/bin/bpython", line 8, in <module>
    load_entry_point('bpython==0.9.5.2', 'console_scripts', 'bpython')()
  File "/usr/lib/python2.6/site-packages/bpython/cli.py", line 2309, in main
    locals_)
  File "/usr/lib/python2.6/site-packages/bpython/cli.py", line 2175, in curses_wrapper
    return func(stdscr, *args, **kwargs)
  File "/usr/lib/python2.6/site-packages/bpython/cli.py", line 2238, in main_curses
    repl.repl()
  File "/usr/lib/python2.6/site-packages/bpython/cli.py", line 1219, in repl
    inp = self.get_line()
  File "/usr/lib/python2.6/site-packages/bpython/cli.py", line 1862, in get_line
    key = self.get_key()
  File "/usr/lib/python2.6/site-packages/bpython/cli.py", line 1903, in get_key
    self.idle(self)
  File "/usr/lib/python2.6/site-packages/bpython/cli.py", line 2121, in idle
    do_resize(caller)
  File "/usr/lib/python2.6/site-packages/bpython/cli.py", line 2139, in do_resize
    caller.resize()
  File "/usr/lib/python2.6/site-packages/bpython/cli.py", line 1263, in resize
    self.statusbar.resize(refresh=False)
  File "/usr/lib/python2.6/site-packages/bpython/cli.py", line 1954, in resize
    self.win.mvwin(self.y, self.x)
_curses.error: mvwin() returned ERR
@ikanobori
Copy link
Contributor Author

@ikanobori ikanobori commented Jul 4, 2014

This should probably be marked resolved.

Original Comment By:

@ikanobori
Copy link
Contributor Author

@ikanobori ikanobori commented Jul 4, 2014

This is fixed by pull request #16.

Original Comment By: Steven Myint
@ikanobori
Copy link
Contributor Author

@ikanobori ikanobori commented Jul 4, 2014

Issue #156 was marked as a duplicate of this issue.

Original Comment By: Andreas Stührk
@ikanobori
Copy link
Contributor Author

@ikanobori ikanobori commented Jul 4, 2014

That's bad because it shadows the real and more interesting traceback.

Original Comment By: Andreas Stührk
@ikanobori
Copy link
Contributor Author

@ikanobori ikanobori commented Jul 4, 2014

I get a different stacktrace (with your patch applied, Andreas):

{{{
#!python

Traceback (most recent call last):
File "cli.py", line 1623, in main()
File "cli.py", line 1610, in main
banner=banner)
File "cli.py", line 1515, in curses_wrapper
curses.nocbreak()
_curses.error: nocbreak() returned ERR
}}}

Original Comment By: Simon de Vlieger
@ikanobori
Copy link
Contributor Author

@ikanobori ikanobori commented Jul 4, 2014

I'll test Andreas' patch tonight.

Original Comment By: Simon de Vlieger
@ikanobori
Copy link
Contributor Author

@ikanobori ikanobori commented Jul 4, 2014

Same happens here. But it also crashes byobu and mutt and I can make it crash a few other curses apps as well, they just don't show a traceback.

This might have to do with the terminal being used and how often it sends its new sizes?

Original Comment By: Simon de Vlieger
@ikanobori
Copy link
Contributor Author

@ikanobori ikanobori commented Jul 4, 2014

I'm unable to reproduce the issue, some more information would be helpful.

Original Comment By: Andreas Stührk
@ikanobori ikanobori mentioned this issue Jul 4, 2014
Closed
@thomasballinger
Copy link
Member

@thomasballinger thomasballinger commented Aug 10, 2014

Still an issue for anyone?
@thomasballinger
Copy link
Member

@thomasballinger thomasballinger commented Aug 21, 2014

closing because we can't reproduce
@sebastinas sebastinas mentioned this issue Jul 22, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@ikanobori @thomasballinger
You can’t perform that action at this time.