4 files changed, 20 insertions, 19 deletions

diff --git a/src/lib/edje/edje_calc.c b/src/lib/edje/edje_calc.c
index 0a9efdcfc9..d752e2360a 100644
--- a/src/lib/edje/edje_calc.c
+++ b/src/lib/edje/edje_calc.c
@@ -3696,13 +3696,13 @@ _edje_svg_recalc_apply(Edje *ed, Edje_Real_Part *ep, Edje_Calc_Params *p3 EINA_U
 {
    int new_svg = -1; //invalid svg
    int w, h;
-   char src_key[20], dest_key[20];
+   char src_key[32], dest_key[32];
    Efl_VG *src_root, *dest_root, *root;
 
    evas_object_geometry_get(ep->object, NULL, NULL, &w, &h);
    if( (w == 0) || (h == 0)) return;
 
-   sprintf(src_key, "edje/vectors/%i", chosen_desc->vg.id);
+   snprintf(src_key, sizeof(src_key), "edje/vectors/%i", chosen_desc->vg.id);
 
    if (ep->param2)
      {
@@ -3719,22 +3719,27 @@ _edje_svg_recalc_apply(Edje *ed, Edje_Real_Part *ep, Edje_Calc_Params *p3 EINA_U
      }
    else
      {
-        sprintf(dest_key, "edje/vectors/%i", new_svg);
+        snprintf(dest_key, sizeof(dest_key), "edje/vectors/%i", new_svg);
 
         efl_file_set(ep->object, ed->file->path, src_key);
         src_root = efl_canvas_vg_root_node_get(ep->object);
+        efl_ref(src_root);
 
         efl_file_set(ep->object, ed->file->path, dest_key);
         dest_root = efl_canvas_vg_root_node_get(ep->object);
+        efl_ref(dest_root);
 
+        // FIXME: root = dup(), root.interpolate(dest).
         root = evas_vg_container_add(NULL);
         evas_vg_node_dup(root, src_root);
 
         if (!evas_vg_node_interpolate(root, src_root, dest_root, pos))
           {
-             ERR(" Can't interpolate check the svg file \n");
+             ERR("Can't interpolate check the svg file");
           }
         efl_canvas_vg_root_node_set(ep->object, root);
+        efl_unref(src_root);
+        efl_unref(dest_root);
      }
 }
 
diff --git a/src/lib/evas/canvas/efl_canvas_vg.c b/src/lib/evas/canvas/efl_canvas_vg.c
index 15d0eb21b9..a869308651 100644
--- a/src/lib/evas/canvas/efl_canvas_vg.c
+++ b/src/lib/evas/canvas/efl_canvas_vg.c
@@ -244,22 +244,18 @@ _efl_canvas_vg_viewbox_align_get(Eo *obj EINA_UNUSED, Efl_Canvas_Vg_Data *pd, do
 EOLIAN static Eina_Bool
 _efl_canvas_vg_efl_file_file_set(Eo *obj, Efl_Canvas_Vg_Data *pd, const char *file, const char *key)
 {
+   Evas_Cache_Vg_Entry *old_entry;
    int w, h;
-   Evas_Cache_Vg_Entry *entry;
 
    if (!file) return EINA_FALSE;
 
+   old_entry = pd->vg_entry;
    evas_object_geometry_get(obj, NULL, NULL, &w, &h);
-   entry = evas_cache_vg_entry_find(file, key, w, h);
-   if (entry != pd->vg_entry)
-     {
-        if (pd->vg_entry)
-          {
-             evas_cache_vg_entry_del(pd->vg_entry);
-          }
-        pd->vg_entry = entry;
-     }
-   evas_object_change(obj, efl_data_scope_get(obj, EFL_CANVAS_OBJECT_CLASS));
+   pd->vg_entry = evas_cache_vg_entry_find(file, key, w, h);
+   if (pd->vg_entry != old_entry)
+     evas_object_change(obj, efl_data_scope_get(obj, EFL_CANVAS_OBJECT_CLASS));
+   evas_cache_vg_entry_del(old_entry);
+
    return EINA_TRUE;
 }
 
@@ -276,7 +272,7 @@ _efl_canvas_vg_efl_file_file_get(Eo *obj EINA_UNUSED, Efl_Canvas_Vg_Data *pd, co
 EOLIAN static Eina_Bool
 _efl_canvas_vg_efl_file_save(const Eo *obj, Efl_Canvas_Vg_Data *pd, const char *file, const char *key, const char *flags)
 {
-   Vg_File_Data tmp;
+   Vg_File_Data tmp = {};
    Vg_File_Data *info = &tmp;
 
    if (pd->vg_entry && pd->vg_entry->file)
@@ -291,8 +287,7 @@ _efl_canvas_vg_efl_file_save(const Eo *obj, Efl_Canvas_Vg_Data *pd, const char *
         info->root = pd->root;
         info->preserve_aspect = EINA_FALSE;
      }
-   evas_vg_save_to_file(info, file, key, flags);
-   return EINA_TRUE;
+   return evas_vg_save_to_file(info, file, key, flags);
 }
 
 static void
@@ -431,7 +426,6 @@ _efl_canvas_vg_render(Evas_Object *eo_obj EINA_UNUSED,
      {
         root = vd->root;
      }
-   //obj->layer->evas->engine.func->ector_begin(output, context,
    obj->layer->evas->engine.func->ector_begin(engine, context,
                                               ector, surface,
                                               vd->engine_data,
diff --git a/src/lib/evas/canvas/efl_canvas_vg.eo b/src/lib/evas/canvas/efl_canvas_vg.eo
index c9fee6dd10..398dd6cf2c 100644
--- a/src/lib/evas/canvas/efl_canvas_vg.eo
+++ b/src/lib/evas/canvas/efl_canvas_vg.eo
@@ -73,6 +73,7 @@ class Efl.Canvas.Vg (Efl.Canvas.Object, Efl.File)
       Efl.Object.constructor;
       Efl.Object.finalize;
       Efl.Object.destructor;
+      // FIXME: Implement mmap only (also fix cache keys)
       Efl.File.file { get; set; }
       Efl.File.save;
    }
diff --git a/src/lib/evas/vg/evas_vg_cache.c b/src/lib/evas/vg/evas_vg_cache.c
index dae36cc290..aa32a6f826 100644
--- a/src/lib/evas/vg/evas_vg_cache.c
+++ b/src/lib/evas/vg/evas_vg_cache.c
@@ -324,5 +324,6 @@ evas_cache_vg_entry_del(Evas_Cache_Vg_Entry *svg_entry)
    if (!svg_entry) return;
 
    svg_entry->ref--;
+   // FIXME implement delete logic (LRU)
 }