diff options
| author | Chih-Wei Huang <cwhuang@linux.org.tw> | 2019-02-26 11:27:33 +0800 |
|---|---|---|
| committer | Xiang, Haihao <haihao.xiang@intel.com> | 2019-03-07 09:34:20 +0800 |
| commit | bf17e8ce7dc8b42afb09e1ba40efc438d6dd9a76 (patch) | |
| tree | 3c2eb268c372cec435a51d40c34c5e8a3c49e716 | |
| parent | 6986a536b0cf9956987e2cb8e2b955420e18d195 (diff) | |
| download | libva-intel-driver-bf17e8ce7dc8b42afb09e1ba40efc438d6dd9a76.tar.gz | |
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>
| -rw-r--r-- | src/gen75_vme.c | 1 | ||||
| -rw-r--r-- | src/gen9_vme.c | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/src/gen75_vme.c b/src/gen75_vme.c index 64b7dc01..040dbf59 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 19a9097e..af7ef12b 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; } |