On 01/30/2013 11:27 AM, Ian Romanick wrote:
From: Ian Romanick <ian.d.roman...@intel.com>
Previously the loop_state was allocated in the loop_analysis
constructor, but not freed in the (nonexistent) destructor. Moving
the allocation of the loop_state makes this code appear less sketchy.
Either way, there is no actual leak. The loop_state is freed by the
single caller of analyze_loop_variables.
Signed-off-by: Ian Romanick <ian.d.roman...@intel.com>
Cc: Dave Airlie <airl...@freedesktop.org>
Cc: Matt Turner <matts...@gmail.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=57753
---
src/glsl/loop_analysis.cpp | 13 ++++++-------
src/glsl/loop_analysis.h | 2 +-
2 files changed, 7 insertions(+), 8 deletions(-)
Oh, Coverity.
This patch shouldn't matter, but it shouldn't hurt, so...meh.
Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev