Replace std::stol with ParseInt

[tanruixiang]

It closes #896

[tanruixiang]

@PragmaTwice Thanks for your warm help. I'm just going to upload a draft to save my progress and keep doing it tomorrow. I didn't even check it myself. You have helped me. Thank you very much😆

[tanruixiang]

@PragmaTwice Hi. The original code uses std:: stol , but using ParseInt<long int> instead the lint will report an error. Do you have any better suggestions?

[tanruixiang]

That is to say, the type used in the original code shoule be the same here. In this pr, regardless of whether the type can be adjusted logically. If the type can be modified, we can do it in the next Pr?

@PragmaTwice HI. What do you think about this. If you want to modify it in this pr, I will modify it immediately, otherwise I will modify it to another pr.

[PragmaTwice]

That is to say, the type used in the original code shoule be the same here. In this pr, regardless of whether the type can be adjusted logically. If the type can be modified, we can do it in the next Pr?

@PragmaTwice HI. What do you think about this. If you want to modify it in this pr, I will modify it immediately, otherwise I will modify it to another pr.

I think it is ok to change it, since it is trivial.

[tanruixiang]

That is to say, the type used in the original code shoule be the same here. In this pr, regardless of whether the type can be adjusted logically. If the type can be modified, we can do it in the next Pr?

@PragmaTwice HI. What do you think about this. If you want to modify it in this pr, I will modify it immediately, otherwise I will modify it to another pr.

I think it is ok to change it, since it is trivial.

Ok, I will change it .

[PragmaTwice]

LGTM, thanks for your contribution!

[PragmaTwice]

Hi @tanruixiang, I notice there is also some code using std::stoul or std::stoull in kvrocks, do you want to also replace them in this PR? Or it is OK to merge it and replace them in another PR.

[tanruixiang]

Hi @tanruixiang, I notice there is also some code using std::stoul or std::stoull in kvrocks, do you want to also replace them in this PR? Or it is OK to merge it and replace them in another PR.

OK I am willing to do this task. I think this PR information is a bit too much, we can continue to do this in the next PR.

[PragmaTwice]

Hi @tanruixiang, I notice there is also some code using std::stoul or std::stoull in kvrocks, do you want to also replace them in this PR? Or it is OK to merge it and replace them in another PR.

OK I am willing to do this task. I think this PR information is a bit too much, we can continue to do this in the next PR.

Thanks. Maybe there are also some atoi and std::stoi, you can replace them too in another PR.

[PragmaTwice]

Thanks. I will merge it soon.