hermet pushed a commit to branch master.

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

commit 5ebebe4698bf6618620a841b5af94afe1ea7a6fd
Author: JunsuChoi <jsuya.c...@samsung.com>
Date:   Wed Jan 12 11:32:42 2022 +0900

    Efl.Gfx.Path: Remove unnecessary optimization code for small arc
    
    Summary:
    This condition(optimization) is not a step suggested by arc implementation.
    https://www.w3.org/TR/SVG11/implnote.html#ArcCorrectionOutOfRangeRadii 
(Step2)
    This code is useful if the arc is too small to represent.
    However, scaling often occurs in vectors, which can create unnecessary 
problems.
    
    Test Plan:
    SVG Image
    ```
    <svg viewBox="0 0 50 50" height="50" width="50">
    <path d="M32.41,20.49a.41.41,0,1,1-.41-.42A.41.41,0,0,1,32.41,20.49Z" 
transform="translate(-70, -50) scale(3.3)" fill="#020202"/>
    </svg>
    ```
    image file
    {F4792225}
    result
    {F4792221}
    
    Reviewers: Hermet, raster, kimcinoo
    
    Reviewed By: Hermet
    
    Subscribers: cedric, #reviewers, #committers
    
    Tags: #efl
    
    Differential Revision: https://phab.enlightenment.org/D12316
---
 src/lib/efl/interfaces/efl_gfx_path.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/src/lib/efl/interfaces/efl_gfx_path.c 
b/src/lib/efl/interfaces/efl_gfx_path.c
index 4c20fb45b2..3b531a7eee 100644
--- a/src/lib/efl/interfaces/efl_gfx_path.c
+++ b/src/lib/efl/interfaces/efl_gfx_path.c
@@ -607,11 +607,6 @@ _efl_gfx_path_append_arc_to(Eo *obj, Efl_Gfx_Path_Data *pd,
    // Correction of out-of-range radii, see F6.6.1 (step 2)
    rx = fabs(rx);
    ry = fabs(ry);
-   if ((rx < 0.5) || (ry < 0.5))
-     {
-        _efl_gfx_path_append_line_to(obj, pd, x, y);
-        return;
-     }
 
    angle = angle * M_PI / 180.0;
    cos_phi = cos(angle);

-- 


Reply via email to