Skip to content
This repository has been archived by the owner on Oct 28, 2019. It is now read-only.

Don't treat KeyboardInterrupt as errors #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Don't treat KeyboardInterrupt as errors #9

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Aug 21, 2015

This makes it possibly to exit Idle Master by using Ctrl+C on the terminal it is run from. Previously, you would need to hit Ctrl+C about five times to actually get Idle Master to exit.

See http://effbot.org/zone/stupid-exceptions-keyboardinterrupt.htm for reference.

This makes it possibly to exit Idle Master by using Ctrl+C on the terminal it is run from. Previously, you would need to hit Ctrl+C about five times to actually get Idle Master to exit.
@ghost
Copy link

ghost commented Sep 13, 2015

This doesn't remove the fake window.

^CTraceback (most recent call last):
  File "start.py", line 261, in <module>
    time.sleep(delay)
KeyboardInterrupt

After this the window still exists.
If i then focus on the window the window closes with this message:

$ Traceback (most recent call last):
  File "steam-idle.py", line 74, in <module>
    gui.mainloop()
  File "/usr/lib/python2.7/lib-tk/Tkinter.py", line 1128, in mainloop
    self.tk.mainloop(n)

@ghost
Copy link
Author

ghost commented Sep 13, 2015

Interesting. I'm not using the GUI window anymore (see PR #7), so I didn't notice this.

@ghost
Copy link

ghost commented Sep 13, 2015

Thank you, I will try that out.

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

Successfully merging this pull request may close these issues.

0 participants