From 3c59e23b7d3b62dc676b1838555f2641a910ea33 Mon Sep 17 00:00:00 2001 From: Hermet Park Date: Tue, 20 Aug 2019 10:54:32 +0900 Subject: [PATCH] evas vg: code refactoring. try reduce code section size, no logical changes. --- src/lib/evas/canvas/efl_canvas_vg_node.c | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/src/lib/evas/canvas/efl_canvas_vg_node.c b/src/lib/evas/canvas/efl_canvas_vg_node.c index 895706c71a..95dcdc5152 100644 --- a/src/lib/evas/canvas/efl_canvas_vg_node.c +++ b/src/lib/evas/canvas/efl_canvas_vg_node.c @@ -141,30 +141,33 @@ _efl_canvas_vg_node_efl_gfx_color_color_set(Eo *obj, Efl_Canvas_Vg_Node_Data *pd, int r, int g, int b, int a) { - if (r > 255) r = 255; + Eina_Bool perr = EINA_FALSE; + + //Exception Handling. if (r < 0) r = 0; - if (g > 255) g = 255; if (g < 0) g = 0; - if (b > 255) b = 255; if (b < 0) b = 0; if (a > 255) a = 255; - if (a < 0) a = 0; + else if (a < 0) a = 0; + if (r > a) { r = a; - ERR("Evas only handles pre multiplied colors!"); + perr = EINA_TRUE; } if (g > a) { g = a; - ERR("Evas only handles pre multiplied colors!"); + perr = EINA_TRUE; } if (b > a) { b = a; - ERR("Evas only handles pre multiplied colors!"); + perr = EINA_TRUE; } + if (perr) ERR("Evas only handles pre-multiplied color!"); + pd->r = r; pd->g = g; pd->b = b;