看到 Linux 内核 Git 仓库的一个 Pull Request, 修正了一个漏分号的错误
snowonion · 2014-08-01 17:26:05 +08:00 · 4236 次点击这是一个创建于 3801 天前的主题，其中的信息可能已经有所发展或是发生改变。
https://github.com/torvalds/linux/pull/84/files
↑ 原先的代码应该是会出编译错误吧? 为什么有编译错误的代码会进入仓库呢...
是不是 Mr. Torvalds 他们对代码的检查是周期性进行的, 而非 merge 每个 Pull Request 时都会检查?
↑ 原先的代码应该是会出编译错误吧? 为什么有编译错误的代码会进入仓库呢...
是不是 Mr. Torvalds 他们对代码的检查是周期性进行的, 而非 merge 每个 Pull Request 时都会检查?
第 1 条附言 · 2014-12-04 17:31:35 +08:00
结论：
list_for_each 是个宏。根据它的展开内容，加分号是不必要的。所以那个 Pull Request 是不必要的。
list_for_each 是个宏。根据它的展开内容，加分号是不必要的。所以那个 Pull Request 是不必要的。
 |
1
wwqgtxx 2014-08-01 17:40:45 +08:00 via Android  1
有可能是git merge导致的错误，有些时候git merge时会导致一些莫名其妙的错误
|
 |
2
ChiangDi 2014-08-01 17:45:08 +08:00
这也行，真是惊呆了，他们不测试就提交的？
|
 |
3
akfish 2014-08-01 17:46:04 +08:00
看下blame不就知道了
|
 |
4
dorentus 2014-08-01 17:48:16 +08:00 1
编译错误一般来说不是人工检查的重点，大都是用自动化的系统来检查的吧，一般是异步的。
然后 Linux 项目并不使用 Github 的 Pull Request 功能： https://github.com/torvalds/linux/pull/17#issuecomment-5654674 |
|
5
akfish 2014-08-01 17:55:11 +08:00
|
|
6
akfish 2014-08-01 17:59:09 +08:00 3
其实我想吐槽的是这不是bug。。。。
list_for_each并不是函数调用，而是一个宏 https://github.com/torvalds/linux/blob/164c09978cebebd8b5fc198e9243777dbaecdfa0/drivers/staging/usbip/userspace/libsrc/list.h#L110 |
|
7
dorentus 2014-08-01 17:59:09 +08:00 1
仔细看了下，这 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 省略了大括号而已；加了分号之后，即使不会报错，语义也变了。 |
 |
8
fo2w 2014-08-01 18:05:22 +08:00 1
从代码和conversation看这不是个bug, 而且是编的过的啊
这展开仅仅是个for循环而且下一行就是循环体snd_usbmidi_disconnect(midi); 还专门缩进了用来提醒! 就这么贸然加个分号, 让循环体怎么想..... |
 |
9
limon 2014-08-01 19:00:17 +08:00
在c里xxx_for_each这种命名明显就是宏嘛。。。
|
 |
10
snowonion OP 谢谢各位~
跟 C 语言不熟, 没想到宏这回事=.= 如果不是 #define list_for_each(pos, head) 而是 #define list_for_each(pos, head): 会不会有助于避免误解 ( ﾟ∀。) |
Select Language