[Mesa-dev] [PATCH 0/2] st/va: minor clean-ups

[Zhang, Boyuan]

Hi Kai,

Thanks for cleaning up the codes. The duplicated end_frame and unused variable are due to the new patch submitted last Friday. I have reviewed both patches.

Regards,
Boyuan

-----Original Message-----
From: Kai Wasserbäch [mailto:kai at dev.carbon-project.org] 
Sent: August-20-16 12:15 PM
To: mesa-dev at lists.freedesktop.org
Cc: Zhang, Boyuan
Subject: [PATCH 0/2] st/va: minor clean-ups

Hey,
just noticed a duplicate call to context->decoder->end_frame() while looking through the recent changes. This is just a trivial clean-up and no functional change is intended. And while I was there, I also noticed an unused variable, which the second patch removes.

Cheers,
Kai

P.S.: If this "series" gets accepted, please commit it for me, since I do not have commit access.


Kai Wasserbäch (3):
  st/va: Remove else case in vlVaEndPicture() made superfluous by
    c59628d11b
  st/va: Remove unused variable coded_size from vlVaEndPicture()

 src/gallium/state_trackers/va/picture.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

--
2.8.1