You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@kvrocks.apache.org by GitBox <gi...@apache.org> on 2022/09/19 15:01:23 UTC

[GitHub] [incubator-kvrocks] tanruixiang opened a new issue, #896: Replace std::stol with ParseInt.

tanruixiang opened a new issue, #896:
URL: https://github.com/apache/incubator-kvrocks/issues/896

   ### Search before asking
   
   - [X] I had searched in the [issues](https://github.com/apache/incubator-kvrocks/issues) and found no similar issues.
   
   
   ### Motivation
   
   There are many places in the code that use std::stol. Replace std::stol with ParseInt. Use ParseInt instead of std::stol for safer parsing.
   
   ### Solution
   
   _No response_
   
   ### 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: issues-unsubscribe@kvrocks.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-kvrocks] PragmaTwice closed issue #896: Replace std::stol with ParseInt.

Posted by GitBox <gi...@apache.org>.

PragmaTwice closed issue #896: Replace std::stol with ParseInt.
URL: https://github.com/apache/incubator-kvrocks/issues/896


-- 
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: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org