canvas vg_container: code refactoring. 51/248951/1
authorHermet Park <chuneon.park@samsung.com>
Thu, 3 Dec 2020 11:54:10 +0000 (20:54 +0900)
committerHermet Park <chuneon.park@samsung.com>
Thu, 3 Dec 2020 11:54:10 +0000 (20:54 +0900)
we can make logic simpler.

Change-Id: Ie688940165299ee8fbdd85afcf08277642df15c8

src/lib/evas/canvas/efl_canvas_vg_container.c
src/lib/evas/canvas/efl_canvas_vg_container.eo

index 1004ea1..2082010 100644 (file)
@@ -394,42 +394,16 @@ _efl_canvas_vg_container_efl_gfx_path_bounds_get(const Eo *obj EINA_UNUSED,
 }
 
 static void
-_efl_canvas_vg_container_efl_gfx_color_color_set(Eo *obj EINA_UNUSED,
-#ifdef HAVE_THORVG
-                                                 Efl_Canvas_Vg_Container_Data *pd,
-#else
-                                                 Efl_Canvas_Vg_Container_Data *pd EINA_UNUSED,
-#endif
-                                                 int r EINA_UNUSED,
-                                                 int g EINA_UNUSED,
-                                                 int b EINA_UNUSED,
-                                                 int a)
+_efl_canvas_vg_container_efl_gfx_color_color_set(Eo *obj, Efl_Canvas_Vg_Container_Data *pd, int r, int g, int b, int a)
 {
-#ifdef HAVE_THORVG
-   if (pd && pd->scene)
-     tvg_paint_set_opacity(pd->scene, a);
-   efl_canvas_vg_node_change(obj);
-#endif
-}
+   efl_gfx_color_set(efl_super(obj, MY_CLASS), r, g, b, a);
 
-static void
-_efl_canvas_vg_container_efl_gfx_color_color_get(const Eo *obj EINA_UNUSED,
 #ifdef HAVE_THORVG
-                                                 Efl_Canvas_Vg_Container_Data *pd,
-#else
-                                                 Efl_Canvas_Vg_Container_Data *pd EINA_UNUSED,
-#endif
-                                                 int* r EINA_UNUSED,
-                                                 int* g EINA_UNUSED,
-                                                 int* b EINA_UNUSED,
-                                                 int* a)
-{
-#ifdef HAVE_THORVG
-   if (pd && pd->scene)
-     tvg_paint_get_opacity(pd->scene, (uint8_t*)a);
+   if (pd->scene) tvg_paint_set_opacity(pd->scene, a);
 #endif
 }
 
+
 static Efl_VG *
 _efl_canvas_vg_container_child_get(Eo *obj EINA_UNUSED, Efl_Canvas_Vg_Container_Data *pd, const char *name)
 {
index 1133e82..70305d7 100644 (file)
@@ -24,7 +24,7 @@ class @beta Efl.Canvas.Vg.Container extends Efl.Canvas.Vg.Node
       Efl.Object.parent { set; }
       Efl.Gfx.Path.bounds_get;
       Efl.Gfx.Path.interpolate;
-      Efl.Gfx.Color.color { get; set; }
+      Efl.Gfx.Color.color { set; }
       Efl.Duplicate.duplicate;
       Efl.Canvas.Vg.Node.comp_method { set; }
    }