From de5dac1a097aa8aa0a32e7b4f5d221a17ceed4d5 Mon Sep 17 00:00:00 2001 From: Bruno Dilly Date: Thu, 6 Dec 2012 16:59:07 +0000 Subject: [PATCH] edje: use switch instead of N if-else with same commands SVN revision: 80373 --- legacy/edje/src/bin/edje_cc_handlers.c | 44 ++++++++++++-------------- 1 file changed, 20 insertions(+), 24 deletions(-) diff --git a/legacy/edje/src/bin/edje_cc_handlers.c b/legacy/edje/src/bin/edje_cc_handlers.c index 3c72d8781a..9a8fa21604 100644 --- a/legacy/edje/src/bin/edje_cc_handlers.c +++ b/legacy/edje/src/bin/edje_cc_handlers.c @@ -8386,32 +8386,28 @@ st_collections_group_programs_program_target(void) memcpy(copy, name, strlen(name) + 1); - if (ep->action == EDJE_ACTION_TYPE_STATE_SET) - data_queue_part_lookup(pc, name, &(et->id)); - else if (ep->action == EDJE_ACTION_TYPE_ACTION_STOP) - data_queue_program_lookup(pc, name, &(et->id)); - else if (ep->action == EDJE_ACTION_TYPE_DRAG_VAL_SET) - data_queue_part_lookup(pc, name, &(et->id)); - else if (ep->action == EDJE_ACTION_TYPE_DRAG_VAL_STEP) - data_queue_part_lookup(pc, name, &(et->id)); - else if (ep->action == EDJE_ACTION_TYPE_DRAG_VAL_PAGE) - data_queue_part_lookup(pc, name, &(et->id)); - else if (ep->action == EDJE_ACTION_TYPE_FOCUS_SET) - data_queue_part_lookup(pc, name, &(et->id)); - else if (ep->action == EDJE_ACTION_TYPE_FOCUS_OBJECT) - data_queue_part_lookup(pc, name, &(et->id)); + switch (ep->action) + { + case EDJE_ACTION_TYPE_ACTION_STOP: + data_queue_program_lookup(pc, name, &(et->id)); + break; + case EDJE_ACTION_TYPE_STATE_SET: + case EDJE_ACTION_TYPE_DRAG_VAL_SET: + case EDJE_ACTION_TYPE_DRAG_VAL_STEP: + case EDJE_ACTION_TYPE_DRAG_VAL_PAGE: + case EDJE_ACTION_TYPE_FOCUS_SET: + case EDJE_ACTION_TYPE_FOCUS_OBJECT: #ifdef HAVE_EPHYSICS - else if (ep->action == EDJE_ACTION_TYPE_PHYSICS_IMPULSE) - data_queue_part_lookup(pc, name, &(et->id)); - else if (ep->action == EDJE_ACTION_TYPE_PHYSICS_TORQUE_IMPULSE) - data_queue_part_lookup(pc, name, &(et->id)); + case EDJE_ACTION_TYPE_PHYSICS_IMPULSE: + case EDJE_ACTION_TYPE_PHYSICS_TORQUE_IMPULSE: #endif - else - { - ERR("parse error %s:%i. target may only be used after action", - file_in, line - 1); - exit(-1); - } + data_queue_part_lookup(pc, name, &(et->id)); + break; + default: + ERR("parse error %s:%i. target may only be used after action", + file_in, line - 1); + exit(-1); + } free(name); } }