From 09455123f34ca3b43f3514ed0eab0b3cf0830257 Mon Sep 17 00:00:00 2001 From: Chad Versace Date: Thu, 22 Jun 2017 11:00:40 -0700 Subject: [PATCH] egl/android: Declare loop vars inside their loops (v2) That is, consistently do this: for (int i = 0; ...) No behavioral change. v2: Rebase. Reviewed-by: Eric Engestrom (v1) --- src/egl/drivers/dri2/platform_android.c | 34 +++++++++++-------------- 1 file changed, 15 insertions(+), 19 deletions(-) diff --git a/src/egl/drivers/dri2/platform_android.c b/src/egl/drivers/dri2/platform_android.c index dba5ad6d551..705f92dffa2 100644 --- a/src/egl/drivers/dri2/platform_android.c +++ b/src/egl/drivers/dri2/platform_android.c @@ -64,9 +64,7 @@ static const struct droid_yuv_format droid_yuv_formats[] = { static int get_fourcc_yuv(int native, int is_ycrcb, int chroma_step) { - int i; - - for (i = 0; i < ARRAY_SIZE(droid_yuv_formats); ++i) + for (int i = 0; i < ARRAY_SIZE(droid_yuv_formats); ++i) if (droid_yuv_formats[i].native == native && droid_yuv_formats[i].is_ycrcb == is_ycrcb && droid_yuv_formats[i].chroma_step == chroma_step) @@ -78,9 +76,7 @@ get_fourcc_yuv(int native, int is_ycrcb, int chroma_step) static bool is_yuv(int native) { - int i; - - for (i = 0; i < ARRAY_SIZE(droid_yuv_formats); ++i) + for (int i = 0; i < ARRAY_SIZE(droid_yuv_formats); ++i) if (droid_yuv_formats[i].native == native) return true; @@ -299,9 +295,8 @@ droid_free_local_buffers(struct dri2_egl_surface *dri2_surf) { struct dri2_egl_display *dri2_dpy = dri2_egl_display(dri2_surf->base.Resource.Display); - int i; - for (i = 0; i < ARRAY_SIZE(dri2_surf->local_buffers); i++) { + for (int i = 0; i < ARRAY_SIZE(dri2_surf->local_buffers); i++) { if (dri2_surf->local_buffers[i]) { dri2_dpy->dri2->releaseBuffer(dri2_dpy->dri_screen, dri2_surf->local_buffers[i]); @@ -951,10 +946,10 @@ static int droid_get_buffers_parse_attachments(struct dri2_egl_surface *dri2_surf, unsigned int *attachments, int count) { - int num_buffers = 0, i; + int num_buffers = 0; /* fill dri2_surf->buffers */ - for (i = 0; i < count * 2; i += 2) { + for (int i = 0; i < count * 2; i += 2) { __DRIbuffer *buf, *local; assert(num_buffers < ARRAY_SIZE(dri2_surf->buffers)); @@ -1046,8 +1041,9 @@ droid_add_configs_for_visuals(_EGLDriver *drv, _EGLDisplay *dpy) EGL_RECORDABLE_ANDROID, EGL_TRUE, EGL_NONE }; + unsigned int format_count[ARRAY_SIZE(visuals)] = { 0 }; - int count, i, j; + int count = 0; /* The nesting of loops is significant here. Also significant is the order * of the HAL pixel formats. Many Android apps (such as Google's official @@ -1067,17 +1063,17 @@ droid_add_configs_for_visuals(_EGLDriver *drv, _EGLDisplay *dpy) * (chadversary) testing on Android Nougat, this was good enough to pacify * the buggy clients. */ - count = 0; - for (i = 0; i < ARRAY_SIZE(visuals); i++) { - const EGLint surface_type = EGL_WINDOW_BIT | EGL_PBUFFER_BIT; - struct dri2_egl_config *dri2_conf; + for (int i = 0; i < ARRAY_SIZE(visuals); i++) { + for (int j = 0; dri2_dpy->driver_configs[j]; j++) { + const EGLint surface_type = EGL_WINDOW_BIT | EGL_PBUFFER_BIT; - for (j = 0; dri2_dpy->driver_configs[j]; j++) { config_attrs[1] = visuals[i].format; config_attrs[3] = visuals[i].format; - dri2_conf = dri2_add_config(dpy, dri2_dpy->driver_configs[j], - count + 1, surface_type, config_attrs, visuals[i].rgba_masks); + struct dri2_egl_config *dri2_conf = + dri2_add_config(dpy, dri2_dpy->driver_configs[j], + count + 1, surface_type, config_attrs, + visuals[i].rgba_masks); if (dri2_conf) { if (dri2_conf->base.ConfigID == count + 1) count++; @@ -1086,7 +1082,7 @@ droid_add_configs_for_visuals(_EGLDriver *drv, _EGLDisplay *dpy) } } - for (i = 0; i < ARRAY_SIZE(format_count); i++) { + for (int i = 0; i < ARRAY_SIZE(format_count); i++) { if (!format_count[i]) { _eglLog(_EGL_DEBUG, "No DRI config supports native format 0x%x", visuals[i].format); -- 2.30.2