On Tuesday, May 27, 2014 07:49:04 PM Ian Romanick wrote: > From: Ian Romanick <[email protected]> > > This has the added perk that if you forget to set ir_type in the > constructor of a new subclass (or a new constructor of an existing > subclass) the compiler will tell you... instead of relying on > ir_validate or similar run-time detection. > > Signed-off-by: Ian Romanick <[email protected]> > --- > src/glsl/ir.cpp | 65 > ++++++++++++++++++++++++++------------------------------- src/glsl/ir.h | > 46 +++++++++++++++++++++++----------------- > 2 files changed, 57 insertions(+), 54 deletions(-)
This seems reasonable to me. Patch 9 is: Reviewed-by: Kenneth Graunke <[email protected]>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
