aboutsummaryrefslogtreecommitdiff
path: root/gcc
diff options
context:
space:
mode:
authorMark Mitchell <mark@codesourcery.com>2003-04-29 20:17:00 +0000
committerMark Mitchell <mmitchel@gcc.gnu.org>2003-04-29 20:17:00 +0000
commit60cde93696c3a8c25f85606b31645fb9c17af56b (patch)
tree7e942eaf69ef180aa8e849168fa65f672ed1d5d7 /gcc
parentcc9d1c78ef92f6a29dc0bb4b4531504320267a20 (diff)
downloadgcc-60cde93696c3a8c25f85606b31645fb9c17af56b.zip
gcc-60cde93696c3a8c25f85606b31645fb9c17af56b.tar.gz
gcc-60cde93696c3a8c25f85606b31645fb9c17af56b.tar.bz2
re PR c++/10527 (confused error message with "new int()" parameter initializer)
PR c++/10527 * error.c (dump_expr): Correctly handling of NEW_EXPR.4 PR c++/10527 * g++.dg/init/new7.C: New test. From-SVN: r66247
Diffstat (limited to 'gcc')
-rw-r--r--gcc/cp/ChangeLog5
-rw-r--r--gcc/cp/error.c12
-rw-r--r--gcc/testsuite/ChangeLog7
-rw-r--r--gcc/testsuite/g++.dg/init/new7.C7
4 files changed, 28 insertions, 3 deletions
diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 98d9fdb..3a65705 100644
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,8 @@
+2003-04-29 Mark Mitchell <mark@codesourcery.com>
+
+ PR c++/10527
+ * error.c (dump_expr): Correctly handling of NEW_EXPR.4
+
2003-04-29 Kriang Lerdsuwanakij <lerdsuwa@users.sourceforge.net>
* call.c (build_operator_new_call): Fix typo.
diff --git a/gcc/cp/error.c b/gcc/cp/error.c
index f836ecc..3e299de 100644
--- a/gcc/cp/error.c
+++ b/gcc/cp/error.c
@@ -1603,6 +1603,7 @@ dump_expr (tree t, int flags)
case NEW_EXPR:
{
tree type = TREE_OPERAND (t, 1);
+ tree init = TREE_OPERAND (t, 2);
if (NEW_EXPR_USE_GLOBAL (t))
print_scope_operator (scratch_buffer);
output_add_string (scratch_buffer, "new ");
@@ -1619,10 +1620,17 @@ dump_expr (tree t, int flags)
TREE_OPERAND (type, 1),
integer_one_node))));
dump_type (type, flags);
- if (TREE_OPERAND (t, 2))
+ if (init)
{
print_left_paren (scratch_buffer);
- dump_expr_list (TREE_OPERAND (t, 2), flags);
+ if (TREE_CODE (init) == TREE_LIST)
+ dump_expr_list (init, flags);
+ else if (init == void_zero_node)
+ /* This representation indicates an empty initializer,
+ e.g.: "new int()". */
+ ;
+ else
+ dump_expr (init, flags);
print_right_paren (scratch_buffer);
}
}
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 45326b2..0a670f8 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,7 +1,12 @@
2003-04-29 Mark Mitchell <mark@codesourcery.com>
+ PR c++/10527
+ * g++.dg/init/new7.C: New test.
+
+2003-04-29 Mark Mitchell <mark@codesourcery.com>
+
* g++.dg/ext/desig1.C: New test.
- * g++.dg/ext/init1.C: Likewise.
+ * g++.dg/ext/init1.C: Update.
* g++.old-deja/g++.pt/deduct5.C: Remove unnecessary initializer.
diff --git a/gcc/testsuite/g++.dg/init/new7.C b/gcc/testsuite/g++.dg/init/new7.C
new file mode 100644
index 0000000..e07d226
--- /dev/null
+++ b/gcc/testsuite/g++.dg/init/new7.C
@@ -0,0 +1,7 @@
+template <class T>
+ struct Foo
+ {};
+
+ template <class T>
+ void Foo<T>::NON_EXISTENT(int* val = new int()) {} // { dg-error "" }
+