t_dd_dmatmp: Replace fprintf with unreachable
authorIan Romanick <ian.d.romanick@intel.com>
Fri, 18 Sep 2015 18:56:13 +0000 (14:56 -0400)
committerIan Romanick <ian.d.romanick@intel.com>
Tue, 6 Oct 2015 17:44:00 +0000 (10:44 -0700)
commit5ca00e0b8dff529f33c8b1685a9109a78090987c
tree87284525bcf02de56e605cbd84ec84ed48b82eee
parent46b13666d8c045acf066f2982cc0a89b6584f09d
t_dd_dmatmp: Replace fprintf with unreachable

From http://lists.freedesktop.org/archives/mesa-dev/2015-May/084883.html:

    "There are no real error cases here, just dead code.
    validate_render() is supposed to make sure we never call these
    functions if the code can't actually render the primitives. The
    fprintf()+return branches should really just contain assert(0) or
    equivalent."

I also rearranged the if-else-block in render_quad_strip_verts to look
more like the other functions.  A future patch is going to change a
bunch of that code anyway.

v2: Make "unreachable" message more descriptive.  Suggested by Iago.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Suggested-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
src/mesa/tnl_dd/t_dd_dmatmp.h