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
[Code enhancements] Fix various explicitness, laziness, hard codes #8011
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than the -1 thing it seems ok.
Doesn't seem like a good idea to use isdigit, because of its pitfalls (UB if outside of unsigned char range). |
This is done mostly for readability reasons. And btw, it works with std::string[indexed] value which are only basic chars. So it doesn't seem like UB for me. |
Offtop. |
You may want to squash some commits |
Done |
This PR fixes numerous code confusions.
Fixes #7997