[EGIT] [core/efl] master 01/01: evas vg: code refactoring.

[Hermet Park]

hermet pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=3c59e23b7d3b62dc676b1838555f2641a910ea33

commit 3c59e23b7d3b62dc676b1838555f2641a910ea33
Author: Hermet Park <hermetp...@gmail.com>
Date:   Tue Aug 20 10:54:32 2019 +0900

    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;

--