看到 Linux 内核 Git 仓库的一个 Pull Request, 修正了一个漏分号的错误

[snowonion]

https://github.com/torvalds/linux/pull/84/files

↑ 原先的代码应该是会出编译错误吧? 为什么有编译错误的代码会进入仓库呢...

是不是 Mr. Torvalds 他们对代码的检查是周期性进行的, 而非 merge 每个 Pull Request 时都会检查?

结论：
list_for_each 是个宏。根据它的展开内容，加分号是不必要的。所以那个 Pull Request 是不必要的。

[wwqgtxx]

有可能是git merge导致的错误，有些时候git merge时会导致一些莫名其妙的错误

[ChiangDi]

这也行，真是惊呆了，他们不测试就提交的？

[akfish]

看下blame不就知道了

[dorentus]

编译错误一般来说不是人工检查的重点，大都是用自动化的系统来检查的吧，一般是异步的。

然后 Linux 项目并不使用 Github 的 Pull Request 功能： https://github.com/torvalds/linux/pull/17#issuecomment-5654674

[akfish]

https://github.com/FriendlyHacker/linux/commit/63978ab3e3e963db28093b53bb4598f2702e1ad7#diff-6cc4d399b1e9db27c11370eeee08bdb3R1406
2009年的一次commit引入的。

[akfish]

其实我想吐槽的是这不是bug。。。。
list_for_each并不是函数调用，而是一个宏
https://github.com/torvalds/linux/blob/164c09978cebebd8b5fc198e9243777dbaecdfa0/drivers/staging/usbip/userspace/libsrc/list.h#L110

[dorentus]

仔细看了下，这 Pull Request 不对……

list_for_each 大概是个宏，用法类似 for 或者 foreach：

例如 https://github.com/torvalds/linux/blob/164c09978cebebd8b5fc198e9243777dbaecdfa0/drivers/usb/mon/mon_main.c:

list_for_each (p, &mon_buses) {
struct mon_bus *m1;
m1 = list_entry(p, struct mon_bus, bus_link);
m1->u_bus->monitored = 1;
}

Pull Request 里面的下一行明显有个锁进，所以只是类似单行 for 省略了大括号而已；加了分号之后，即使不会报错，语义也变了。

[fo2w]

从代码和conversation看这不是个bug, 而且是编的过的啊
这展开仅仅是个for循环而且下一行就是循环体snd_usbmidi_disconnect(midi);
还专门缩进了用来提醒!
就这么贸然加个分号, 让循环体怎么想.....

[limon]

在c里xxx_for_each这种命名明显就是宏嘛。。。

[snowonion]

谢谢各位~
跟 C 语言不熟, 没想到宏这回事=.=

如果不是
#define list_for_each(pos, head)
而是
#define list_for_each(pos, head):
会不会有助于避免误解 ( ﾟ∀。)

[zoudm]

@snowonion

C里面目测不这样用 [：] 。。。

看着真像python。。

[Youmu]

@snowonion 这个编译就会出错呀，宏替换的时候不会替换那多余的`:'的。

[snowonion]

@Youmu 多谢~ 窝又没试就口胡啦_(:3」∠)_