Ranxy opened a new issue, #766: URL: https://github.com/apache/incubator-kvrocks/issues/766
### Search before asking - [X] I had searched in the [issues](https://github.com/apache/incubator-kvrocks/issues) and found no similar issues. ### Motivation There is some code in the project as follows ```cpp try { slot_ = atoi(args[2].c_str()); dst_node_id_ = args[3]; } catch (std::exception &e) { return Status(Status::RedisParseErr, errValueNotInterger); } ``` However `atoi` does not throw exceptions, so perhaps we should use `std::stoi` to replace these. ### Solution Replace `atoi` to `std::stoi` ### Are you willing to submit a PR? - [X] I'm willing to submit a PR! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
