This patch fixes some formatting and code clarity issues I've tripped
over in the past few months.
duplicate_decls has some function- and var- specific behaviour that was
confusingly commented, and, for the var case mixed the checking for var
with checking for the flag to copy.
process_outer
A few cleanups from the modules branch.
* The TEMPLATE_INFO comments still described TREE_LIST contents.
* we didn't check we were looking at a TEMPLATE_INFO in one case
* we declared some (now?) non-existent dump functions
* we made a name-lookup internal dumping function extern
* we made a temp
