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

Add handling of input functions #60

Merged
merged 7 commits into from
Jan 13, 2017
Merged

Add handling of input functions #60

merged 7 commits into from
Jan 13, 2017

Conversation

blink1073
Copy link
Contributor

@blink1073 blink1073 commented Jan 13, 2017

Fixes #59.
Uses add_input_event_hook to get a known stdin prompt, allowing input, menu, yes_or_no, and keyboard to work.
Uses @class overrides for kbhit, pause, sleep, and usleep. kbhit is a no-op with a printed warning and the others notify the user how long of a pause and to use the kernel interrupt to abort.

@blink1073 blink1073 mentioned this pull request Jan 13, 2017
@blink1073 blink1073 changed the title Add an override for the built in input function Add an override for the built-in input function Jan 13, 2017
@blink1073 blink1073 changed the title Add an override for the built-in input function Add overrides for the built-in input functions Jan 13, 2017
@blink1073 blink1073 changed the title Add overrides for the built-in input functions Add handling of input functions Jan 13, 2017
@blink1073 blink1073 changed the title Add handling of input functions [WIP] Add handling of input functions Jan 13, 2017
@blink1073 blink1073 changed the title [WIP] Add handling of input functions Add handling of input functions Jan 13, 2017
@blink1073 blink1073 changed the title Add handling of input functions [WIP] Add handling of input functions Jan 13, 2017
@blink1073 blink1073 changed the title [WIP] Add handling of input functions Add handling of input functions Jan 13, 2017
@blink1073 blink1073 merged commit 21f6da9 into master Jan 13, 2017
@blink1073
Copy link
Contributor Author

Released as v0.20.0.

@dsblank
Copy link
Member

dsblank commented Jan 13, 2017

Excellent work, @blink1073 !

@blink1073
Copy link
Contributor Author

That was a fun one, and something I had been trying to figure out for going on 4 years now in oct2py. Thanks again, @DrXyzzy!

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

Successfully merging this pull request may close these issues.

2 participants