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

Reply via email to