Include <cstdint> in json_value.h for uint8_t #45

Merged
andrew merged 1 commits from weaselbot/weaseljson:weaselbot/issue-37 into main 2026-06-29 18:57:42 +00:00
Member

Closes #37

src/json_value.h uses uint8_t in escapeAsJsonString but did not include <cstdint>, so any translation unit that included this header without first including <cstdint> failed to compile. Add the missing include so the header is self-contained.

Closes #37 `src/json_value.h` uses `uint8_t` in `escapeAsJsonString` but did not include `<cstdint>`, so any translation unit that included this header without first including `<cstdint>` failed to compile. Add the missing include so the header is self-contained.
weaselbot added 1 commit 2026-06-29 18:04:12 +00:00
Include <cstdint> in json_value.h for uint8_t
CI / build (-DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++, clang-arm64, ubuntu-latest-arm64, true) (pull_request) Successful in 52s
CI / build (-DCMAKE_C_COMPILER=gcc -DCMAKE_CXX_COMPILER=g++, gcc-arm64, ubuntu-latest-arm64, false) (pull_request) Successful in 50s
CI / pre-commit (pull_request) Successful in 53s
CI / build (-DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++, clang-amd64, ubuntu-latest-amd64, true) (pull_request) Successful in 1m29s
CI / build (-DCMAKE_C_COMPILER=gcc -DCMAKE_CXX_COMPILER=g++, gcc-amd64, ubuntu-latest-amd64, false) (pull_request) Successful in 1m24s
e5c970a605
`escapeAsJsonString` uses `uint8_t` but the header did not include
`<cstdint>`, making it dependent on other headers to define the type.
Add the missing include so `json_value.h` is self-contained.
andrew scheduled this pull request to auto merge when all checks succeed 2026-06-29 18:38:16 +00:00
andrew merged commit 4bd1088018 into main 2026-06-29 18:57:42 +00:00
andrew deleted branch weaselbot/issue-37 2026-06-29 18:57:42 +00:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: weaselab/weaseljson#45