书中在sds模块API http://www.redisbook.com/en/latest/internal-datastruct/sds.html
写到
“sdsupdatelen 更新给定 sds 所对应 sdshdr 结构的 free 和 len 算法复杂度 O(1)”

void sdsupdatelen(sds s) {

struct sdshdr *sh = (void*) (s-(sizeof(struct sdshdr)));

// 计算正确的 buf 长度
int reallen = strlen(s);

// 更新属性
sh->free += (sh->len-reallen);
sh->len = reallen;

}

C 函数库 中的 strlen 不是O(n) 么？

如题。

/*

• 释放 sds 所对应的 sdshdr 结构的内存
• 给定 sds 必须为 NULL
  *
• T = O(N)
  */

void sdsfree(sds s) {
if (s == NULL) return;
zfree(s-sizeof(struct sdshdr));
}
这里的 “给定 sds 必须为 NULL” 是不是笔误写错了

这里似乎会产生一个误解，ZADD 一开始都是使用的ziplist，只有满足 zset_max_ziplist_entries 或是 zset_max_ziplist_value 时才会把ziplist转换成skiplist。

我看到后面的部分书中也解释了这个问题。
但是在我读到跳跃表着一个章节，调试源代码的时候，被这个例子纠结了不少时间。我感觉标注一下会好很多。

我评论了一下，但是似乎不会收到回复，所以我就在这里new issue了 ： ）

这一行
https://github.com/huangz1990/annotated_redis_source/blob/unstable/src/rio.c#L83
应该是
“将长度为 len 的内容从 RIO 缓存复制到 buf 中。”
：）

你好，请问一下，要用什么来搭建redis的编译环境？我想进行调试，跟踪代码，我的系统是mac。谢谢！！！

在rdb.c 的第121行:
len = htonl(len);
为什么要这步?

求解答.谢谢.

建议把添加注视前的代码checkout到一个分支(base). 方便对比, 看了看 好像是从bf0852e5ed56dfbe32e19d926f8d5cc76f750cc8 后开始修改的.
c5e913e 注释 t_string.c/getGenericCommand 函数
7060e07 注释 t_string.c/setGenericCommand 函数
bf0852e Merge pull request #807 from bmcmanus/issue-804
ab2924c Memory leak fixed: release client's bpop->keys dictionary.
还有,从unstable开发,不是一个好的选择.

https://github.com/huangz1990/annotated_redis_source/blob/unstable/src/t_zset.c
上的注释说的是o(n),但是为啥官方文档上说是log(n)+m呢?

建议加个导读，就是源码的推荐阅读顺序

Found a typo in line 967, src/redis.h

the "lopp" should be "loop".
:)

防止类似这样的情况： #7

https://github.com/huangz1990/annotated_redis_source/blob/67758fdfe196d96ddc8a83784fa48f22226a3818/deps/lua/src/loslib.c#L60

https://github.com/huangz1990/redis-3.0-annotated/blob/8e60a75884e75503fb8be1a322406f21fb455f67/deps/lua/src/loslib.c#L60

Good afternoon.
I noticed in your code the functions using tmpnam. the problem with this function is that it only determines the availability of the filename at the time it is called.
those. when you start working with this name, you are not sure about the absence of a file with this name.
if it is important then you need to use mkstemp or mkdtemp.