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

#267 Issue: Update whats.py #268

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

Conversation

Harsh0707005
Copy link

Whats.py may crash as time.sleep() function doesn't accept negative integers

Closes issue number #267

The issue has a syntax error by the user but upon looking over it, I found an error in whats.py. In line 38, time.sleep() function takes wait_time-4 as an argument but this will raise an error and will crash the script when the user feeds a number less than 4 as a value for wait_time, as time.sleep() cannot take negative integer as argument

The input 'wait_time - 4' for time.sleep() function on line 38 was causing a problem as for wait_time <4 it will be a negative value which is not accepted by time.sleep() function.
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.

1 participant