Re: Ping [PATCH] c++/42121 - diagnose invalid flexible array members

[Jason Merrill]

On 12/14/2015 11:45 AM, Martin Sebor wrote:

+  if (NULL_TREE == size)

Usually NULL_TREE goes on the right.

@@ -8744,6 +8748,7 @@ compute_array_index_type (tree name, tree size, 
tsubst_flags_t complain)
          else
            pedwarn (input_location, OPT_Wpedantic, "ISO C++ forbids zero-size 
array");
        }
+
     }

Unnecessary blank line.

+           if (TREE_CODE (type) != ARRAY_TYPE
+               || !COMPLETE_TYPE_P (TREE_TYPE (type)))
+             {
+
                if (unqualified_id)

Here too.

OK with those changes and the one Jakub suggested.

Jason