From df8321370204434f2a7ca7cf29be973e060781ed Mon Sep 17 00:00:00 2001 From: Emil Velikov Date: Mon, 31 Jul 2017 19:45:27 +0100 Subject: [PATCH] i965: simplify intel_image_format_lookup() Drop the local variable and return directly. Signed-off-by: Emil Velikov Reviewed-by: Matt Turner --- src/mesa/drivers/dri/i965/intel_screen.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/src/mesa/drivers/dri/i965/intel_screen.c b/src/mesa/drivers/dri/i965/intel_screen.c index e9974a35d98..8ec3e03963f 100644 --- a/src/mesa/drivers/dri/i965/intel_screen.c +++ b/src/mesa/drivers/dri/i965/intel_screen.c @@ -349,16 +349,12 @@ intel_image_warn_if_unaligned(__DRIimage *image, const char *func) static const struct intel_image_format * intel_image_format_lookup(int fourcc) { - const struct intel_image_format *f = NULL; - for (unsigned i = 0; i < ARRAY_SIZE(intel_image_formats); i++) { - if (intel_image_formats[i].fourcc == fourcc) { - f = &intel_image_formats[i]; - break; - } + if (intel_image_formats[i].fourcc == fourcc) + return &intel_image_formats[i]; } - return f; + return NULL; } static boolean intel_lookup_fourcc(int dri_format, int *fourcc) -- 2.11.0