[Libva] [PATCH 1/2] test/encode: remove unused variables and functions

2015-09-16 Thread U. Artie Eoff
This gets rid of compiler warnings caused by various unused variables and functions. Signed-off-by: U. Artie Eoff --- test/encode/avcenc.c | 44 ++-- test/encode/mpeg2vaenc.c | 1 - test/loadsurface.h | 24

Re: [Libva] [PATCH 1/2] test/encode: remove unused variables and functions

2015-09-16 Thread Sean V Kelley
+Haihao, Yakui Can you comment on what appears to be code brought in from h264encode that isn't being exercised? There are two things here I want to understand: 1) Brought in from h264encode, oversight? 2) Commented out for specific manual testing or not? I want to hold off on this merge until

Re: [Libva] [PATCH 1/2] test/encode: remove unused variables and functions

2015-09-16 Thread Kelley, Sean V
On Thu, 2015-09-17 at 08:54 +0800, Zhao Yakui wrote: On 09/17/2015 12:12 AM, U. Artie Eoff wrote: > This gets rid of compiler warnings caused by various unused > variables and functions. The variables are not used and it is OK to remove them for fixing the compiler warnings. This patch looks

Re: [Libva] [PATCH 1/2] test/encode: remove unused variables and functions

2015-09-16 Thread Zhao Yakui
On 09/17/2015 12:12 AM, U. Artie Eoff wrote: This gets rid of compiler warnings caused by various unused variables and functions. The variables are not used and it is OK to remove them for fixing the compiler warnings. This patch looks good to me. Signed-off-by: U. Artie