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

-s doesn't accept an int #8

Open
bacalec opened this issue Feb 21, 2023 · 0 comments
Open

-s doesn't accept an int #8

bacalec opened this issue Feb 21, 2023 · 0 comments

Comments

@bacalec
Copy link

bacalec commented Feb 21, 2023

Hello,
Hhen parsing option -s: its parameter is parsed as string line 218
later it's assigned without a cast line 53
then it's compared to string now and (in that case it's overwritten with an int) or it's verified that it can be cast to a positive integer line 72
So far so good.

Problem is that it's not cast to an int when trying to add it to window_second line 103 and when comparing it to end time line 108
A fix consist in casting it to an int in these lines

Another option maybe more consistent with what is done when it's now would consist in overwriting the string by an int asap.

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

No branches or pull requests

1 participant