From a216b11ce51a199ef48b2a44983a1cc8eca5be2e Mon Sep 17 00:00:00 2001 From: Petr Kulhavy Date: Wed, 17 May 2017 15:58:39 +0200 Subject: [PATCH] pmc: optimize duplicated code in do_set_action() TLV_PRIORITY1 and TLV_PRIORITY2 cases in do_set_action() use the same repeated piece of generic code for setting one-value parameter. Remove the duplicated code and let both cases use the same code. Signed-off-by: Petr Kulhavy --- pmc.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/pmc.c b/pmc.c index 261f2a2..cefa771 100644 --- a/pmc.c +++ b/pmc.c @@ -515,14 +515,6 @@ static void do_set_action(int action, int index, char *str) } switch (code) { case TLV_PRIORITY1: - cnt = sscanf(str, " %*s %*s %hhu", &mtd.val); - if (cnt != 1) { - fprintf(stderr, "%s SET needs 1 value\n", - idtab[index].name); - break; - } - pmc_send_set_action(pmc, code, &mtd, sizeof(mtd)); - break; case TLV_PRIORITY2: cnt = sscanf(str, " %*s %*s %hhu", &mtd.val); if (cnt != 1) {