I noticed I made a bit of a mistake in grokdeclarator:find_xobj_parm, this code: ``` if (!parm_list || parm_list == void_list_node) return false; if (TREE_PURPOSE (parm_list) != this_identifier) return false; ``` Can be simplified to this code: ``` if (!parm_list || TREE_PURPOSE (parm_list) != this_identifier) return false; ```
While working on lambda support I found that I need to find the xobj parameter before we get to grokdeclarator, so I was going to reuse the same code. After looking at it I realized, hey, I wonder what void_list_node's purpose field holds. So I checked, and found that tree.cc:build_common_tree_nodes initializes void_list_node with a NULL_TREE for it's purpose field. It's obviously not going to be a big deal but simpler code is better. I will most likely fix this if I have time later just so the code doesn't look semantically different in the lambda support patch. Alex
