| commit | 67778e6080696fdb422f2b0e2dea85b00f26aca2 | |
| author | Johannes Sixt <j6t@kdbg.org> | |
| Sun, 4 Jan 2009 16:39:35 +0000 (4 17:39 +0100) | ||
| committer | Johannes Sixt <j6t@kdbg.org> | |
| Mon, 5 Jan 2009 13:07:15 +0000 (5 14:07 +0100) | ||
| tree | 9e89bdc367e4563117b0f9b7e994c4bbc3d2de2d | treesnapshot (tar.gz zip) |
| parent | 2fa4a36c3f07f024eb64ca231c19bf9cdbdfcf86 | commitdiff |
Convert parseLocals() from Q[Ptr]List to std::list.
Note that since ExprValue represents a whole tree, we do not make the list
value-based, but rather keep it as a list of pointers in order to avoid
that whole expression trees need to be copied.
We could have changed the functions parseLocals() so that they return the
result in the return value. But since we keep pointers to ExprValue
objects, we define that the owner of the objects is the caller (even if
there were parse errors). To stress the responsibility of the caller to
free the items, we keep the call-by-reference style.
Note that since ExprValue represents a whole tree, we do not make the list
value-based, but rather keep it as a list of pointers in order to avoid
that whole expression trees need to be copied.
We could have changed the functions parseLocals() so that they return the
result in the return value. But since we keep pointers to ExprValue
objects, we define that the owner of the objects is the caller (even if
there were parse errors). To stress the responsibility of the caller to
free the items, we keep the call-by-reference style.