warning: using the result of an assignment as a condition

警告的意思 :使用了表达式的结果作为条件判断的变量

GCC还是希望代码尽量的没有争议,将表达式的结果去进行判断。

改法 :先将表达式的结果计算出来,在传入 if ,for,while 之类的()里面去

if (payload.error = std::forward<Invocable>(f)(payload.attribute))

改为

 payload.error = std::forward<Invocable>(f)(payload.attribute);
 if (payload.error)


https://github.com/NVIDIA/cub/issues/219

In file included from /thrust/cub/block/../iterator/cache_modified_input_iterator.cuh:42:
/thrust/cub/block/../iterator/../util_device.cuh:323:35: warning: using the result of an assignment as a condition without parentheses [-Wparentheses]
        if (payload.error = std::forward<Invocable>(f)(payload.attribute))
                    ~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/thrust/cub/block/../iterator/../util_device.cuh:323:35: note: place parentheses around the assignment to silence this warning
        if (payload.error = std::forward<Invocable>(f)(payload.attribute))
                          ^
                    (                                                            )
/thrust/cub/block/../iterator/../util_device.cuh:323:35: note: use '==' to turn this assignment into an equality comparison
        if (payload.error = std::forward<Invocable>(f)(payload.attribute))
                          ^
                          ==



Would it make sense to split this into two lines?


 payload.error = std::forward<Invocable>(f)(payload.attribute);
 if (payload.error)

评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包

打赏作者

魔尊moon

你的鼓励将是我创作的最大动力

¥1 ¥2 ¥4 ¥6 ¥10 ¥20
扫码支付:¥1
获取中
扫码支付

您的余额不足,请更换扫码支付或充值

打赏作者

实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值