[Tyagiquamar]

Problem

When compiling JsonCpp with -Wmissing-prototypes (or strict compiler flags treating missing function prototypes as errors), GCC/Clang emits a compiler error in src/lib_json/json_reader.cpp:

error: no previous prototype for 'size_t& Json::newlineScanByteCountForTesting()' [-Werror=missing-prototypes]

Fixes #1705.

Solution

Add explicit prototype declaration JSON_API size_t& newlineScanByteCountForTesting(); matching the function signature before its definition in src/lib_json/json_reader.cpp.

Verification

• Built in Docker with GCC 14 and -DCMAKE_CXX_FLAGS='-Wmissing-prototypes': compiled cleanly with zero errors/warnings on newlineScanByteCountForTesting.
• Ran full test suite in Docker: all 131 tests passed (100% pass rate).

[greptile-apps]

Confidence Score: 5/5

[Low impact] The PR appears safe to merge.

Summary

Adds a declaration for newlineScanByteCountForTesting() immediately before its definition in src/lib_json/json_reader.cpp.

• Strict builds see the newline scan counter's prototype before its definition.

_{Reviews (2) · Last reviewed commit: "Merge branch 'master' into fix-missing-p..." · Reviewed by Greptile}

[coveralls]

Coverage Report for CI Build 40347608363

Coverage remained the same at 89.926%

Details

• Coverage remained the same as the base build.
• Patch coverage: No coverable lines changed in this PR.
• No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.

---

Coverage Stats

Relevant Lines:	2740
Covered Lines:	2615
Line Coverage:	95.44%
Relevant Branches:	2640
Covered Branches:	2223
Branch Coverage:	84.2%
Branches in Coverage %:	Yes
Coverage Strength:	23835.16 hits per line

---

💛 - Coveralls