-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Replaced old atoi() with better strtol() #1422
base: master
Are you sure you want to change the base?
Conversation
I don't see any actual benefin otherwise. |
There's no big benefit. It's just a little cleanup to remove an obsolete function. |
atoi is not obsolete, it is just not recommended any longer :) strtol is better /if/ you add error checking. |
Draft error checking now added. |
d7bbd5a
to
e0ecc6a
Compare
atoi() gives no opportunity for error checking, strtol() does. Improved error checking.
atoi() gives no opportunity for error checking, strtol() does. Though I did not actually improve error checking...