Merge 642060a into 903c6b6

Author: vitstn

ydb/library/yql/public/decimal/ut/yql_decimal_ut.cpp

@@ -215,6 +215,8 @@ Y_UNIT_TEST_SUITE(TYqlDecimalTest) {
         UNIT_ASSERT(FromStringEx("-1e-99", 10, 2) == 0);
         UNIT_ASSERT(FromStringEx("-510e-3", 1, 0) == -1);
         UNIT_ASSERT(FromStringEx("+99E3", 5, 0) == 99000);
+        UNIT_ASSERT(FromStringEx("2.1E-130", 35, 2) == 0);
+        UNIT_ASSERT(FromStringEx("2.1E0", 35, 2) == 210);
     }
 
     Y_UNIT_TEST(TestFormStringExInvalidValues) {
@@ -225,7 +227,6 @@ Y_UNIT_TEST_SUITE(TYqlDecimalTest) {
 
         UNIT_ASSERT(IsError(FromStringEx("E2", 35, 15))); // empty
         UNIT_ASSERT(IsError(FromStringEx("E2E4", 35, 15))); // empty
-        UNIT_ASSERT(IsError(FromStringEx("12E0", 35, 15))); // zero isn't avail
         UNIT_ASSERT(IsError(FromStringEx("NANE5", 35, 15))); // nan with exp
     }
 

ydb/library/yql/public/decimal/yql_decimal.cpp

@@ -3,13 +3,18 @@
 #include <cstring>
 #include <ostream>
 #include <string>
+#include <charconv>
 
 namespace NYql {
 namespace NDecimal {
 
 static const TUint128 Ten(10U);
 
 TUint128 GetDivider(ui8 scale) {
+    if (scale > MaxPrecision) {
+        return Inf();
+    }
+
     TUint128 d(1U);
     while (scale--)
         d *= Ten;
@@ -215,8 +220,13 @@ TInt128 FromStringEx(const TStringBuf& str, ui8 precision, ui8 scale) {
             if (!len)
                 return Err();
 
-            const auto exp = std::atoi(++ptr);
-            if (!exp)
+            ++ptr;
+            if (ptr != s + str.size() && *ptr == '+') {
+                ++ptr;
+            }
+
+            int exp;
+            if (std::from_chars(ptr, s + str.size(), exp).ec != std::errc())
                 return Err();
 
             const int p = precision, s = int(scale) + exp;