From fe60e94007280b2a71ef900a4fd3204fc8e26a2c Mon Sep 17 00:00:00 2001 From: JunsuChoi Date: Fri, 18 Jun 2021 10:24:58 +0100 Subject: [PATCH] evas_vg_load_svg: Fix color parsing Summary: Remove unnecessary point movement in rgb(255, 255, 255) case in svg parsing. In svg parsing, move the pointer by 'rgb(' before calling _color_parser() in the rgb(255, 255, 255) case. In function, string pointer moved unnecessary, so parsing is incorrect. Therefore, remove unnecessary point movement. Test Plan: svg sample code ``` ``` before {F4504779} after {F4504778} Reviewers: Hermet Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D12284 --- src/modules/evas/vg_loaders/svg/evas_vg_load_svg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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 e8c46ceb1f..1d93741ba3 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 @@ -337,7 +337,7 @@ _color_parser(const char *value, char **end) { double r; - r = eina_convert_strtod_c(value + 4, end); + r = eina_convert_strtod_c(value, end); *end = _skip_space(*end, NULL); if (**end == '%') r = 255 * r / 100;