github-mirrors/thirdparty-ArduinoJson
1
0
Fork 0
mirror of https://github.com/eledio-devices/thirdparty-ArduinoJson.git synced 2025-10-31 08:42:39 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix 9.22337e18 is outside the range of representable values of 'long'

Browse Source
This commit is contained in:
Benoit Blanchon
2022-04-05 13:02:24 +02:00
parent 67b6797b6d
commit c65692ab31
5 changed files with 139 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
CHANGELOG.md
View File

@@ -1,6 +1,11 @@
ArduinoJson: change log
=======================
HEAD
----
* Fix `9.22337e+18 is outside the range of representable values of type 'long'`
v6.19.4 (2022-04-05)
-------

1
extras/tests/MsgPackSerializer/serializeVariant.cpp
View File

@@ -109,6 +109,7 @@ TEST_CASE("serialize MsgPack value") {
SECTION("float 32") {
checkVariant(1.25, "\xCA\x3F\xA0\x00\x00");
checkVariant(9.22337204e+18f, "\xca\x5f\x00\x00\x00");
}
SECTION("float 64") {

4
extras/tests/Numbers/CMakeLists.txt
View File

@@ -2,6 +2,9 @@
# Copyright © 2014-2022, Benoit BLANCHON
# MIT License
set(CMAKE_CXX_STANDARD 11)
set(CMAKE_CXX_STANDARD_REQUIRED OFF)
add_executable(NumbersTests
convertNumber.cpp
parseFloat.cpp
@@ -10,7 +13,6 @@ add_executable(NumbersTests
parseNumber.cpp
)
add_test(Numbers NumbersTests)
set_tests_properties(Numbers

58
extras/tests/Numbers/convertNumber.cpp
View File

@@ -75,4 +75,62 @@ TEST_CASE("canConvertNumber<TOut, TIn>()") {
CHECK((canConvertNumber<uint16_t, uint8_t>(128)) == true);
CHECK((canConvertNumber<uint16_t, uint8_t>(255)) == true);
}
SECTION("float -> int32_t") {
CHECK((canConvertNumber<int32_t, float>(0)) == true);
CHECK((canConvertNumber<int32_t, float>(-2.147483904e9f)) == false);
CHECK((canConvertNumber<int32_t, float>(-2.147483648e+9f)) == true);
CHECK((canConvertNumber<int32_t, float>(2.14748352e+9f)) == true);
CHECK((canConvertNumber<int32_t, float>(2.14748365e+9f)) == false);
}
SECTION("double -> int32_t") {
CHECK((canConvertNumber<int32_t, double>(0)) == true);
CHECK((canConvertNumber<int32_t, double>(-2.147483649e+9)) == false);
CHECK((canConvertNumber<int32_t, double>(-2.147483648e+9)) == true);
CHECK((canConvertNumber<int32_t, double>(2.147483647e+9)) == true);
CHECK((canConvertNumber<int32_t, double>(2.147483648e+9)) == false);
}
SECTION("float -> uint32_t") {
CHECK((canConvertNumber<uint32_t, float>(0)) == true);
CHECK((canConvertNumber<uint32_t, float>(-1.401298e-45f)) == false);
CHECK((canConvertNumber<uint32_t, float>(4.29496704e+9f)) == true);
CHECK((canConvertNumber<uint32_t, float>(4.294967296e+9f)) == false);
}
#if ARDUINOJSON_HAS_LONG_LONG
SECTION("float -> int64_t") {
CHECK((canConvertNumber<int64_t, float>(0)) == true);
CHECK((canConvertNumber<int64_t, float>(-9.22337204e+18f)) == true);
CHECK((canConvertNumber<int64_t, float>(9.22337149e+18f)) == true);
CHECK((canConvertNumber<int64_t, float>(9.22337204e+18f)) == false);
}
SECTION("double -> int64_t") {
CHECK((canConvertNumber<int64_t, double>(0)) == true);
CHECK((canConvertNumber<int64_t, double>(-9.2233720368547758e+18)) == true);
CHECK((canConvertNumber<int64_t, double>(9.2233720368547748e+18)) == true);
CHECK((canConvertNumber<int64_t, double>(9.2233720368547758e+18)) == false);
}
SECTION("float -> uint64_t") {
CHECK((canConvertNumber<uint64_t, float>(0)) == true);
CHECK((canConvertNumber<uint64_t, float>(-1.401298e-45f)) == false);
CHECK((canConvertNumber<uint64_t, float>(1.844674297419792384e+19f)) ==
true);
CHECK((canConvertNumber<uint64_t, float>(1.8446744073709551616e+19f)) ==
false);
}
SECTION("double -> uint64_t") {
CHECK((canConvertNumber<uint64_t, double>(0)) == true);
CHECK((canConvertNumber<uint64_t, double>(-4.94065645841247e-324)) ==
false);
CHECK((canConvertNumber<uint64_t, double>(1.8446744073709549568e+19)) ==
true);
CHECK((canConvertNumber<uint64_t, double>(1.8446744073709551616e+19)) ==
false);
}
#endif
}

75
src/ArduinoJson/Numbers/convertNumber.hpp
View File

@@ -15,6 +15,7 @@
#endif
#include <ArduinoJson/Numbers/Float.hpp>
#include <ArduinoJson/Numbers/FloatTraits.hpp>
#include <ArduinoJson/Polyfills/limits.hpp>
#include <ArduinoJson/Polyfills/type_traits.hpp>
@@ -95,17 +96,83 @@ canConvertNumber(TIn value) {
return value <= TIn(numeric_limits<TOut>::highest());
}
// float -> int32
// float -> int64
// float32 -> int8
// float32 -> int16
// float64 -> int32
template <typename TOut, typename TIn>
typename enable_if<is_floating_point<TIn>::value &&
!is_floating_point<TOut>::value,
typename enable_if<is_floating_point<TIn>::value && is_integral<TOut>::value &&
sizeof(TOut) < sizeof(TIn),
bool>::type
canConvertNumber(TIn value) {
return value >= numeric_limits<TOut>::lowest() &&
value <= numeric_limits<TOut>::highest();
}
// float32 -> int32
template <typename TOut, typename TIn>
typename enable_if<is_floating_point<TIn>::value && sizeof(TIn) == 4 &&
is_integral<TOut>::value && is_signed<TOut>::value &&
sizeof(TOut) == 4,
bool>::type
canConvertNumber(TIn value) {
return value >= numeric_limits<TOut>::lowest() && value <= 2.14748352e+9f;
}
// float32 -> uint32
template <typename TOut, typename TIn>
typename enable_if<is_floating_point<TIn>::value && sizeof(TIn) == 4 &&
is_integral<TOut>::value && is_unsigned<TOut>::value &&
sizeof(TOut) == 4,
bool>::type
canConvertNumber(TIn value) {
return value >= numeric_limits<TOut>::lowest() && value <= 4.29496704e+9f;
}
// float32 -> int64
template <typename TOut, typename TIn>
typename enable_if<is_floating_point<TIn>::value && sizeof(TIn) == 4 &&
is_integral<TOut>::value && is_signed<TOut>::value &&
sizeof(TOut) == 8,
bool>::type
canConvertNumber(TIn value) {
return value >= numeric_limits<TOut>::lowest() && value <= 9.22337149e+18f;
}
// float32 -> uint64
template <typename TOut, typename TIn>
typename enable_if<is_floating_point<TIn>::value && sizeof(TIn) == 4 &&
is_integral<TOut>::value && is_unsigned<TOut>::value &&
sizeof(TOut) == 8,
bool>::type
canConvertNumber(TIn value) {
return value >= numeric_limits<TOut>::lowest() &&
value <= 1.844674297419792384e+19f;
}
// float64 -> int64
template <typename TOut, typename TIn>
typename enable_if<is_floating_point<TIn>::value && sizeof(TIn) == 8 &&
is_integral<TOut>::value && is_signed<TOut>::value &&
sizeof(TOut) == 8,
bool>::type
canConvertNumber(TIn value) {
return value >= numeric_limits<TOut>::lowest() &&
value <= FloatTraits<TIn>::forge(
0x43DFFFFF, 0xFFFFFFFF); // 9.2233720368547748e+18
}
// float64 -> uint64
template <typename TOut, typename TIn>
typename enable_if<is_floating_point<TIn>::value && sizeof(TIn) == 8 &&
is_integral<TOut>::value && is_unsigned<TOut>::value &&
sizeof(TOut) == 8,
bool>::type
canConvertNumber(TIn value) {
return value >= numeric_limits<TOut>::lowest() &&
value <= FloatTraits<TIn>::forge(
0x43EFFFFF, 0xFFFFFFFF); // 1.8446744073709549568e+19
}
template <typename TOut, typename TIn>
TOut convertNumber(TIn value) {
return canConvertNumber<TOut>(value) ? TOut(value) : 0;