Skip to content

Properly stringify both large and small int64's when converting to json #1060

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
vladtr merged 6 commits into from
Apr 20, 2023
Merged

Properly stringify both large and small int64's when converting to json #1060

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
9501ea2
Convert small neg values to string as well
vladtr Apr 19, 2023
24f886d
updated if condition
vladtr Apr 20, 2023
27325ff
Merge branch 'main' into stringify-json-fix
vladtr Apr 20, 2023
d3a36d3
fix
vladtr Apr 20, 2023
876910a
Merge branch 'stringify-json-fix' of github.com:AntelopeIO/leap into …
vladtr Apr 20, 2023
c9c84d6
adjustments
vladtr Apr 20, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion libraries/libfc/src/io/json.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -595,8 +595,9 @@ namespace fc
case variant::int64_type:
{
int64_t i = v.as_int64();
constexpr int64_t max_value(0xffffffff);
if( format == json::output_formatting::stringify_large_ints_and_doubles &&
i > 0xffffffff )
(i > max_value || i < -max_value))
os << '"'<<v.as_string()<<'"';
else
os << i;
Expand Down
21 changes: 21 additions & 0 deletions libraries/libfc/test/io/test_json.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,27 @@ BOOST_AUTO_TEST_CASE(to_string_test)
BOOST_CHECK_NO_THROW(length_exception_in_mid_str = json::to_string( v, fc::time_point::maximum(), json::output_formatting::stringify_large_ints_and_doubles, json::max_length_limit));
BOOST_CHECK_EQUAL(length_exception_in_mid_str, "\"" + json_test_util::repeat_chars + "\"");
}
{
variant v(4294967296); // 0xFFFFFFFF + 1
std::string large_int = json::to_string( v, fc::time_point::maximum(), json::output_formatting::stringify_large_ints_and_doubles, json::max_length_limit);
BOOST_CHECK_EQUAL(large_int, "\"4294967296\"");

variant v1(4294967295); // 0xFFFFFFFF
std::string normal_int = json::to_string( v1, fc::time_point::maximum(), json::output_formatting::stringify_large_ints_and_doubles, json::max_length_limit);
BOOST_CHECK_EQUAL(normal_int, "4294967295");

variant v2(-4294967296);
std::string large_int_neg = json::to_string( v2, fc::time_point::maximum(), json::output_formatting::stringify_large_ints_and_doubles, json::max_length_limit);
BOOST_CHECK_EQUAL(large_int_neg, "\"-4294967296\"");

variant v3(-4294967295);
std::string normal_int_neg = json::to_string( v3, fc::time_point::maximum(), json::output_formatting::stringify_large_ints_and_doubles, json::max_length_limit);
BOOST_CHECK_EQUAL(normal_int_neg, "-4294967295");

variant v4(-90909090909090909);
std::string super_neg = json::to_string( v4, fc::time_point::maximum(), json::output_formatting::stringify_large_ints_and_doubles, json::max_length_limit);
BOOST_CHECK_EQUAL(super_neg, "\"-90909090909090909\"");
}
}
{ // to_string( const variant& v, const yield_function_t& yield, output_formatting format = stringify_large_ints_and_doubles);
const variant v(json_test_util::repeat_chars);
Expand Down