[EGIT] [core/efl] master 01/01: vg_load_svg: Supports case when only rx or ry is declared

[JunsuChoi]

hermet pushed a commit to branch master.

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

commit d9c3b3f1b9fa87451aa39da1654a32d3023d9e75
Author: JunsuChoi <jsuya.c...@samsung.com>
Date:   Thu Sep 10 14:59:29 2020 +0900

    vg_load_svg: Supports case when only rx or ry is declared
    
    Summary:
    In relation to the declaration of rx and ry attribute of rect, the 
following three cases occur.
    rx="10" (or ry="10"
    rx="10" ry = "0" (or rx="0" ry = "10")
    rx="10" ry = "10"
    To cover these case, we check the rx and ry declarations.
    
    Test Plan: N/A
    
    Reviewers: Hermet, smohanty
    
    Subscribers: kimcinoo, herb, cedric, #reviewers, #committers
    
    Tags: #efl
    
    Differential Revision: https://phab.enlightenment.org/D12137
---
 src/modules/evas/vg_loaders/svg/evas_vg_load_svg.c | 15 +++++++++++++--
 src/static_libs/vg_common/vg_common.h              |  2 ++
 2 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/src/modules/evas/vg_loaders/svg/evas_vg_load_svg.c 
b/src/modules/evas/vg_loaders/svg/evas_vg_load_svg.c
index 6498f1ab10..d4ea28659c 100644
--- a/src/modules/evas/vg_loaders/svg/evas_vg_load_svg.c
+++ b/src/modules/evas/vg_loaders/svg/evas_vg_load_svg.c
@@ -1381,6 +1381,13 @@ _attr_parse_rect_node(void *data, const char *key, const 
char *value)
      if (rect_tags[i].sz - 1 == sz && !strncmp(rect_tags[i].tag, key, sz))
        {
           *((double*) (array + rect_tags[i].offset)) = 
_to_double(loader->svg_parse, value, rect_tags[i].type);
+
+          //Case if only rx or ry is declared
+          if (!strncmp(rect_tags[i].tag, "rx", sz)) rect->has_rx = EINA_TRUE;
+          if (!strncmp(rect_tags[i].tag, "ry", sz)) rect->has_ry = EINA_TRUE;
+
+          if (!EINA_DBL_EQ(rect->rx, 0) && EINA_DBL_EQ(rect->ry, 0) && 
rect->has_rx && !rect->has_ry) rect->ry = rect->rx;
+          if (!EINA_DBL_EQ(rect->ry, 0) && EINA_DBL_EQ(rect->rx, 0) && 
!rect->has_rx && rect->has_ry) rect->rx = rect->ry;
           return EINA_TRUE;
        }
 
@@ -1397,8 +1404,6 @@ _attr_parse_rect_node(void *data, const char *key, const 
char *value)
         _parse_style_attr(loader, key, value);
      }
 
-   if (!EINA_DBL_EQ(rect->rx, 0) && EINA_DBL_EQ(rect->ry, 0)) rect->ry = 
rect->rx;
-   if (!EINA_DBL_EQ(rect->ry, 0) && EINA_DBL_EQ(rect->rx, 0)) rect->rx = 
rect->ry;
 
    return EINA_TRUE;
 }
@@ -1408,6 +1413,10 @@ _create_rect_node(Evas_SVG_Loader *loader, Svg_Node 
*parent, const char *buf, un
 {
    loader->svg_parse->node = _create_node(parent, SVG_NODE_RECT);
 
+   if (loader->svg_parse->node) {
+        loader->svg_parse->node->node.rect.has_rx = 
loader->svg_parse->node->node.rect.has_ry = EINA_FALSE;
+   }
+
    eina_simple_xml_attributes_parse(buf, buflen,
                                     _attr_parse_rect_node, loader);
    return loader->svg_parse->node;
@@ -1601,6 +1610,8 @@ _copy_attribute(Svg_Node *to, Svg_Node *from)
            to->node.rect.h = from->node.rect.h;
            to->node.rect.rx = from->node.rect.rx;
            to->node.rect.ry = from->node.rect.ry;
+           to->node.rect.has_rx = from->node.rect.has_rx;
+           to->node.rect.has_ry = from->node.rect.has_ry;
            break;
         case SVG_NODE_LINE:
            to->node.line.x1 = from->node.line.x1;
diff --git a/src/static_libs/vg_common/vg_common.h 
b/src/static_libs/vg_common/vg_common.h
index 83f9224e45..43e36df4d8 100644
--- a/src/static_libs/vg_common/vg_common.h
+++ b/src/static_libs/vg_common/vg_common.h
@@ -126,6 +126,8 @@ struct _Svg_Rect_Node
    double h;
    double rx;
    double ry;
+   Eina_Bool has_rx;
+   Eina_Bool has_ry;
 };
 
 struct _Svg_Line_Node

--