忽略gcc/clang的“-Wmissing-braces”警告是明智的吗？

Question

考虑下面的程序：

#include <array> 

int main() 
{ 
    std::array<int, 1> x = { 0 }; // warning! 
    x = { { 0 } }; // no warning 
    return 0; 
} 

首先初始化导致对GCC 4.7.2警告...

main.cpp:5:22: warning: unused variable ‘x’ [-Wunused-variable] 

...铛和3.1

main.cpp:5:28: warning: suggest braces around initialization of subobject [-Wmissing-braces] 
    std::array<int, 1> x = { 0 }; 

据作为标准，至少在本例中，双花括号或单花括号应该没有区别。

有两种方法来处理警告：

1. 就关掉它
2. 修复代码，所以编译器是幸福

你有什么建议？恕我直言，双卷曲表达看起来有些丑陋。另一方面，这个警告可能会在更复杂的例子中发现真正的问题。你知道一个警告会帮助你的例子吗？

Answer 1

-Wmissing-braces will no longer be enabled in GCC's -Wall (for C++ mode), as of 4.8, for precisely the reason you describe.对于当前版本的GCC，无论是禁用还是忽略警告，您所拥有的代码都是按照它的方式编写的。

警告可能是用来支付代码，比如

struct A { int a; int b; }; 
struct B { A a; int b; }; 
B b = { 
    1, 
    2 // initialises b.a.b, not b.b 
}; 

但是，恕我直言，这已经被-Wmissing-field-initializers，不警告你原来的代码来处理不够好。

Answer 2

我在Xcode 6.1.1（截至2015年3月9日当前版本）中得到相同的警告。当我在每个子对象周围添加额外的大括号时，出现错误。当我在整个初始化列表中添加一组额外的大括号时，警告消失。根据标准规范14882：2011 23.3.2.1 [array.overview]第2明确规定

array<T, N> a = { initializer-list }; 

其中初始化-list是逗号分隔的多达N个元素列表 ，其类型可转换为Ť

在Xcode 6.1.1（下面）

array<int, 2> key1 = {1, 2}; // warning: suggest braces around initialization of subobject 

array<int, 2> key2 = { {1}, {2} }; // error: no viable overload = 

array<int, 2> key3 = array<int, 2> { {1}, {2} }; // error: excess elements in struct initializer 

array<int, 2> key4 = { {1, 2} }; // no warning and no error 

的代码结果当我们看14882：2011 8.5 [dcl.init]第1，我们看到'initializer-list'可以选择性地包含'initializer-clause'，它本身可以是'braced-init-list'。所以任何一种方式都应该是正确虽然基于规范，我个人认为单个大括号不应该为std :: array初始化程序列表输出编译器警告，并且双大括号是矫枉过正的。

Answer 3

当忽略与-Wno-missing-braces叮当警告，我会建议启用-Wmissing-field-initializers（或使用-Wextra，其中也包括它）。否则，你错过了一个有用的警告就像这个例子：

#include <cstdio> 

struct A 
{ 
    int i; 
    int arr[2]; 
    int j; 
}; 

void print(const A& a) 
{ 
    printf("i=%d, arr={%d,%d}, j=%d\n", a.i, a.arr[0], a.arr[1], a.j); 
} 

int main() { 
    A a = {1, 2, 3}; // this is the critical line 
    print(a); // output: i=1, arr={2,3}, j=0 

    A b = {1, {2}, 3}; 
    print(b); // output: i=1, arr={2,0}, j=3 

    A c = {1, {2,0}, 3}; 
    print(c); // output: i=1, arr={2,0}, j=3 

    return 0; 
} 

$ clang++ -Wall example.cpp 
example.cpp:16:13: warning: suggest braces around initialization of 
     subobject [-Wmissing-braces] 
    A a = {1, 2, 3}; 
      ^~~~ 
      { } 
1 warning generated. 

$ clang++ -Wall -Wno-missing-braces example.cpp 
(no warnings) 

$ clang++ -Wall -Wno-missing-braces -Wmissing-field-initializers example.cpp 
example.cpp:16:17: warning: missing field 'j' initializer 
     [-Wmissing-field-initializers] 
    A a = {1, 2, 3}; 
       ^
1 warning generated. 

$ clang++ --version 
clang version 3.8.1 (tags/RELEASE_381/final) 

为了便于比较，这是海湾合作委员会的作用：

$ g++ -Wall -Wextra example.cpp 
(no warning) 

$ g++ -Wall -Wmissing-field-initializers example.cpp 
example.cpp: In function ‘int main()’ 
example.cpp:16:17: warning: missing initializer for member ‘A::j’ [-Wmissing-field-initializers] 
    A a = {1, 2, 3}; 
       ^

总结：

• 对于铛，我会建议-Wno-missing-braces -Wmissing-field-initializers在不丢失其他有用警告的情况下使警告消失
• GCC在原始std::array<int, 1> x = { 0 };示例中没有抱怨，因此不需要禁用任何警告。但是，我建议启用-Wmissing-field-initializers（或使用-Wextra），因为它不是由-Wall启用的。

Answer 4

即将发布的叮当声6.0会压制关于失踪大括号的警告。 svn日志说：

Suppress -Wmissing-braces警告当聚合 - 初始化一个结构与一个单一的领域本身是一个聚合。在C++中，std :: array类型的这种初始化是保证按标准工作，完全是惯用的，并且来自 Clang的“建议”替代在技术上是无效的。

因此，我会省略大括号，并禁用-Wmissing-braces Cla之前6.0。