glsl: Switch ast_node to the non-zeroing allocator.
authorFrancisco Jerez <currojerez@riseup.net>
Fri, 20 Sep 2013 22:36:38 +0000 (15:36 -0700)
committerFrancisco Jerez <currojerez@riseup.net>
Wed, 2 Oct 2013 00:30:51 +0000 (17:30 -0700)
commit8bd1c69f3bc731a53d785aa7b3e8fea05bacef7d
tree313a298efb8b34f92169efa1774f43f185d4171d
parent70953b5fea1445fe121ac4b4a816c984742f2e19
glsl: Switch ast_node to the non-zeroing allocator.

All member variables of ast_node are already being initialized from
its constructor, but some of its derived classes were leaving members
uninitialized -- Fix them.

Using rzalloc makes it more likely that we will start relying on the
allocator to zero out all memory if the class is ever extended with
new member variables.  That's bad because it ties objects to some
specific allocation scheme, and gives unpredictable results when an
object is created with a different allocator -- Stack allocation,
array allocation, or aggregation inside a different object are some of
the useful possibilities that come to my mind.

v2: Use NULL initialization instead of default construction for pointers.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/glsl/ast.h
src/glsl/glsl_parser_extras.cpp