On Tue, Jan 3, 2017 at 5:39 PM, Jakub Jelinek <ja...@redhat.com> wrote: > + if (auto_function_declaration > + && (TREE_CODE (decl) == FUNCTION_DECL > + || auto_function_declaration != error_mark_node)) > + { > + error_at (decl_specifiers.locations[ds_type_spec], > + "non-variable %qD in declaration with more than one " > + "declarator with placeholder type", > + TREE_CODE (decl) == FUNCTION_DECL > + ? decl : auto_function_declaration); > + auto_function_declaration = error_mark_node; > + } > + else if (auto_function_declaration == NULL_TREE) > + auto_function_declaration > + = TREE_CODE (decl) == FUNCTION_DECL ? decl : error_mark_node;
I might reorder the then/else clauses so you don't need to duplicate the test whether auto_function_declaration is null, but OK either way. Jason