From 7832e02b2de7573d0ff4455bb69d9bcc134195a5 Mon Sep 17 00:00:00 2001 From: Cedric BAIL Date: Mon, 5 Jun 2017 12:01:17 -0700 Subject: [PATCH] evas: TGA does use switch case statement fallthrough to factorize code. --- src/modules/evas/image_loaders/tga/evas_image_load_tga.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/src/modules/evas/image_loaders/tga/evas_image_load_tga.c b/src/modules/evas/image_loaders/tga/evas_image_load_tga.c index 74d2c62..e2f7896 100644 --- a/src/modules/evas/image_loaders/tga/evas_image_load_tga.c +++ b/src/modules/evas/image_loaders/tga/evas_image_load_tga.c @@ -292,18 +292,25 @@ evas_image_load_file_data_tga(void *loader_data, { case 1: a = (a << 7) | (a << 6) | (a << 5) | (a << 4) | (a << 3) | (a << 2) | (a << 1) | (a); + EINA_FALLTHROUGH; case 2: a = (a << 6) | (a << 4) | (a << 2) | (a); + EINA_FALLTHROUGH; case 3: a = (a << 5) | (a << 2) | (a >> 1); + EINA_FALLTHROUGH; case 4: a = (a << 4) | (a); + EINA_FALLTHROUGH; case 5: a = (a << 3) | (a >> 2); + EINA_FALLTHROUGH; case 6: a = (a << 2) | (a >> 4); + EINA_FALLTHROUGH; case 7: a = (a << 1) | (a >> 6); + EINA_FALLTHROUGH; default: break; } @@ -382,18 +389,25 @@ evas_image_load_file_data_tga(void *loader_data, { case 1: a = (a << 7) | (a << 6) | (a << 5) | (a << 4) | (a << 3) | (a << 2) | (a << 1) | (a); + EINA_FALLTHROUGH; case 2: a = (a << 6) | (a << 4) | (a << 2) | (a); + EINA_FALLTHROUGH; case 3: a = (a << 5) | (a << 2) | (a >> 1); + EINA_FALLTHROUGH; case 4: a = (a << 4) | (a); + EINA_FALLTHROUGH; case 5: a = (a << 3) | (a >> 2); + EINA_FALLTHROUGH; case 6: a = (a << 2) | (a >> 4); + EINA_FALLTHROUGH; case 7: a = (a << 1) | (a >> 6); + EINA_FALLTHROUGH; default: break; } -- 2.7.4