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:k...@dev.carbon-project.org] Sent: August-20-16 12:15 PM To: mesa-dev@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 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
