v1.0.0 couldn't work in vs2013

redis / hiredis

Minimalistic C client for Redis >= 1.2

BSD 3-Clause "New" or "Revised" License

6.21k stars 1.81k forks source link

v1.0.0 couldn't work in vs2013 #878

Open zhangxued923 opened 3 years ago

zhangxued923 commented 3 years ago

"%zu" is not supported int vs2013. In the function redisvFormatCommand, "pos" will be bad.

michael-grunder commented 3 years ago

Thanks for the report. I will try to take a look tomorrow.

I don't have vs2013 but I should be able to come up with a portable solution.

Edit: From what I'm reading VS2013 does have "%zu". Do you mean VS2010?

zhangxued923 commented 3 years ago

Thanks for the report. I will try to take a look tomorrow.

I don't have vs2013 but I should be able to come up with a portable solution.

Edit: From what I'm reading VS2013 does have "%zu". Do you mean VS2010?

yes, it's vs2013.

I got the message from here: https://docs.microsoft.com/en-us/previous-versions/tcxf1dw6(v=vs.140)?redirectedfrom=MSDN

and It does not look right while i was debugging.