acl-dev / acl

C/C++ server and network library, including coroutine,redis client,http/https/websocket,mqtt, mysql/postgresql/sqlite client with C/C++ for Linux, Android, iOS, MacOS, Windows, etc..
https://acl-dev.cn
GNU Lesser General Public License v3.0
2.83k stars 937 forks source link

incrbyfloat 函数错误使用sizeof判断char* #294

Closed openkoding closed 1 year ago

openkoding commented 1 year ago

最近公司项目需要支持redis,所以找到了acl这个库,在写测试代码时发现incrbyfloat不能按预期返回数据。 我在最近修改记录中发现此函数中的bufchar []改为了char*, 但是后面调用get_string时传递字符串大小的写法并未改动,仍然是sizeof,这会导致获取到的和预期不一致,希望您能抽空修复下这个问题。

zhengshuxin commented 1 year ago

谢谢提醒,此处为手误,该问题已经修复,长度应该用 FLOAT_LEN 而不应该用 sizeof(char*)。

zhengshuxin commented 1 year ago

谢谢提醒,此处为手误,该问题已经修复,长度应该用 FLOAT_LEN 而不应该用 sizeof(char*)。

openkoding commented 1 year ago

hincrbyfloat也有同样的问题

zhengshuxin commented 1 year ago

是的,问题已经修复,谢谢。