On 15.12.2015 11:08, Emil Velikov wrote:
On 11 December 2015 at 12:33, Christian König <deathsim...@vodafone.de> wrote:
From: Christian König <christian.koe...@amd.com>

Signed-off-by: Christian König <christian.koe...@amd.com>
---
  src/gallium/state_trackers/va/picture.c | 1 -
  1 file changed, 1 deletion(-)

diff --git a/src/gallium/state_trackers/va/picture.c 
b/src/gallium/state_trackers/va/picture.c
index 8623139..7b30bf8 100644
--- a/src/gallium/state_trackers/va/picture.c
+++ b/src/gallium/state_trackers/va/picture.c
@@ -92,7 +92,6 @@ vlVaGetReferenceFrame(vlVaDriver *drv, VASurfaceID surface_id,
  static VAStatus
  handlePictureParameterBuffer(vlVaDriver *drv, vlVaContext *context, 
vlVaBuffer *buf)
  {
-   unsigned int i;
     VAStatus vaStatus = VA_STATUS_SUCCESS;

Can I bribe you to also remove the "set once, used once" variable vaStatus ?

Unfortunately I already pushed this one yesterday after Julien gave me his rb. But going to keep that in mind when I touch the function the next time.

Regards,
Christian.


Either way
Reviewed-by: Emil Velikov <emil.l.veli...@gmail.com>

-Emil

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to