On 05/25/2011 10:21 PM, Nathan Froyd wrote:
An alternative solution would be to initialize cur_stmt_list somewhere with an actual 1-element VEC;
Or just push NULL onto the stack and let append_to_statement_list_1 allocate the VEC?
the check in add_stmt would then be unnecessary, as we'd always be assured of having someplace in the stack to store it. I don't trust myself to write a patch like that tonight; I'll twiddle with that tomorrow.
Right, that's what I was thinking about. I think we should only need to do this once per function.
behavior; I don't think it's worthwhile to change all the callers to ensure the pointer-to-statement_list points to a non-NULL thing.
Agreed. Jason
