c-typeck.c (c_incomplete_type_error): Refactor to use %qT.
authorMarek Polacek <polacek@redhat.com>
Thu, 30 Apr 2015 09:35:05 +0000 (09:35 +0000)
committerMarek Polacek <mpolacek@gcc.gnu.org>
Thu, 30 Apr 2015 09:35:05 +0000 (09:35 +0000)
* c-typeck.c (c_incomplete_type_error): Refactor to use %qT.  Print
the type of a decl.

* gcc.dg/incomplete-typedef-1.c: New test.

From-SVN: r222614

gcc/c/ChangeLog
gcc/c/c-typeck.c
gcc/testsuite/ChangeLog
gcc/testsuite/gcc.dg/incomplete-typedef-1.c [new file with mode: 0644]

index 0ee7f1bb8232514e422d9cc4ae64b39ccac9a426..3894bb9cbe1274902e51ee84d604b433c8e23dff 100644 (file)
@@ -3,6 +3,9 @@
        * c-typeck.c (set_init_label): Call error_at instead of error and
        pass LOC to it.
 
+       * c-typeck.c (c_incomplete_type_error): Refactor to use %qT.  Print
+       the type of a decl.
+
 2015-04-29  Thomas Schwinge  <thomas@codesourcery.com>
 
        * c-parser.c (c_parser_oacc_enter_exit_data): Use
index 466079f42c5372f4bf0d95f95991d8857418e928..413cd07b2e345cda997ab885949950f3148b9891 100644 (file)
@@ -229,15 +229,13 @@ require_complete_type (tree value)
 void
 c_incomplete_type_error (const_tree value, const_tree type)
 {
-  const char *type_code_string;
-
   /* Avoid duplicate error message.  */
   if (TREE_CODE (type) == ERROR_MARK)
     return;
 
   if (value != 0 && (TREE_CODE (value) == VAR_DECL
                     || TREE_CODE (value) == PARM_DECL))
-    error ("%qD has an incomplete type", value);
+    error ("%qD has an incomplete type %qT", value, type);
   else
     {
     retry:
@@ -246,15 +244,8 @@ c_incomplete_type_error (const_tree value, const_tree type)
       switch (TREE_CODE (type))
        {
        case RECORD_TYPE:
-         type_code_string = "struct";
-         break;
-
        case UNION_TYPE:
-         type_code_string = "union";
-         break;
-
        case ENUMERAL_TYPE:
-         type_code_string = "enum";
          break;
 
        case VOID_TYPE:
@@ -280,11 +271,10 @@ c_incomplete_type_error (const_tree value, const_tree type)
        }
 
       if (TREE_CODE (TYPE_NAME (type)) == IDENTIFIER_NODE)
-       error ("invalid use of undefined type %<%s %E%>",
-              type_code_string, TYPE_NAME (type));
+       error ("invalid use of undefined type %qT", type);
       else
        /* If this type has a typedef-name, the TYPE_NAME is a TYPE_DECL.  */
-       error ("invalid use of incomplete typedef %qD", TYPE_NAME (type));
+       error ("invalid use of incomplete typedef %qT", type);
     }
 }
 
index 861077bab8bbd44271a22f7634c6266a7455738d..29e177ee35295438345a234b6331c799b3a07ce3 100644 (file)
@@ -2,6 +2,8 @@
 
        * gcc.dg/init-bad-8.c: New test.
 
+       * gcc.dg/incomplete-typedef-1.c: New test.
+
 2015-04-30  Paolo Carlini  <paolo.carlini@oracle.com>
 
        PR c++/59955
diff --git a/gcc/testsuite/gcc.dg/incomplete-typedef-1.c b/gcc/testsuite/gcc.dg/incomplete-typedef-1.c
new file mode 100644 (file)
index 0000000..622bf65
--- /dev/null
@@ -0,0 +1,12 @@
+/* { dg-do compile } */
+/* { dg-options "" } */
+
+typedef struct S TS;
+typedef union U TU;
+
+void
+foo (void)
+{
+  (TS) { }; /* { dg-error "invalid use of incomplete typedef" } */
+  (TU) { }; /* { dg-error "invalid use of incomplete typedef" } */
+}