loaders svg: code refactoring
authorHermet Park <chuneon.park@samsung.com>
Tue, 20 Apr 2021 11:32:22 +0000 (20:32 +0900)
committerJunsuChoi <jsuya.choi@samsung.com>
Thu, 22 Apr 2021 00:38:54 +0000 (09:38 +0900)
split functions as its purpose.

src/loaders/svg/meson.build
src/loaders/svg/tvgSvgLoader.cpp
src/loaders/svg/tvgSvgPath.cpp
src/loaders/svg/tvgSvgUtil.cpp [new file with mode: 0644]
src/loaders/svg/tvgSvgUtil.h [new file with mode: 0644]

index fd75e44582ad335b0f62566cf208276dfb89a391..d6d8cec64d77c6ac67f8e794fe74ba06b79cfba6 100644 (file)
@@ -3,10 +3,12 @@ source_file = [
    'tvgSvgLoaderCommon.h',
    'tvgSvgPath.h',
    'tvgSvgSceneBuilder.h',
+   'tvgSvgUtil.h',
    'tvgXmlParser.h',
    'tvgSvgLoader.cpp',
    'tvgSvgPath.cpp',
    'tvgSvgSceneBuilder.cpp',
+   'tvgSvgUtil.cpp',
    'tvgXmlParser.cpp'
 ]
 
index eb73033061fc50994e47821a9fadf4f8aeb27121..17df996fbbf6e5f49787bcce2714e4fde9b9c85c 100644 (file)
@@ -26,6 +26,7 @@
 #include "tvgXmlParser.h"
 #include "tvgSvgLoader.h"
 #include "tvgSvgSceneBuilder.h"
+#include "tvgSvgUtil.h"
 
 /************************************************************************/
 /* Internal Class Implementation                                        */
@@ -35,206 +36,6 @@ typedef SvgNode* (*FactoryMethod)(SvgLoaderData* loader, SvgNode* parent, const
 typedef SvgStyleGradient* (*GradientFactoryMethod)(SvgLoaderData* loader, const char* buf, unsigned bufLength);
 
 
-/*
- * https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/strtof-strtof-l-wcstof-wcstof-l?view=msvc-160
- *
- * src should be one of the following form :
- *
- * [whitespace] [sign] {digits [radix digits] | radix digits} [{e | E} [sign] digits]
- * [whitespace] [sign] {INF | INFINITY}
- * [whitespace] [sign] NAN [sequence]
- *
- * No hexadecimal form supported
- * no sequence supported after NAN
- */
-static bool floatExact(float a, float b)
-{
-    return memcmp(&a, &b, sizeof (float)) == 0;
-}
-
-
-float customStrtof(const char *nPtr, char **endPtr)
-{
-    const char *iter;
-    const char *a;
-    float val;
-    unsigned long long integerPart;
-    int minus;
-
-    if (endPtr) *endPtr = (char*)nPtr;
-    if (!nPtr) return 0.0f;
-
-    a = iter = nPtr;
-
-    //ignore leading whitespaces
-    while (isspace(*iter)) iter++;
-
-    //signed or not
-    minus = 1;
-    if (*iter == '-')
-    {
-        minus = -1;
-        iter++;
-    }
-    else if (*iter == '+') iter++;
-
-    if (tolower(*iter) == 'i')
-    {
-        if ((tolower(*(iter + 1)) == 'n') && (tolower(*(iter + 2)) == 'f'))
-        {
-            iter += 3;
-        }
-        else goto on_error;
-
-        if (tolower(*(iter + 3)) == 'i')
-        {
-            if ((tolower(*(iter + 4)) == 'n') &&
-                (tolower(*(iter + 5)) == 'i') &&
-                (tolower(*(iter + 6)) == 't') &&
-                (tolower(*(iter + 7)) == 'y'))
-            {
-               iter += 5;
-            }
-            else goto on_error;
-         }
-         if (endPtr) *endPtr = (char *)iter;
-         return (minus == -1) ? -INFINITY : INFINITY;
-    }
-
-    if (tolower(*iter) == 'n')
-    {
-         if ((tolower(*(iter + 1)) == 'a') && (tolower(*(iter + 2)) == 'n')) iter += 3;
-         else goto on_error;
-
-         if (endPtr) *endPtr = (char *)iter;
-         return (minus == -1) ? -NAN : NAN;
-    }
-
-    integerPart = 0;
-
-    //(optional) integer part before dot
-    if (isdigit(*iter))
-    {
-        for (; isdigit(*iter); iter++) integerPart = integerPart * 10ULL + (unsigned long long)(*iter - '0');
-
-        a = iter;
-    }
-    else if (*iter != '.')
-    {
-        val = 0.0;
-        goto on_success;
-    }
-
-    val = (float)integerPart;
-
-    //(optional) decimal part after dot
-    if (*iter == '.')
-    {
-        unsigned long long decimalPart;
-        unsigned long long pow10;
-        int count;
-
-        iter++;
-
-        decimalPart = 0;
-        count = 0;
-        pow10 = 1;
-
-        if (isdigit(*iter))
-        {
-            for (; isdigit(*iter); iter++, count++)
-            {
-                if (count < 19)
-                {
-                    decimalPart = decimalPart * 10ULL +  + (unsigned long long)(*iter - '0');
-                    pow10 *= 10ULL;
-                }
-            }
-        }
-        val += (float)decimalPart / (float)pow10;
-        a = iter;
-    }
-
-    //(optional) exponent
-    if ((*iter == 'e') || (*iter == 'E'))
-    {
-        float scale = 1.0f;
-        unsigned int expo_part;
-        int minus_e;
-
-        iter++;
-
-        //signed or not
-        minus_e = 1;
-        if (*iter == '-')
-        {
-            minus_e = -1;
-            iter++;
-        }
-        else if (*iter == '+') iter++;
-
-        //exponential part
-        expo_part = 0;
-        if (isdigit(*iter))
-        {
-            while (*iter == 0) iter++;
-
-            for (; isdigit(*iter); iter++)
-            {
-                expo_part = expo_part * 10U + (unsigned int)(*iter - '0');
-            }
-        }
-        else if (!isdigit(*(a - 1)))
-        {
-            a = nPtr;
-            goto on_success;
-        }
-        else if (*iter == 0) goto on_success;
-
-        if ((floatExact(val, 2.2250738585072011)) && ((minus_e * (int)expo_part) == -308))
-        {
-            val *= 1.0e-308;
-            a = iter;
-            errno = ERANGE;
-            goto on_success;
-        }
-
-        if ((floatExact(val, 2.2250738585072012)) && ((minus_e * (int)expo_part) <= -308))
-        {
-            val *= 1.0e-308;
-            a = iter;
-            goto on_success;
-        }
-
-        a = iter;
-
-        while (expo_part >= 8U)
-        {
-            scale *= 1E8;
-            expo_part -= 8U;
-        }
-        while (expo_part > 0U)
-        {
-            scale *= 10.0f;
-            expo_part--;
-        }
-
-        val = (minus_e == -1) ? (val / scale) : (val * scale);
-    }
-    else if ((iter > nPtr) && !isdigit(*(iter - 1)))
-    {
-        a = nPtr;
-        goto on_success;
-    }
-
-on_success:
-    if (endPtr) *endPtr = (char *)a;
-    return minus * val;
-
-on_error:
-    if (endPtr) *endPtr = (char *)nPtr;
-    return 0.0f;
-}
 
 
 static char* _skipSpace(const char* str, const char* end)
@@ -266,7 +67,7 @@ static bool _parseNumber(const char** content, float* number)
 {
     char* end = nullptr;
 
-    *number = customStrtof(*content, &end);
+    *number = svgUtilStrtof(*content, &end);
     //If the start of string is not number
     if ((*content) == end) return false;
     //Skip comma if any
@@ -283,7 +84,7 @@ static bool _parseNumber(const char** content, float* number)
  */
 static float _toFloat(const SvgParser* svgParse, const char* str, SvgParserLengthType type)
 {
-    float parsedValue = customStrtof(str, nullptr);
+    float parsedValue = svgUtilStrtof(str, nullptr);
 
     if (strstr(str, "cm")) parsedValue = parsedValue * 35.43307;
     else if (strstr(str, "mm")) parsedValue = parsedValue * 3.543307;
@@ -312,7 +113,7 @@ static float _gradientToFloat(const SvgParser* svgParse, const char* str, SvgPar
 {
     char* end = nullptr;
 
-    float parsedValue = customStrtof(str, &end);
+    float parsedValue = svgUtilStrtof(str, &end);
     float max = 1;
 
     /**
@@ -343,7 +144,7 @@ static float _toOffset(const char* str)
 {
     char* end = nullptr;
 
-    float parsedValue = customStrtof(str, &end);
+    float parsedValue = svgUtilStrtof(str, &end);
 
     if (strstr(str, "%")) parsedValue = parsedValue / 100.0;
 
@@ -355,7 +156,7 @@ static int _toOpacity(const char* str)
 {
     char* end = nullptr;
     int a = 0;
-    float opacity = customStrtof(str, &end);
+    float opacity = svgUtilStrtof(str, &end);
 
     if (end && (*end == '\0')) a = lrint(opacity * 255);
     return a;
@@ -442,7 +243,7 @@ _parseDashArray(const char *str, SvgDash* dash)
     while (*str) {
         // skip white space, comma
         str = _skipComma(str);
-        (*dash).array.push(customStrtof(str, &end));
+        (*dash).array.push(svgUtilStrtof(str, &end));
         str = _skipComma(end);
     }
     //If dash array size is 1, it means that dash and gap size are the same.
@@ -476,7 +277,7 @@ static unsigned char _parserColor(const char* value, char** end)
 {
     float r;
 
-    r = customStrtof(value + 4, end);
+    r = svgUtilStrtof(value + 4, end);
     *end = _skipSpace(*end, nullptr);
     if (**end == '%') r = 255 * r / 100;
     *end = _skipSpace(*end, nullptr);
@@ -713,7 +514,7 @@ static char* _parseNumbersArray(char* str, float* points, int* ptCount, int len)
 
     str = _skipSpace(str, nullptr);
     while ((count < len) && (isdigit(*str) || *str == '-' || *str == '+' || *str == '.')) {
-        points[count++] = customStrtof(str, &end);
+        points[count++] = svgUtilStrtof(str, &end);
         str = end;
         str = _skipSpace(str, nullptr);
         if (*str == ',') ++str;
@@ -900,7 +701,7 @@ static float _parseLength(const char* str, SvgLengthType* type)
     for (unsigned int i = 0; i < sizeof(lengthTags) / sizeof(lengthTags[0]); i++) {
         if (lengthTags[i].sz - 1 == sz && !strncmp(lengthTags[i].tag, str, sz)) *type = lengthTags[i].type;
     }
-    value = customStrtof(str, nullptr);
+    value = svgUtilStrtof(str, nullptr);
     return value;
 }
 
index 331a833bf303477e9f0b74113337f1a12906d2e6..6f9bb3d85ce0b4d9236598218da20c582ba82c70 100644 (file)
@@ -24,6 +24,7 @@
 #include <ctype.h>
 #include "tvgSvgLoaderCommon.h"
 #include "tvgSvgPath.h"
+#include "tvgSvgUtil.h"
 
 /************************************************************************/
 /* Internal Class Implementation                                        */
@@ -42,7 +43,7 @@ static char* _skipComma(const char* content)
 static bool _parseNumber(char** content, float* number)
 {
     char* end = NULL;
-    *number = customStrtof(*content, &end);
+    *number = svgUtilStrtof(*content, &end);
     //If the start of string is not number
     if ((*content) == end) return false;
     //Skip comma if any
diff --git a/src/loaders/svg/tvgSvgUtil.cpp b/src/loaders/svg/tvgSvgUtil.cpp
new file mode 100644 (file)
index 0000000..6c95598
--- /dev/null
@@ -0,0 +1,238 @@
+/*
+ * Copyright (c) 2020-2021 Samsung Electronics Co., Ltd. All rights reserved.
+
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+
+ * The above copyright notice and this permission notice shall be included in all
+ * copies or substantial portions of the Software.
+
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+#include <math.h>
+#include <memory.h>
+#include <ctype.h>
+#include <errno.h>
+#include "tvgSvgUtil.h"
+
+
+/************************************************************************/
+/* Internal Class Implementation                                        */
+/************************************************************************/
+
+static inline bool _floatExact(float a, float b)
+{
+    return memcmp(&a, &b, sizeof (float)) == 0;
+}
+
+
+/************************************************************************/
+/* External Class Implementation                                        */
+/************************************************************************/
+
+
+/*
+ * https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/strtof-strtof-l-wcstof-wcstof-l?view=msvc-160
+ *
+ * src should be one of the following form :
+ *
+ * [whitespace] [sign] {digits [radix digits] | radix digits} [{e | E} [sign] digits]
+ * [whitespace] [sign] {INF | INFINITY}
+ * [whitespace] [sign] NAN [sequence]
+ *
+ * No hexadecimal form supported
+ * no sequence supported after NAN
+ */
+float svgUtilStrtof(const char *nPtr, char **endPtr)
+{
+    const char *iter;
+    const char *a;
+    float val;
+    unsigned long long integerPart;
+    int minus;
+
+    if (endPtr) *endPtr = (char*)nPtr;
+    if (!nPtr) return 0.0f;
+
+    a = iter = nPtr;
+
+    //ignore leading whitespaces
+    while (isspace(*iter)) iter++;
+
+    //signed or not
+    minus = 1;
+    if (*iter == '-')
+    {
+        minus = -1;
+        iter++;
+    }
+    else if (*iter == '+') iter++;
+
+    if (tolower(*iter) == 'i')
+    {
+        if ((tolower(*(iter + 1)) == 'n') && (tolower(*(iter + 2)) == 'f'))
+        {
+            iter += 3;
+        }
+        else goto on_error;
+
+        if (tolower(*(iter + 3)) == 'i')
+        {
+            if ((tolower(*(iter + 4)) == 'n') &&
+                (tolower(*(iter + 5)) == 'i') &&
+                (tolower(*(iter + 6)) == 't') &&
+                (tolower(*(iter + 7)) == 'y'))
+            {
+               iter += 5;
+            }
+            else goto on_error;
+         }
+         if (endPtr) *endPtr = (char *)iter;
+         return (minus == -1) ? -INFINITY : INFINITY;
+    }
+
+    if (tolower(*iter) == 'n')
+    {
+         if ((tolower(*(iter + 1)) == 'a') && (tolower(*(iter + 2)) == 'n')) iter += 3;
+         else goto on_error;
+
+         if (endPtr) *endPtr = (char *)iter;
+         return (minus == -1) ? -NAN : NAN;
+    }
+
+    integerPart = 0;
+
+    //(optional) integer part before dot
+    if (isdigit(*iter))
+    {
+        for (; isdigit(*iter); iter++) integerPart = integerPart * 10ULL + (unsigned long long)(*iter - '0');
+
+        a = iter;
+    }
+    else if (*iter != '.')
+    {
+        val = 0.0;
+        goto on_success;
+    }
+
+    val = (float)integerPart;
+
+    //(optional) decimal part after dot
+    if (*iter == '.')
+    {
+        unsigned long long decimalPart;
+        unsigned long long pow10;
+        int count;
+
+        iter++;
+
+        decimalPart = 0;
+        count = 0;
+        pow10 = 1;
+
+        if (isdigit(*iter))
+        {
+            for (; isdigit(*iter); iter++, count++)
+            {
+                if (count < 19)
+                {
+                    decimalPart = decimalPart * 10ULL +  + (unsigned long long)(*iter - '0');
+                    pow10 *= 10ULL;
+                }
+            }
+        }
+        val += (float)decimalPart / (float)pow10;
+        a = iter;
+    }
+
+    //(optional) exponent
+    if ((*iter == 'e') || (*iter == 'E'))
+    {
+        float scale = 1.0f;
+        unsigned int expo_part;
+        int minus_e;
+
+        iter++;
+
+        //signed or not
+        minus_e = 1;
+        if (*iter == '-')
+        {
+            minus_e = -1;
+            iter++;
+        }
+        else if (*iter == '+') iter++;
+
+        //exponential part
+        expo_part = 0;
+        if (isdigit(*iter))
+        {
+            while (*iter == 0) iter++;
+
+            for (; isdigit(*iter); iter++)
+            {
+                expo_part = expo_part * 10U + (unsigned int)(*iter - '0');
+            }
+        }
+        else if (!isdigit(*(a - 1)))
+        {
+            a = nPtr;
+            goto on_success;
+        }
+        else if (*iter == 0) goto on_success;
+
+        if ((_floatExact(val, 2.2250738585072011)) && ((minus_e * (int)expo_part) == -308))
+        {
+            val *= 1.0e-308;
+            a = iter;
+            errno = ERANGE;
+            goto on_success;
+        }
+
+        if ((_floatExact(val, 2.2250738585072012)) && ((minus_e * (int)expo_part) <= -308))
+        {
+            val *= 1.0e-308;
+            a = iter;
+            goto on_success;
+        }
+
+        a = iter;
+
+        while (expo_part >= 8U)
+        {
+            scale *= 1E8;
+            expo_part -= 8U;
+        }
+        while (expo_part > 0U)
+        {
+            scale *= 10.0f;
+            expo_part--;
+        }
+
+        val = (minus_e == -1) ? (val / scale) : (val * scale);
+    }
+    else if ((iter > nPtr) && !isdigit(*(iter - 1)))
+    {
+        a = nPtr;
+        goto on_success;
+    }
+
+on_success:
+    if (endPtr) *endPtr = (char *)a;
+    return minus * val;
+
+on_error:
+    if (endPtr) *endPtr = (char *)nPtr;
+    return 0.0f;
+}
diff --git a/src/loaders/svg/tvgSvgUtil.h b/src/loaders/svg/tvgSvgUtil.h
new file mode 100644 (file)
index 0000000..1f2d9ec
--- /dev/null
@@ -0,0 +1,28 @@
+/*
+ * Copyright (c) 2020-2021 Samsung Electronics Co., Ltd. All rights reserved.
+
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+
+ * The above copyright notice and this permission notice shall be included in all
+ * copies or substantial portions of the Software.
+
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+#ifndef _TVG_SVG_UTIL_H_
+#define _TVG_SVG_UTIL_H_
+
+float svgUtilStrtof(const char *nPtr, char **endPtr);
+
+#endif //_TVG_SVG_UTIL_H_