Fails to compile, but should work: struct A { char x[4]; A():x("bug") { } }; Error i get is: "main.cpp:3: error: array used as initializer"
(In reply to comment #0) > Fails to compile, but should work: > > struct A { > char x[4]; > A():x("bug") { } > }; > > Error i get is: > > "main.cpp:3: error: array used as initializer" > Why do you think it should work? For example, the following equivalent code is invalid as well: char x [4] ("bug");
(In reply to comment #1) > (In reply to comment #0) > > Fails to compile, but should work: > > > > struct A { > > char x[4]; > > A():x("bug") { } > > }; > > > > Error i get is: > > > > "main.cpp:3: error: array used as initializer" > > > > Why do you think it should work? > For example, the following equivalent code is invalid as well: > > char x [4] ("bug"); > This code is equivalent and is valid. At least, I don't see the Standard forbidding it. GCC is the only compiler I tested (comeau/edg, clang) that rejects it.
(In reply to comment #2) > (In reply to comment #1) > > (In reply to comment #0) > > > Fails to compile, but should work: > > > > > > struct A { > > > char x[4]; > > > A():x("bug") { } > > > }; > > > > > > Error i get is: > > > > > > "main.cpp:3: error: array used as initializer" > > > > > > > Why do you think it should work? > > For example, the following equivalent code is invalid as well: > > > > char x [4] ("bug"); > > > > This code is equivalent and is valid. At least, I don't see the Standard > forbidding it. GCC is the only compiler I tested (comeau/edg, clang) that > rejects it. I'm not actually sure anymore about the validity of this code. One can make a point about the initializer not being a mere string literal. At least the draft n3126 makes a difference of this, in that an initializer like "({a, b, c})" is not regarded as a braced-init-list, but rather as a parenthesized expression-list where the initializer list is handed as one argument. So I'm unsure whether an initializer like `("foo")` should be regarded as a string literal or not. I think I will send an issue report about this.
(In reply to comment #3) > (In reply to comment #2) > > (In reply to comment #1) > > > (In reply to comment #0) > > > > Fails to compile, but should work: > > > > > > > > struct A { > > > > char x[4]; > > > > A():x("bug") { } > > > > }; > > > > > > > > Error i get is: > > > > > > > > "main.cpp:3: error: array used as initializer" > > > > > > > > > > Why do you think it should work? > > > For example, the following equivalent code is invalid as well: > > > > > > char x [4] ("bug"); > > > > > > > This code is equivalent and is valid. At least, I don't see the Standard > > forbidding it. GCC is the only compiler I tested (comeau/edg, clang) that > > rejects it. > > I'm not actually sure anymore about the validity of this code. One can make a > point about the initializer not being a mere string literal. > > At least the draft n3126 makes a difference of this, in that an initializer > like "({a, b, c})" is not regarded as a braced-init-list, but rather as a > parenthesized expression-list where the initializer list is handed as one > argument. So I'm unsure whether an initializer like `("foo")` should be > regarded as a string literal or not. > > I think I will send an issue report about this. Subsequent discussion with Jason showed that this is covered by 8.5p13: The form of initialization (using parentheses or =) is generally insignificant, but does matter when the initializer or the entity being initialized has a class type; As this is an array, the text in the Standard in general has to be interpreted that a "=" or a "(..)" initializer are equivalent, unless otherwise stated. So this is indeed a GCC bug (both that it rejects the member initialization and the parenthesized non-member initialization).
Confirmed.
Mine.
Author: paolo Date: Wed Jun 4 22:30:39 2014 New Revision: 211248 URL: http://gcc.gnu.org/viewcvs?rev=211248&root=gcc&view=rev Log: /cp 2014-06-04 Paolo Carlini <paolo.carlini@oracle.com> PR c++/43453 * typeck.c (cp_build_modify_expr): Handle array of characters initialized by a string literal. * decl.c (check_initializer): Handle parenthesized string literal as initializer. * typeck2.c (store_init_value): Remove redundant check. /testsuite 2014-06-04 Paolo Carlini <paolo.carlini@oracle.com> PR c++/43453 * g++.dg/init/pr43453.C: New. Added: trunk/gcc/testsuite/g++.dg/init/pr43453.C Modified: trunk/gcc/cp/ChangeLog trunk/gcc/cp/decl.c trunk/gcc/cp/typeck.c trunk/gcc/cp/typeck2.c trunk/gcc/testsuite/ChangeLog
Fixed for 4.10.0.
Author: jason Date: Thu Jun 5 20:55:44 2014 New Revision: 211290 URL: http://gcc.gnu.org/viewcvs?rev=211290&root=gcc&view=rev Log: PR c++/43453 * decl.c (check_initializer): Collapse a TREE_LIST here. * typeck2.c (store_init_value): Not here. Modified: trunk/gcc/cp/ChangeLog trunk/gcc/cp/decl.c trunk/gcc/cp/typeck2.c