amorfo77/json
1
0
Fork 0
mirror of https://github.com/nlohmann/json.git synced 2026-09-30 03:30:31 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix CI: resolve clang-tidy findings in the stream position tests

Browse Source 
#5344 added two lines to unit-deserialization.cpp that clang-tidy
reports: modernize-return-braced-init-list for the remaining() helper
and readability-isolate-declaration for "json j1, j2, j3;".

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
This commit is contained in:
Niels Lohmann
2026-09-25 18:04:53 +02:00
parent 465407f3ce
commit b6db36adb4
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/src/unit-deserialization.cpp
+5 -3
View File
@@ -1240,9 +1240,9 @@ TEST_CASE("deserialization")
// the stream is left one byte too far after a number (and only after a // the stream is left one byte too far after a number (and only after a
// number). JSON_PRECISE_STREAM_POSITION changes this; see // number). JSON_PRECISE_STREAM_POSITION changes this; see
// unit-precise-stream-position.cpp. These checks pin the default. // unit-precise-stream-position.cpp. These checks pin the default.
const auto remaining = [](std::istream & is) const auto remaining = [](std::istream & is) -> std::string
{ {
return std::string(std::istreambuf_iterator<char>(is), std::istreambuf_iterator<char>()); return {std::istreambuf_iterator<char>(is), std::istreambuf_iterator<char>()};
}; };
SECTION("the character after a number is consumed") SECTION("the character after a number is consumed")
@@ -1266,7 +1266,9 @@ TEST_CASE("deserialization")
SECTION("comma-separated numbers can be read one by one") SECTION("comma-separated numbers can be read one by one")
{ {
std::istringstream ss("1,2,3"); std::istringstream ss("1,2,3");
json j1, j2, j3; json j1;
json j2;
json j3;
ss >> j1 >> j2 >> j3; ss >> j1 >> j2 >> j3;
CHECK(j1 == 1); CHECK(j1 == 1);
CHECK(j2 == 2); CHECK(j2 == 2);