Commit: a9d2c1f0e21041163a252a023bfaad3e929be5ab - hardware-intel-common-vaapi (git)

hardware/intel/common/vaapi

Commit MetaInfo

• Revision: a9d2c1f0e21041163a252a023bfaad3e929be5ab (tree)
• Time: 2019-02-26 12:27:33
• Author: Chih-Wei Huang <cwhuang@linu...>
• Commiter: Chih-Wei Huang

Log Message

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 <[e-mail]>

Change Summary

• modified: src/gen75_vme.c
• modified: src/gen9_vme.c

Incremental Difference

--- 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;
     }

--- 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;
     }