c++ - 捕获未分配的 r 值的编译器选项

Question

相当尴尬的是，我写了类似下面的东西(经过 sanitizer ):

vector_item next()
{
    if (this->it == this->myVector.end()){
        this->it == this->myVector.begin();
    }
    return *(this->it++);
}

明显的错误非常明显。然而，它确实花了很短的时间来追踪。

没有为此生成编译器警告，但应该生成一个吗？这里创建了一个未使用的 r 值(不是说，一个未使用的函数返回值，因为它的副作用而被调用)，在我看来这表明代码有问题。

使用 g++ -Wall -Wextra 编译。 (海合会 4.8.3)

我知道 -Wunused-result 但这里不适用。

Answer 1

No compiler warnings were generated for this, but should one have been?

我想不出我在标准中读到的任何内容需要在此处发出警告。

但是，clang 开发团队似乎认为有必要这样做:

18 : <source>:18:18: warning: equality comparison result unused [-Wunused-comparison]
        this->it == this->myVector.begin();
        ~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
18 : <source>:18:18: note: use '=' to turn this equality comparison into an assignment
        this->it == this->myVector.begin();
                 ^~
                 =
1 warning generated.

我的第一个想法是这是 gcc 中的 QoI 问题。可能值得将其作为一个问题提出。

我很幸运，因为我们的软件是为 mac (clang)、linux (gcc) 和 windows (msvc) 编译的，因此可以及早发现违反标准的情况和边缘情况。

在一次又一次地寻找错误之前，在另一个编译器上重新编译您的代码可能是个好主意 - 它对我有帮助。