GCC 6.3 警告常量“0”与布尔表达式的比较总是错误的

Question

争吵指向这段代码

#define DO_SAFE(do_work, goto_error)                \
 DO_IF(unlikely((do_work) < 0), goto_error)

这是来自一个头文件。我不明白这段代码试图做什么或比较。对我来说似乎 do_work if false 然后 goto 错误。但如果这是正确的，它会是

 (do_work != 0) 

但这只是猜测。

如果需要更多信息，请告知。

函数在c文件中出错

for (page_num = 0; page_num < PAGE_MAX_NUM; page_num++) {
    DO_SAFE(touch_i2c_write_byte(client, PAGE_SELECT_REG, page_num), error);

for (u_address = DESCRIPTION_TABLE_START; u_address > 10; u_address -= sizeof(struct function_descriptor)) 
{
DO_SAFE(touch_i2c_read(client, u_address, sizeof(buffer), (unsigned char *)&buffer) < 0, error);

Answer 1

哦，我看到了问题（如果我水平滚动 - 线条相当长）：

DO_SAFE(touch_i2c_read(client, u_address, sizeof(buffer), (unsigned char *)&buffer) < 0, error);

您正在宏调用中测试< 0 条件，这意味着您最终会扩展DO_SAFE：

if ((touch_i2c_read(…) < 0) < 0)

并且内部条件的结果永远不会小于零。

用途：

DO_SAFE(touch_i2c_read(client, u_address, sizeof(buffer), (unsigned char *)&buffer), error);

（这已经在您直接调用DO_SAFE 时消除了< 0，让宏来测试< 0。）