From: Chih-Wei Huang <cwhuang@linux.org.tw>
Date: Tue, 26 Feb 2019 03:27:33 +0000 (+0800)
Subject: android: avoid compile warnnings
X-Git-Tag: android-x86-8.1-r2
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=refs%2Ftags%2Fandroid-x86-8.1-r2;hp=8a3d57d2b9c5f998cae73e31129421de90db896c;p=android-x86%2Fhardware-intel-common-vaapi.git

android: avoid compile warnnings

To avoid the warnings(errors):

hardware/intel/common/vaapi/src/gen75_vme.c:1070:5: error: variable 'i965_kernel_num' is used uninitialized whenever switch default is taken [-Werror,-Wsometimes-uninitialized]
    default:
    ^~~~~~~
hardware/intel/common/vaapi/src/gen75_vme.c:1078:35: note: uninitialized use occurs here
    vme_context->vme_kernel_sum = i965_kernel_num;
                                  ^~~~~~~~~~~~~~~

hardware/intel/common/vaapi/src/gen9_vme.c:2036:5: error: variable 'i965_kernel_num' is used uninitialized whenever switch default is taken [-Werror,-Wsometimes-uninitialized]
    default:
    ^~~~~~~
hardware/intel/common/vaapi/src/gen9_vme.c:2044:35: note: uninitialized use occurs here
    vme_context->vme_kernel_sum = i965_kernel_num;
                                  ^~~~~~~~~~~~~~~

I am not going to disable the warning (by -Wno-sometimes-uninitialized)
since it's still a useful checking from the compiler.

Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>
---

diff --git a/src/gen75_vme.c b/src/gen75_vme.c
index 64b7dc0..040dbf5 100644
--- a/src/gen75_vme.c
+++ b/src/gen75_vme.c
@@ -1070,6 +1070,7 @@ Bool gen75_vme_context_init(VADriverContextP ctx, struct intel_encoder_context *
     default:
         /* never get here */
         assert(0);
+        i965_kernel_num = 0;
 
         break;
     }
diff --git a/src/gen9_vme.c b/src/gen9_vme.c
index 19a9097..af7ef12 100644
--- a/src/gen9_vme.c
+++ b/src/gen9_vme.c
@@ -2036,6 +2036,7 @@ Bool gen9_vme_context_init(VADriverContextP ctx, struct intel_encoder_context *e
     default:
         /* never get here */
         assert(0);
+        i965_kernel_num = 0;
 
         break;
     }